Posted by bcobin on November 7, 2008 at 10:47pm
Jump to:
| Project: | Blue Droplet Video |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | jbrown |
| Status: | closed (fixed) |
Issue Summary
Have I told you guys (or girls) recently how much I love this module?
One small point: it seems that the module doesn't generate an "alt" tag for the thumbnail. Unfortunately, that breaks validation.
Let me know if there's a quick fix for this - otherwise, congrats on a brilliant module - it's the best out there. (I won't mention any names...)
Keep up the great work!
Comments
#1
Easy - figured it out. I just inserted the following at line 125 of preview_image.inc:
$parts[] = 'alt=""';Front page now validates. Woo hoo! Take a look here.
Open Package video rocks! Great work...
#2
Cool, I hope it will be added in new releases as it's always important to be valid :)
#3
#4
http://drupal.org/cvs?commit=160292
#5
But now is going to be blank, right? What about adding the title text into the alt tag...??
#6
I just set it to "Video preview image". In the future we could also include the node title.
#7
Automatically closed -- issue fixed for two weeks with no activity.
#8
The alt and title attributes in the img tag now use the node title.
http://drupal.org/cvs?commit=316032
#9
Automatically closed -- issue fixed for 2 weeks with no activity.