fix: a11y - adding titles to the video iframes #5346
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hey, just a quick accessibility fix to add titles to the video iFrames on the videos page, according to this rule:
https://dequeuniversity.com/rules/axe/4.7/frame-title
I just used the titles of the videos, figured that made the most sense :). (I'm not quite sure how all screen readers might handle
"
so I just removed the quotes)