Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cannot modify <iframe> title attribute on youtube videos #8991

Open
jhoude5 opened this issue Nov 30, 2020 · 4 comments
Open

Cannot modify <iframe> title attribute on youtube videos #8991

jhoude5 opened this issue Nov 30, 2020 · 4 comments

Comments

@jhoude5
Copy link

jhoude5 commented Nov 30, 2020

Using code from https://wet-boew.github.io/v4.0-ci/demos/multimedia/multimedia-youtube-en.html, The youtube if placed in an <iframe> with always the same title for any language "Youtube video player".
For screen readers, the user would not know the purpose of the video till they get to the

after the video. Especially if this is an alternative video to the page (Sign language video).

@ghost
Copy link

ghost commented Nov 30, 2020

So looks like they are using the YouTube iFrame API which is overriding it:

load: "https://www.youtube.com/iframe_api"

It looks like it possible for WET to override the iFrame API as there is access to the video title:

title = $media.attr( "title" ) || "",

I am just a volunteer on this GitHub project.

@jhoude5
Copy link
Author

jhoude5 commented Dec 3, 2020

I am adding the title attribute to the <video> element but the iFrame is still rendering the original Youtube title not the video title.

@ghost
Copy link

ghost commented Dec 3, 2020

As I explained the iframe is generated by YouTube. So technically speaking this is a YouTube issue. But a workaround could be add to the plug-in.

I am just a volunteer on this GitHub project.

@EricDunsworth
Copy link
Member

@jaube05 FYI I adjusted the <video> tag in #8991 (comment) to make it show up properly (<video> and everything that came after it previously weren't showing up).

Just had to replace <video> with ``<video>`` to do the trick.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants