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

[grid] Remove aria-expanded from cells with expanded details [1 day] #3425

Closed
10 tasks
yuriy-fix opened this issue Feb 11, 2022 · 1 comment · Fixed by #3441
Closed
10 tasks

[grid] Remove aria-expanded from cells with expanded details [1 day] #3425

yuriy-fix opened this issue Feb 11, 2022 · 1 comment · Fixed by #3441
Assignees
Labels
a11y Accessibility issue needs research More information needed to estimate

Comments

@yuriy-fix
Copy link
Contributor

What is the problem?

Remove aria-expanded from cells with expanded details.
Research if there is a proper way to expose the collapsed / expanded state of the cell with details without using aria-expanded.

Browsers

  • Chrome
  • Firefox
  • Safari
  • Safari on iOS
  • Edge

Screen Readers

  • None
  • NVDA
  • JAWS
  • VoiceOver on MacOS
  • VoiceOver on iOS
@yuriy-fix yuriy-fix added needs research More information needed to estimate a11y Accessibility issue labels Feb 11, 2022
@yuriy-fix yuriy-fix changed the title [grid] Remove aria-expanded from cells with expanded details [grid] Remove aria-expanded from cells with expanded details [1 day] Feb 14, 2022
@web-padawan web-padawan self-assigned this Feb 14, 2022
@web-padawan
Copy link
Member

web-padawan commented Feb 14, 2022

One possible option is to try aria-details attribute which is used by Bootstrap Vue:

Screenshot 2022-02-14 at 14 47 42

UPD: tested and it does not seem to be supported by neither VoiceOver nor NVDA at the moment.

Here is a blog post that indicates JAWS is the only screen reader supporting this feature. I could not get JAWS running on my Windows 11 virtual machine yet. We can still add as hopefully other screen readers will catch up.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a11y Accessibility issue needs research More information needed to estimate
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants