Closed (fixed)
Project:
Flickr
Version:
6.x-1.x-dev
Component:
flickr (main module)
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
22 May 2007 at 20:57 UTC
Updated:
16 Jun 2007 at 05:16 UTC
Jump to comment: Most recent file
Comments
Comment #1
drewish commentedhumm, not sure how worth while the longdesc would be... it's supposed to be a URL so we'd end up having to add a callback that took a photo id and fetched the description... do-able but kind of clunky. i think you could put it into the alt or title tags in a theme function though.
Comment #2
andrewlevine commentedI think drewish is right that the longdesc would add little value to most users and have an inelegant implementation.
However, I think not having working alt tags is a bug. I'm not sure why it isn't working but I'll try to have a patch sometime tomorrow.
Comment #3
Rebecca@category.greenash.net.au commentedYou're right about "longdesc." Why did I not know it was a URL? I just mentioned it as an option because I'm a bit long-winded. ;-)
Anyway, yeah, "alt" is the right place for that data to show up.
Looking forward to that patch! Thanks, y'all!
Comment #4
andrewlevine commentedSorry this patch took so long.
We actually did have this functionality in the module, it was just broken because we were using the wrong variable name. I went ahead and fixed a couple other places where we were using non-existent variables.
I had to re-save my node to get the alt tags to show up for photos displayed by flickr_filter.
Comment #5
andrewlevine commentedComment #6
drewish commentedis that $format value not defined? the other stuff looks good.
Comment #7
andrewlevine commented$format was not defined or mentioned anywhere else in the flickr_img function. I'm not sure when it was taken out or even if it was ever there...
Comment #8
drewish commentedcool, yeah that's one that been slowly dieing off.
Comment #9
andrewlevine commentedCommitted, thanks for the report Rebecca.
Comment #10
(not verified) commented