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) Kibana Version 7. Heading Structure #37539

Closed
15 tasks done
barlowm opened this issue May 30, 2019 · 4 comments
Closed
15 tasks done

(Accessibility) Kibana Version 7. Heading Structure #37539

barlowm opened this issue May 30, 2019 · 4 comments

Comments

@barlowm
Copy link
Collaborator

barlowm commented May 30, 2019

Steps to reproduce (Use WAVE Extension)

Using WAVE extension to analyze the page reports an empty heading

EmptyHeading_1

Examining the source shows the H5 tag is empty

EmptyHeading

Meta Issue:

Accessibility Audit for Kibana 7.0
Make Uptime App Accessible for 7.0

Kibana Version:

7.0

Relevant WCAG Criteria:

WCAG Criterion

1.3.1 Info and Relationships: Information, structure, and relationships conveyed through presentation can be programmatically determined or are available in text. - Level A

2.4.6 - Headings and Labels - Level AA
G130: Providing descriptive headings

Headings and labels describe topic or purpose.
Proper Nesting of Headings (H5 must be preceeded by H1-H4)

Headings should not be empty. If the content of the heading tag is to be provided dynamically then the heading tag itself should be provided at the same time. If the content of the heading tag is static but should only be displayed at a specific time then the entire heading tag (and content) must be rendered hidden from Assistive Technologies.

Each app needs to be fixed separately. Checklist for the apps:

myasonik pushed a commit that referenced this issue Oct 3, 2019
Focusing on heading structure and page layout for Home, Discover, Dashboard, and Visualize.

This is progress on #37539
myasonik pushed a commit that referenced this issue Oct 3, 2019
Focusing on heading structure and page layout for Home, Discover, Dashboard, and Visualize.

This is progress on #37539
@mattkime
Copy link
Contributor

I believe this has been resolved in Management, @myasonik will verify. Just ping me if anything is needed.

@rayafratkina
Copy link
Contributor

@weltenwort @peteharverson @cachedout any update on your apps? We are trying to get all of the basic heading issues resolved in 7.6, if possible.

@peteharverson
Copy link
Contributor

@rayafratkina I raised the issue #52278 to track work for the ML plugin. We will aim to resolve any issues we find in 7.6.

@myasonik
Copy link
Contributor

All of these are fixed 🎉

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

5 participants