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

[a11y] /challenge-the-default/ page elements marked as aria-hidden are still focusable #15094

Closed
alexgibson opened this issue Sep 9, 2024 · 1 comment
Labels
a11y Issues related to accessibility Bug 🐛 Something's not working the way it should Help wanted 👋 Community contributions welcome P3 Third level priority - Nice to have

Comments

@alexgibson
Copy link
Member

alexgibson commented Sep 9, 2024

Description

There are several elements on the /firefox/challenge-the-default/ page that are marked up using aria-hidden="true" but they are still focus-able DOM elements. A focus-able element with aria-hidden="true" is ignored as part of the reading order for screen readers, but still part of the focus order, making it’s state of visible or hidden unclear.

https://dequeuniversity.com/rules/axe/4.10/aria-hidden-focus

Steps to reproduce

  1. Open https://www.mozilla.org/de/firefox/challenge-the-default/
  2. Notice there are several element on the page using aria-hidden (such as the browser close/X button in the hero), but these are still possible to tab through to using a keyboard.

Expected result

If these visible elements should be hidden from screen readers, then we should make them non-focus-able. Alternatively, if there are elements we could make more accessible then we could remove aria-hidden.

Actual result

image

Environment

N/A

@alexgibson alexgibson added Bug 🐛 Something's not working the way it should P3 Third level priority - Nice to have a11y Issues related to accessibility Help wanted 👋 Community contributions welcome labels Sep 9, 2024
@alexgibson
Copy link
Member Author

Duplicate of #15137

@alexgibson alexgibson marked this as a duplicate of #15137 Sep 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a11y Issues related to accessibility Bug 🐛 Something's not working the way it should Help wanted 👋 Community contributions welcome P3 Third level priority - Nice to have
Projects
None yet
Development

No branches or pull requests

1 participant