The best time to register for DrupalCon Dublin is now. Earlybird discounts end July 29.
I think the title explains all.
Bump for this. Really need the ability to set the alt tag based on video attributes (at least default to video title for now so html can validate). This should also work on views.
Thanks for the great work!
I'll second this. For accessibility purposes, this is a must.
I'm changing the title. Ultimately, Alt tags are more useful for accessibility than titles. There is a strong precedent for this in Core.
Simple patch to add this feature.
Thank you for your patch. It solves the problem fine in case you select the thumbnail formatter, but would be great if you could also gice this support in case you enable the colorbox module and select the 'Thumbnail Preview w/Colorbox' formatter.
Then you should play with the video_embed_field_colorbox_code theme callback to add there a new #image_alt property.
If you need some feedback, please ping me on IRC
Thanks @plopesc :-)
I think that now is ok.
Hey @diego21, grea twork here
Your patch looks pretty good. I just have a question about this line:
@@ -452,6 +454,7 @@ function video_embed_field_field_formatter_view($entity_type, $entity, $field, $
+ '#alt' => $alt,
I think this line is not necessary here, video_embed_field_embed_code theme callback is related to the video player, and we are dealing with image thumbnails in this patch.
Could you confirm me that this line is not used anywhere and could be removed?
You are right, that line is not necessary. Indeed, the hook_theme() wasn't difining that variable for video_embed_field display :-)
Thanks for the comment.
Issue #1926872 by diego21: Add ALT attributes to thumbnail
Committed and pushed to 7.x-2.x
Thank you @diego21!!
Automatically closed - issue fixed for 2 weeks with no activity.
Drupal is a registered trademark of Dries Buytaert.