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

When using `role="heading"` and `aria-level`, jumping directly to a specific heading level is buggy in IE #5434

Closed
nvaccessAuto opened this Issue Oct 21, 2015 · 3 comments

Comments

Projects
None yet
1 participant
@nvaccessAuto

nvaccessAuto commented Oct 21, 2015

Reported by jmuheim on 2015-10-21 13:02
I have added a demo of how to overrule heading levels using ARIA:

http://developer-guide.access4all.ch/en/examples/overrule_heading_levels_using_aria

While playing with it, I noticed that NVDA jumps to headings of all levels when pressing the 1 key. The 2 key seems to work as expected.

Is this a bug in NVDA with IE? In FF it works, and JAWS works in both FF and IE.

@nvaccessAuto

This comment has been minimized.

Show comment
Hide comment
@nvaccessAuto

nvaccessAuto Oct 21, 2015

Comment 1 by jteh on 2015-10-21 22:53
Confirmed. It is a bug. The code is looking for an h1 or a role="heading" aria-level="1", but it doesn't account for the fact that aria-level should override the tag. Unfortunately, this is going to be tricky to fix, since we can only match using restricted criteria for quick navigation.

nvaccessAuto commented Oct 21, 2015

Comment 1 by jteh on 2015-10-21 22:53
Confirmed. It is a bug. The code is looking for an h1 or a role="heading" aria-level="1", but it doesn't account for the fact that aria-level should override the tag. Unfortunately, this is going to be tricky to fix, since we can only match using restricted criteria for quick navigation.

@nvaccessAuto

This comment has been minimized.

Show comment
Hide comment
@nvaccessAuto

nvaccessAuto Oct 22, 2015

Comment 2 by jteh on 2015-10-22 01:02
Technical: We can do this with the following matches:

  • h1-6 tag with correct aria-level
  • hx tag where x is correct level and no aria-level
  • role="heading" with correct aria-level

nvaccessAuto commented Oct 22, 2015

Comment 2 by jteh on 2015-10-22 01:02
Technical: We can do this with the following matches:

  • h1-6 tag with correct aria-level
  • hx tag where x is correct level and no aria-level
  • role="heading" with correct aria-level
@nvaccessAuto

This comment has been minimized.

Show comment
Hide comment
@nvaccessAuto

nvaccessAuto Nov 2, 2015

Comment 3 by Michael Curran <mick@... on 2015-11-02 05:24
In commit 113f9e2:
Merge branch 't5434' into next. Incubates #5434
Changes:
Added labels: incubating

nvaccessAuto commented Nov 2, 2015

Comment 3 by Michael Curran <mick@... on 2015-11-02 05:24
In commit 113f9e2:
Merge branch 't5434' into next. Incubates #5434
Changes:
Added labels: incubating

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment