Hello
I installed Brilliant Gallery and then Lightbox V2 (dev), everything worked and I found it very very great, until I saw that the Lightbox V2 (dev) breaks all ajax requests!
So I guess Brilliant Gallery doesn't work at all on D7? :/

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Vacilando’s picture

Title: D7 needs Lightbox V2 but can't » D7 should not rely only on Lightbox2
Version: 7.x-1.0 » 7.x-1.x-dev
Assigned: Unassigned » Vacilando
Priority: Normal » Major

Agreed; BG should not depend on just one overlay system.

I am planning to support Colorbox very soon. Any other candidates you would recommend?

Of course, if somebody posts a working patch, I can apply it within a day!

Leeteq’s picture

Title: D7 should not rely only on Lightbox2 » Brilliant Gallery should support more image display modules, not only Lightbox
Category: bug » feature

Any update on the colorbox support?

Anonymous’s picture

"...Colorbox very soon" ... Any progress on that? Colorbox performs so much better than Lightbox2.

mike1966’s picture

FileSize
2.68 KB

Patch to support Colorbox
Not sure if this is the right way to do, but there are only small changes, they could be done by hand.

mike1966’s picture

FileSize
2.68 KB
robertstaddon’s picture

I'm really glad you're adding Colorbox support for this module! The patch is working well! Can you integrate this into the next version?

Anthony Fok’s picture

Reuploading mike1996's excellent patch with the .patch extension, and hope the Drupal issue tracker will pick it up as such and test it. :-)

Anthony Fok’s picture

Status: Active » Needs review
Anthony Fok’s picture

Title: Brilliant Gallery should support more image display modules, not only Lightbox » Adding Colorbox support to Brilliant Gallery
Anthony Fok’s picture

Status: Needs review » Reviewed & tested by the community
Vacilando’s picture

Issue summary: View changes

Thanks to the effort. Support for Colorbox is now committed to DEV, although I used my own code. Also removed support for Thickbox and Greybox -- those modules have no D7 support. Plus some code cleaning. Please test.

Vacilando’s picture

Assigned: Vacilando » Unassigned
Status: Reviewed & tested by the community » Fixed

Support for Colorbox committed to 7.x-1.8!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.