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

Add text alternative to inline frames #8392

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

lyssaprince
Copy link
Contributor

@lyssaprince lyssaprince commented Feb 28, 2025

Summary

Adds title to inline frames (YouTube videos, Google Maps, and SlideShare)

Preview

Link to Preview

Solution

Provide a summary of the solution this PR offers.

How To Test

See list of pages that were fixed.

  1. Run Siteimprove checker to ensure that "Inline frame missing a text alternative" error isn't occurring.
  2. Check accessibility properties to ensure that accessible name describes the content either using a tool like ANDI or going into Dev Tools.

Copy link

🔍 Preview in Federalist

@nick-mon1 nick-mon1 self-requested a review March 3, 2025 15:47
@lyssaprince lyssaprince requested review from bonnieAcameron and removed request for nick-mon1 March 7, 2025 23:37
@lyssaprince lyssaprince changed the title Add title to inline frames Add text alternative to inline frames Mar 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants