Currently, the module does not support the addition of a title attribute for the iframe it generates, which is necessary for WCAG 2.0 A compliance. See https://www.w3.org/TR/WCAG20-TECHS/H64.html (particularly the related success criteria) for more details.

Patch to follow that adds this feature.

CommentFileSizeAuthor
#6 2785093--title-attr--6.patch3.91 KBalexiswatson
#4 2785093--title-attr--3.patch3.07 KBalexiswatson
#3 2785093--title-attr--2.patch5.44 KBalexiswatson
#2 2785093--title-attr--1.patch2.15 KBalexiswatson

Comments

davidwatson created an issue. See original summary.

alexiswatson’s picture

alexiswatson commented
Status: Active » Needs review
StatusFileSize
new 2785093--title-attr--1.patch2.15 KB

Patch attached. Review and feedback welcome!

alexiswatson’s picture

alexiswatson commented
StatusFileSize
new 2785093--title-attr--2.patch5.44 KB

Previous was munged, disregard.

alexiswatson’s picture

alexiswatson commented
StatusFileSize
new 2785093--title-attr--3.patch3.07 KB

Once more, with feeling...

mforbes’s picture

mforbes commented

The docblocks also want to be patched...

shortcode_video_macro_process
* <iframe width="420" height="315"

_shortcode_video_get_youtube_videoid
* <iframe width="420" height="315" src="http://www.youtube.com/embed/EJu8ihVdygY?rel=0" frameborder="0" allowfullscreen></iframe>

Other than that, looks solid.

alexiswatson’s picture

alexiswatson commented
StatusFileSize
new 2785093--title-attr--6.patch3.91 KB

Updated documentation. One more for RTBC?

andrewmacpherson’s picture

andrewmacpherson commented
Issue tags: -a11y

Tag clean up: "accessibility" is the preferred one. I know this issue is for a contrib module, but the "a11y" tag doesn't have many issues so I'm moving them all to "accessibility". This is so the core accessibility maintainers don't have to monitor two tags.

denes.szabo’s picture

denes.szabo commented

@davidwatson thanks for the patch!

denes.szabo’s picture

denes.szabo commented
Status: Needs review » Fixed
denes.szabo’s picture

denes.szabo commented
Status: Fixed » Closed (fixed)