iframes right now have no indication of what the content is. For screen readers, this can be problematic, as the only information they get is that it's an iframe. Simple patch below, just adds title="Embedded video" to the youtube and vimeo handlers.

Files: 

Comments

wbobeirne’s picture

Status: Active » Needs review
FileSize
1.41 KB
rsmylski’s picture

Issue summary: View changes
FileSize
1.73 KB

re-rolled patch against latest 7.x-2.x code

sheldonreed3’s picture

The patch seems to work as designed. While it addresses the issue, I think we should have an option to make this title more descriptive/definable, or grab it from the video. If there is interest I can roll some options around in my spare time. Any thoughts?