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.

Members fund testing for the Drupal project. Drupal Association Learn more

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?

Kleve’s picture

Agree with @sheldonreed3, it should be more descriptive. At least we should give the editor the ability to do so. The title should describe the contents of the embedded content, not that it is an embedded content.

A separate title field on the edit form with the fallback text "Embedded video" if left empty maybe?

karenann’s picture

This is by no means a "fix" but I'm attaching a stop gap for Drupal 8.

Status: Needs review » Needs work

The last submitted patch, 5: video_embed_field-1888488-5.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.