I had a problem with the current version of JLightbox, where clicking on an Image thumbnail the lightbox "version" would popup and close as expected but when clicked a second time, the resulting popup image would NOT close (and only a page refresh would bring the original page back).
I've noticed that the current JLightbox module is build against the 0.2 version of the jQuery Lightbox from waren. I tried to:
- Substitute the jlightbox.js (provided with the JLightbox module) with the lightbox.js of the 0.3 version of Jquery Lightbox from waren (http://warren.mesozen.com/category/jquery/)
- change line 45 of jlightbox.module to load lightbox.js instead of jlightbox.js.
- change the references to the images in ligthbox.js:
- fileLoadingImage : "/sites/all/modules/jlightbox/images/loading.gif",
- fileBottomNavCloseImage:'/sites/all/modules/jlightbox/images/closelabel.gif
but unfortunately, although the original problem goes away (repeated open and close of the same image thumbnail works fine) now I am lacking the close image button at the bottom of the popped up image.
Bottom line is that further work must be done in order to update to the 0.3 version of jQuery Lightbox but I am at a loss on where to further change it.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | lightbox.js_.txt | 11.16 KB | stratosgear |
Comments
Comment #1
stratosgear commentedOK,
It seems that warren's 0.3 version of lightbox.js is using hardcoded image paths for the loading and close image icons. I am including a version with the correct image paths.
NOTE: I do not really know what other repercussions exist by changing the original jlightbox.js file with this lightbox.js version (so far works fine with me) but your mileage might vary. Maybe the owners of the project might want to have a better look.
Comment #2
sunThis should be fixed now. Please update to the latest version 5.x-1.1, which is basically identical (but improved) to Warren's v0.3.
If this bug still exists, feel free to re-open this issue.
Comment #3
(not verified) commented