Closed (fixed)
Project:
Lightbox2
Version:
5.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Support request
Assigned:
Unassigned
Reporter:
Created:
8 Jan 2008 at 06:13 UTC
Updated:
13 Feb 2008 at 14:01 UTC
Jump to comment: Most recent file
Comments
Comment #1
stella commentedWhat version of Acidfree are you using? What browser(s) are you using and on what platform(s)?
In what way does lightbox stop working? Does it crash or give any errors? Does the image open on a white page? Or does the lightbox appear but the image never loads? Or does it bring you to the node page when clicked on?
What are your lightbox2 settings? Are you using lightbox2 lite? Is automatic image handling enabled? etc.
Cheers,
Stella
Comment #2
artis commentedMy installation of Lightbox V2 5.x-2.2 is working with Acidfree 5.x-1.x-dev except for the Galleries option. When I check "Disable Lightbox for Gallery Lists" then nothing changes. If I click on an acidfree album thumbnail it opens the thumbnail in Lightbox2. Any ideas?
BTW.. tested in both Firefox 2 and IE7.
Comment #3
stella commentedThat option (disable for gallery lists) only applies to image galleries created by the "image gallery" module. I'll update the help text to make this clearer. You might want to consider the "disable lightbox for these urls" setting for your acidfree galleries.
Cheers,
Stella
Comment #4
artis commentedFYI: I made it work by hacking /js/image_nodes.js
Line 18:
if (!settings.disable_for_gallery_lists || (settings.disable_for_gallery_lists && !$(this).parents(".galleries").length)) {I changed ".galleries" to ".acidfree-folder"
if (!settings.disable_for_gallery_lists || (settings.disable_for_gallery_lists && !$(this).parents(".acidfree-folder").length)) {Bet it wouldn't be too hard to make a checkbox in the Lightbox settings for Acidfree galleries.
Artis
Comment #5
stella commentedComment #6
stella commentedTry the attached patch.
Cheers,
Stella
Comment #7
artis commentedNope. That doesn't work. I was using 5.x-2.2 and the patch won't apply to it. I did download the 5.x-2.x-dev since I saw it has the checkbox, but it doesn't work either. The new "Image trigger size:" option seems to be interfering. If I don't select thumbnail then lightbox doesn't work at all. If I select thumbnail then the Acidfree album opens in lightbox when I click it.
Thanks for the try. I'll go back to using my hack. I'll look back here to see if you have another patch I can try later.
Artis
Comment #8
artis commentedI just tried replacing that same line again on the 5.x-2.x-dev code with my hack above and it works. Go figure. I'll look and see if I can figure out what it wrong with that line and let you know.
Artis
Comment #9
artis commentedStill don't know what's wrong with this line 18 of /js/image_node.js:
if ((!settings.disable_for_gallery_lists && !settings.disable_for_acidfree_gallery_lists) || (settings.disable_for_gallery_lists && !$(this).parents(".galleries").length) || (settings.disable_for_acidfree_gallery_lists && !$(this).parents(".acidfree-folder").length)) {But this works:
line 57 add:
}You also have to uncheck "Disable Lightbox for Gallery Lists".
and check "Disable Lightbox for Acidfree Gallery Lists"
Hope that helps.
Artis
Comment #10
stella commentedI've just modified the javascript again. Please try out the latest dev version. I've changed the check to be:
Cheers,
Stella
Comment #11
stella commentedReleased in 5.x-2.3 and 6.x-1.3.
Cheers,
Stella
Comment #12
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.