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

Flag aria-level values > 6 #3041

Closed
straker opened this issue Jun 25, 2021 · 2 comments
Closed

Flag aria-level values > 6 #3041

straker opened this issue Jun 25, 2021 · 2 comments
Labels
pr A pr has been created for the issue rules Issue or false result from an axe-core rule
Milestone

Comments

@straker
Copy link
Contributor

straker commented Jun 25, 2021

After doing some testing, it seems aria-level values > 6 are handled inconsistently across screen readers. For VO and NVDA, any positive value is allowed and applied to the heading. But in JAWS and Talkback they are not allowed and the aria-level will be announced as the default value (2 for role=heading elements, the heading level for H elements).

We decided we should create a new check for aria-valid-attr-value that looks specifically at the aria-level and if it's greater than 6 mark the element as Needs Review with a message stating lack of browser support.

See https://codepen.io/straker/pen/jOBjNNe

@straker straker added the rules Issue or false result from an axe-core rule label Jun 25, 2021
@WilcoFiers WilcoFiers added this to the axe-core 4.3 milestone Jul 14, 2021
@straker straker added the pr A pr has been created for the issue label Jul 14, 2021
@padmavemulapati
Copy link

Validated with the latest axe-core develop branch code (dated 15th July, 2021)
When aria-level value passed as greater than 6, either in div or in heading (does not matter where we are passing),
its showing message as
aria-level values greater than 6 are not supported in all screenreader and browser combinations
also for this , aria-valid-attr-value is appearing in needs review
test script:

<div role="heading" aria-level="12" id="target">Twelve</div>
    <h1 aria-level="10" id="target">Ten</h1>

image

@straker
Copy link
Contributor Author

straker commented Sep 28, 2021

No docs

@straker straker closed this as completed Sep 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr A pr has been created for the issue rules Issue or false result from an axe-core rule
Projects
None yet
Development

No branches or pull requests

3 participants