Closed (fixed)
Project:
YouTube Field
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Reporter:
Created:
25 Jun 2015 at 00:31 UTC
Updated:
13 Aug 2015 at 16:24 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
plazma commentedPatch to provide accessibility text
Comment #2
plazma commentedComment #3
plazma commentedComment #4
guschilds commentedHi plazma,
Thanks for the patch!
In Example 2 in the link you provided, the alternative content is a link. That link's
hrefmatches the iframe'ssrcand the link's text matches the iframe'stitleattribute value. I think it's best to follow that example here because:I've attached a patch that does so. Please have a look and let me know what you think.
Thanks,
Gus
Comment #7
guschilds commentedI've committed the patch from #4 to the 7.x-1.x branch and it will be a part of a future 7.x-1.7 release. I've also committed an equivalent to the 8.x-1.x branch. Thanks again for the suggestion and the initial patch.