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

Aria-Current invalid issue #1773

Closed
aparnapasi opened this issue Aug 20, 2019 · 5 comments
Closed

Aria-Current invalid issue #1773

aparnapasi opened this issue Aug 20, 2019 · 5 comments
Assignees
Labels
fix Bug fixes

Comments

@aparnapasi
Copy link

Aria-current value of slide is being shown as an issue by axe. As per the spec, it seems to be fine. Here is the spec - https://www.w3.org/TR/wai-aria-1.1/#aria-current

With slide value, axe is throwing a violation which should not be the case.

@straker
Copy link
Contributor

straker commented Aug 20, 2019

Do you have an example or code I could look at?

@WilcoFiers WilcoFiers added the info needed More information or research is needed to continue label Aug 22, 2019
@WilcoFiers
Copy link
Contributor

Lets do some testing in AT to make sure you can put aria-current=whatever on an element and still have it be announced as the current element. If that's the case, we can mark this for review.

@straker
Copy link
Contributor

straker commented Jan 2, 2020

Results from screen readers:

<ul>
  <li><a href="foo" aria-current="fllalal">Hi</a></li>
  <li><a href="foo">Hi</a></li>
  <li><a href="foo">Hi</a></li>
</ul>
  • NVDA / Friefox - "Hi, current link"
  • JAWS / IE11 - "Hi, link current"
  • VoiceOver / Safar - "current item, link, HI"
  • TalkBack / Android - doesn't read aria-current

@straker straker self-assigned this Jan 24, 2020
@straker straker added fix Bug fixes and removed info needed More information or research is needed to continue labels Jan 24, 2020
@straker
Copy link
Contributor

straker commented Jan 27, 2020

@jeankaplansky shouldn't be any doc changes

@padmavemulapati
Copy link

Ran through code base, as i am not seeing the expected behavour in the latest agora build. Yes, aria-current - issue is in needs-review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix Bug fixes
Projects
None yet
Development

No branches or pull requests

4 participants