Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Just upgraded to 1.10 and no longer have a newline between the links under the image. I'm using Lightbox2 as slideshow with the "Gallery Assist" module. You can see an example here.
Comment | File | Size | Author |
---|---|---|---|
#14 | gallery_assist_module_copyright_symbol.patch | 620 bytes | jcmc |
Comments
Comment #1
stella CreditAttribution: stella commentedI can't reproduce this locally with imagefields, image, etc, but make sure you ran update.php after updating the module. I don't have a Gallery Assist installation to test with. However Gallery Assist integration is not supported by the Lightbox2 module - the integration is done within Gallery Assist, so if there is a problem with that module and no others, you should re-assign this issue to that issue queue.
Comment #2
roball CreditAttribution: roball commentedOK, I have assigned the issue to Gallery Assist's queue. I had no problems with previous versions of Lightbox2 - after updating it to 6.x-1.10 this problem occured. I did run the DB update.
Comment #3
jcmc CreditAttribution: jcmc commentedHello roball,
I think the right place is GA-Lightboxes. I will try to reproduce this issue and investigate which are the causes.
Regards
Juan Carlos
Comment #4
jcmc CreditAttribution: jcmc commentedComment #5
jcmc CreditAttribution: jcmc commentedHello roball,
Here the results of my investigation. You are right, after I have updated to lb2-1.10 I have the same issue as you.
The lightbo2 1.19 respect the send or formated data from others modules. I case of GA-Lightboxes are text as title, caption and download link.
The lb2 filter the html code, why? I don't know. Stella will have their reasons.
http://assist-series.com/content/ga-lb-or-lightbox2-issue/gallery-assist...
http://assist-series.com/content/ga-lb-or-lightbox2-issue/gallery-assist...
http://assist-series.com/content/ga-lb-or-lightbox2-issue/gallery-assist...
The causes of this issue are really the changes in the lightbox2 module. It does not matter, I find a solution for all GA users :-)
Regards
Juan Carlos
Comment #6
jcmc CreditAttribution: jcmc commenteddone
I've added an ID and a clear-block class to the download link.
the patchs are available at http://drupal.org/cvs?commit=426176.
the changes will be available in the dev version, today at 2 PM.
Regards
Juan Carlos
Comment #7
roball CreditAttribution: roball commentedThanks for the quick fix!
Will try the new Gallery Assist Lightboxes dev when it's rolled out.
Comment #8
stella CreditAttribution: stella commentedfiltering is being done to prevent XSS attacks. The next release will contain the ability to configure allowed tags - currently it's the default list from filter_xss() (see api docs).
Comment #9
jcmc CreditAttribution: jcmc commentedHi stella,
It is not so bad, I know you have your reasosns and you see, the posted issue was solved with a class and I have added a ID to increment the css formating possibilities.
I think, one thing still missing: The fields should be separated by at least one space. To avoid the results mentioned here in case equal which filter find more as one items.
For me is the issue here (bug or not) fixed.
When you add the "allowed tags" feature to the lb2, it will be much better :-) and I can remove the hard coded class.
Thanks and regards
Juan Carlos
Comment #10
roball CreditAttribution: roball commentedI have now updated Gallery Assist Lightboxes to 6.x-1.x-dev (2010-Sep-23) and the newline after the "Go to image page" link is there again - Thanks.
However, the newline after the copyright information (the first of the 3 elements displayed under the image) is still missing. Juan, please click on the example link in my initial post above to see what I mean. It would also be nice to have a "©" displayed before the actual copyright text, as on hovering over the image.
Comment #11
roball CreditAttribution: roball commentedComment #12
jcmc CreditAttribution: jcmc commentedhm,
I was "zu schnell".
the fixes and adds are committed.
Comment #13
roball CreditAttribution: roball commentedThank you very much - Problem is fixed with Gallery Assist Lightboxes 6.x-1.x-dev (2010-Sep-24), and it looks nice again. good to see the © in front of the actual copyright text. When you hover over the thumbnail, you see "(c)" instead in front of the copyright text. I know this is not related to the original report, and I can open a feature request for this if you like. I think having © in both cases gives a more robust feeling, or just looks better.
Comment #14
jcmc CreditAttribution: jcmc commentedYes I see this so as you.
Now I don't have the intention to commit right now a new release for GA or to the dev.
I less here a patch which replace the (c) from the copyright.
Comment #15
stella CreditAttribution: stella commentedThe allowed tags feature was added and released in lightbox2 6.x-1.11 earlier today.
Comment #16
roball CreditAttribution: roball commentedWhy didn't you include the patch from #14 to the recently released Gallery Assist 6.x-1.16?
Comment #17
jcmc CreditAttribution: jcmc commentedHello roball,
I can't give answer "why", I have added the patch and have published a new release. This release contain fixes to the module and install files.
I have visited your pages (Photo galleries Pretoria 2010) but as anonymous I can't see Photos, is this so correct?
If not, uncomment the commented part here and add it to your init hook.
Comment #18
roball CreditAttribution: roball commentedDear Juan Carlos,
thank you for the 6.x-1.17 release containing this improvement.
it seems that while updating 6.x-1.15 to 6.x-1.16 the Gallery Public Status on all galleries changed from "public for all" to "public". I had to reset all to "public for all" and they are again visible to anonymous users.