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

APM Accessibility #26856

Open
1 of 6 tasks
aphelionz opened this issue Dec 8, 2018 · 10 comments
Open
1 of 6 tasks

APM Accessibility #26856

aphelionz opened this issue Dec 8, 2018 · 10 comments
Labels
a11y Accessibility issue Meta Team:APM All issues that need APM UI Team support

Comments

@aphelionz
Copy link
Contributor

aphelionz commented Dec 8, 2018

This is the meta issues to track the work to make APM UI accessible and specific issues found in APM UI.

@elasticmachine
Copy link
Contributor

Pinging @elastic/apm-ui

@formgeist
Copy link
Contributor

@alvarolobato Are you intending that we fix all the accessibility issues above? This issue was placed in Implementation ready.

@jasonrhodes
Copy link
Member

@rayafratkina Can you help us understand what the various "announce" / "self-announce" tasks are in this list? I'm not familiar with what that means and don't see anything in WCAG when I looked...

TIA!

@rayafratkina
Copy link
Contributor

Glad to help!
Self-announce refers to automatically reading out the app header when user opens the app to orient them to the screen they are on.
Announce is about having correct screen-readable labels when user tabs to an element of the page.
In case of Graph (#28145) the convention we have adopted is to announce when there is a component on the page that is not announced in detail by the screen reader.

@jasonrhodes
Copy link
Member

Self-announce refers to automatically reading out the app header when user opens the app to orient them to the screen they are on.

@rayafratkina thanks for the help! How exactly is this done? It's not clear what we would need to do to make this announcement happen?

@rayafratkina
Copy link
Contributor

If it does not make sense to have a heading in that place, the best practice is to add aria labels. See https://www.w3.org/TR/WCAG20-TECHS/ARIA14.html
aria-label will be read by the screen reader to announce the element.

@jasonrhodes
Copy link
Member

@rayafratkina I'm sorry, I still don't understand! For instance, one of the recommendations here is "APM should self-announce" ... you described self-announce to mean this: "Self-announce refers to automatically reading out the app header when user opens the app to orient them to the screen they are on." That sounds like something the screen reader does, so it sounds like there's no action to take on our part? If that's true, can I close those announce recommendations as long as there is a heading on every page (all APM pages have a heading)?

@sorenlouv
Copy link
Member

@jasonrhodes I think we just have to make sure, that the query bar input contains the necessary aria labels, so the screen reader can announce that on enter/focus.

@formgeist
Copy link
Contributor

@formgeist will reach out to the Kibana team that was responsible for the audit in the first place, to understand what needs happen in these cases.

@formgeist formgeist self-assigned this Jun 14, 2019
@formgeist formgeist removed their assignment Nov 22, 2019
@botelastic
Copy link

botelastic bot commented Nov 1, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@botelastic botelastic bot added the stale Used to mark issues that were closed for being stale label Nov 1, 2021
@alexwizp alexwizp added the a11y Accessibility issue label Oct 17, 2024
@botelastic botelastic bot removed the stale Used to mark issues that were closed for being stale label Oct 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a11y Accessibility issue Meta Team:APM All issues that need APM UI Team support
Projects
None yet
Development

No branches or pull requests

8 participants