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: Settings headings are not marked up in HTML #24922

Closed
daniellekirkwood opened this issue Mar 24, 2023 · 4 comments
Closed

A11y: Settings headings are not marked up in HTML #24922

daniellekirkwood opened this issue Mar 24, 2023 · 4 comments

Comments

@daniellekirkwood
Copy link
Contributor

In all submenu items in the web Settings screen, the headings are not marked up in HTML which makes it difficult for screen readers to navigate

Example of this issue are highlighted in red:
Screenshot 2023-03-24 at 11 48 38

@MadLittleMods
Copy link
Contributor

What about these A11y issues requires https://github.com/vector-im/element-web/labels/X-Needs-Design? Are the screens/flows changing beyond making the existing content accessible?

@daniellekirkwood
Copy link
Contributor Author

What about these A11y issues requires X-Needs-Design May require input from the design team ? Are the screens/flows changing beyond making the existing content accessible?

The Design Systems team marked this as something that they would likely fix as part of the refit they're working on this year.

@MadLittleMods
Copy link
Contributor

@daniellekirkwood Feels like this can be addressed without any design input.

If we want to fix it in the current Element, just update to use some semantic heading elements.

If we want to make sure headings in the Compound design system are accessible, just use semantic heading elements.

@luixxiul
Copy link

luixxiul commented Apr 29, 2023

@kerryarchibald I would like to ask you if the issues you referenced to this are also to address #25227.

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

4 participants