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) - Uptime Broken ARIA reference #35315

Closed
barlowm opened this issue Apr 18, 2019 · 2 comments
Closed

(Accessibility) - Uptime Broken ARIA reference #35315

barlowm opened this issue Apr 18, 2019 · 2 comments

Comments

@barlowm
Copy link
Collaborator

barlowm commented Apr 18, 2019

Steps to reproduce (Use WebAIM WAVE chrome plugin)

  1. Open the Kibana/Uptime page
  2. Launch WAVE

ADB_3

They are for the various buttons and links on the page. Specifically:

  • Expand button
  • Recently viewed button
  • Discover link
  • Visualize link
  • Dashboard link
  • Canvas link
  • Maps link
  • Machine Learning link
  • Infrastructure link
  • Logs link
  • APM link
  • Uptime link
  • Graph link
  • Dev Tools link
  • Stack Monitoring link
  • Management link

These links are in the sidebar so this issue probably exists on all pages.

Each element has an aria-describedby attribute which references a random id (e.g. - aria-describedby="0nyangvc") which does not exist on the page, but changes with each load of the page.

Each button/link DOES have an aria-label which is properly read by a screen reader but the aria-describedby attribute is used to indicate the IDs of the elements that describe the object. (See "Using the aria-describedby attribute"

Expected Result
When navigating to a button/link containing an aria-describedby attribute the screen reader should read the description in addition to the lagel for the button/link

Meta Issue
Make Uptime App Accessible for 7.0

Kibana Version:
7.0

Relevant WCAG Criteria:
4.1.2 Name, Role, Value (Level A, Primary Success Criterion)

Other Related Success Criterion:

1.3.1 Info and Relationships (Level A)
3.3.2 Labels or Instructions (Level A)

@rashmivkulkarni rashmivkulkarni added the Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability label Apr 19, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/uptime

@alvarolobato alvarolobato added :uptime and removed Project:Accessibility Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability labels Jun 19, 2019
@shahzad31
Copy link
Contributor

This has been fixed , although there was only one accessibility issue i saw with wave, but that is related to EUI Basic Table so i have reopened an old issue in EUI, that will be take care there if needed elastic/eui#840 (comment)

@shahzad31 shahzad31 self-assigned this Sep 2, 2019
@zube zube bot unassigned shahzad31 Sep 2, 2019
@zube zube bot reopened this Sep 2, 2019
@zube zube bot assigned shahzad31 Sep 2, 2019
@zube zube bot closed this as completed Sep 2, 2019
@zube zube bot unassigned shahzad31 Sep 2, 2019
@zube zube bot reopened this Sep 2, 2019
@zube zube bot assigned shahzad31 Sep 2, 2019
@zube zube bot closed this as completed Sep 2, 2019
@zube zube bot unassigned shahzad31 Sep 2, 2019
@zube zube bot reopened this Sep 2, 2019
@zube zube bot assigned shahzad31 Sep 2, 2019
@zube zube bot closed this as completed Sep 2, 2019
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

7 participants