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

(Accessibility) Management heading structure #51493

Closed
rayafratkina opened this issue Nov 22, 2019 · 2 comments
Closed

(Accessibility) Management heading structure #51493

rayafratkina opened this issue Nov 22, 2019 · 2 comments

Comments

@rayafratkina
Copy link
Contributor

Per #37539, we'll want to audit all of the Management screens and ensure we're using headers appropriately.

Currently known issues from Security and Spaces audit

  • Main landmark must not be contained in another landmark (must be fixed by Management app)
  • Document must have 1 main landmark (must be fixed by Management app)
  • Ensure landmarks are unique (must be fixed by Management app)

Relevant WCAG Criteria: [#.#.# WCAG Criterion - Level A/AA/AAA](link to https://www.w3.org/WAI/WCAG21/quickref/)

Relevant ARIA spec: [](link to https://www.w3.org/TR/wai-aria-practices/)

@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app-arch (Team:AppArch)

@mattkime mattkime self-assigned this Nov 22, 2019
@myasonik
Copy link
Contributor

Looks like the landmarks and headings were largely fine. The one Management specific issue I noticed was a duplication of navigation landmarks.

Adding a (screen reader only) heading (probably an h2) and a mapping aria-labelledby to the management specific navigation would make that UI a little easier to navigate.

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