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.

Sign up for GitHub

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

Jump to bottom

(Accessibility) APM - euiTabs missing required ARIA child roles #80599

Closed
barlowm opened this issue Oct 14, 2020 · 3 comments
Closed

(Accessibility) APM - euiTabs missing required ARIA child roles #80599

barlowm opened this issue Oct 14, 2020 · 3 comments
Assignees
@smith
Labels
loe:small Small Level of Effort Project:Accessibility Team:APM All issues that need APM UI Team support v7.11.0 WCAG A

Comments

@barlowm
Copy link
Collaborator

barlowm commented Oct 14, 2020

The main euiTabs component on the APM page uses the ARIA role "tablist" which requires the "tab" role to be a child of the "tablist" element. There are no child elements containing the "tab" role.
See Example of Tabs with Automatic Activation

image

Kibana Version:
7.8

OS:
Windows 7

Browser:
Chrome

Screen reader: [if relevant]
NVDA

Relevant WCAG Criteria: WCAG Criterion - Level A/AA/AAA
1.3.1 Info and Relationships Level A

Relevant ARIA spec: WAI-ARIA Authoring Practices
Accessible Rich Internet Applications (WAI-ARIA) 1.1 - The Roles Model
@elasticmachine
Copy link
Contributor

Pinging @elastic/eui-design (EUI)

@myasonik myasonik added the loe:small Small Level of Effort label Oct 20, 2020
@myasonik myasonik mentioned this issue Oct 30, 2020
Closed
@myasonik myasonik added the Team:APM All issues that need APM UI Team support label Oct 30, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/apm-ui (Team:apm)

@smith smith self-assigned this Nov 9, 2020
@smith
Copy link
Contributor

smith commented Nov 16, 2020 •
edited
Loading

This was fixed in #82870 and will be shipped in 7.11.

image

@smith smith closed this as completed Nov 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@smith smith

Labels
loe:small Small Level of Effort Project:Accessibility Team:APM All issues that need APM UI Team support v7.11.0 WCAG A
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@smith @barlowm @myasonik @elasticmachine