The colorbox slideshow is broken on my system since the version 2012-May-04

I'll need a bit more information than this, how many pictures do you have uploaded on there? Are any JS error messages showing up in the firebug js console? Also, what settings do you have for galleryformatter?

Looks like you're setup is not picking up all the settings (you can now choose how you link to the full image). Have you tried clearing the cache after upgrading?

Yes this is broken on mine as well, was driving me crazy for a while. The colorbox modal is never picked up in the latest dev version when trying to view an image, the image is displayed as a file. This still works with lightbox2 and shadowbox.

The 1.0 version doesn't work for me (image title issues) and the current dev doesn't work with colorbox. Good thing I had an old dev version laying around, it's attached if anyone else is having these problems.

The same happens here: The Colorbox was not showing after upgrding to the latest version of 8-may-2012.
I resaved the Formatter settings and switched from and to .

These settings work fine, and show the Colorbox (but i do not want 'on slide click'):

Linking to full image
Full image style: None (original image)
Modal used for full image: colorbox
Linking method:<strong> On slide click</strong>

These settings do NOT show Colorbox (and this is what I like: a nice Hourglass):

Slides style: galleryformatter_slide
Full image style: None (original image)
Modal used for full image: colorbox
Linking method:<strong> Using a link</strong>

better title

Thank you @johnv for the very useful information. I can now duplicate and troubleshoot this bug. Working on it.

OK, this was an easy fix once we knew in what situation it was failing...

Again, thanks johnv and everyone else for reporting.

Committed: http://drupalcode.org/project/galleryformatter.git/commit/2afed70

Ow, an invisible typo - hard to find. Glad it is solved.

For some reason this was committed, but didn't make it to the 7.x-1.x-dev download on the project page.

the change is in the tarball johnv... is it stil broken you mean?

It is not in the 'Development releases' of the project page:

Version	 Downloads	 Date	 Links
7.x-1.x-dev	tar.gz (40.57 KB) | zip (43.67 KB)	 2012-Jun-14	Notes

So this is fixed on my installs...

@johnv let me know if you stil have trouble with the latest (make sure to update)

Hi Manuel,
the 7.x-1.x-dev .zip archive of 2012-Jul-20 is still broken (didn't test the .tar, but I suppose they are the same).
the same goes for 7.x-1.1 2012-Jul-13

Please do a download of the .zip file of 20-july. You will find this line in the .module file, still containing the space:

         $link_attributes['class'] .= ($settings['linking_method'] == 'onclick_full') ? 'colorbox' : ' colorbox';
 

I didn't take a closer look, but I guess none of the latest commits are published on the project page. there is a problem with the git-tagging of your branches.

Indeed that line is exactly like you pasted in the latest release, which is correct - that is the result of what the patch did, take a look:
http://drupalcode.org/project/galleryformatter.git/blobdiff/a5cc96902175...

I just retested both ways of linking to full, using colorbox and they both work.

OK, now I am confused. I didn't realize that this patch switched the spaces, instead of removing it.
I removed both spaces, and it works fine for me, too.
I have no access to a system right now, but given the fact that you already have 277 1.0 releases out there, it MUST be solved. (277 happy users can't be wrong :-) )

Release Jul 8, 2012 Jul 1, 2012 Jun 24, 2012 Jun 17, 2012
7.x-1.x-dev 565 578 591 607
7.x-1.1 277 0 0 0

A space is necessary to concatenate with the 'shownext' class:

      if ($settings['linking_method'] !== 'onclick_full') {
        $link_attributes['class'] = 'shownext';
      }

If you need to remove the space in both cases here:
$link_attributes['class'] .= ($settings['linking_method'] == 'onclick_full') ? 'colorbox' : ' colorbox';

I'd like to know what settings you're using so that we can double check with those... I've tried combinations, they all seem to work as it is now...

You are right. This works flawlessly.