Skip to content

fix: update heading 1#418

Open
courtneylinder wants to merge 1 commit intomainfrom
clinder/gh-409/er-heading1
Open

fix: update heading 1#418
courtneylinder wants to merge 1 commit intomainfrom
clinder/gh-409/er-heading1

Conversation

@courtneylinder
Copy link
Contributor

Problem:

Heading 1 on escape room homepage didn't match visual text.

Solution:

Update the visual text to be the heading 1.

@netlify
Copy link

netlify bot commented Oct 16, 2025

Deploy Preview for accessiblecommunity ready!

Name Link
🔨 Latest commit 24b3105
🔍 Latest deploy log https://app.netlify.com/projects/accessiblecommunity/deploys/68f05e56746b850008c1046c
😎 Deploy Preview https://deploy-preview-418--accessiblecommunity.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@rachaelbradley
Copy link
Contributor

This approach is currently consistent with the approach taken on the rest of our site. The approach is not a failure and does provide support for screen reader users who navigate using the heading structure. That said, as noted, it can cause confusion for sighted screen reader users. We need a much wider conversation about the tradeoffs of various approaches before making a decision and then that decision should be applied throughout the site.

@courtneylinder
Copy link
Contributor Author

This approach is currently consistent with the approach taken on the rest of our site. The approach is not a failure and does provide support for screen reader users who navigate using the heading structure. That said, as noted, it can cause confusion for sighted screen reader users. We need a much wider conversation about the tradeoffs of various approaches before making a decision and then that decision should be applied throughout the site.

Hi Rachel!

For sighted screen reader users, do you mean it causes confusion because of the Accessible Escape room above it? Or is it because of something else?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants