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.

Sign up for GitHub

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

Jump to bottom

[Skeleton] fails WCAG level A guidelines as it exposes no state to assistive technologies #31363

Closed
2 tasks done
brucelawson opened this issue Mar 8, 2022 · 2 comments
Closed
2 tasks done

[Skeleton] fails WCAG level A guidelines as it exposes no state to assistive technologies #31363

brucelawson opened this issue Mar 8, 2022 · 2 comments
Labels
accessibility a11y component: skeleton This is the name of the generic UI component, not the React module! docs Improvements or additions to the documentation duplicate This issue or pull request already exists

Comments

@brucelawson
Copy link

Duplicates

  • I have searched the existing issues

Latest version

  • I have tested the latest version

Current behavior 😯

Currently, no aria role or state is exposed to AT when Skeleton is displayed (tested all variants on https://mui.com/components/skeleton/ with Voiceover+Safari, NVDA+ Chrome Windows)

This fails Success Criterion 4.1.2: Name, Role, Value https://www.w3.org/WAI/WCAG21/Understanding/name-role-value.html

Expected behavior 🤔

As https://adrianroselli.com/2020/11/more-accessible-skeletons.html

Steps to reproduce 🕹

Steps:

  1. Navigate to https://mui.com/components/skeleton/#animations with screen reader on
  2. Notice no 'busy'/ 'loading' status is announced to user

Context 🔦

No response

Your environment 🌎

`npx @mui/envinfo` 
  Don't forget to mention which browser you used.
  Output from `npx @mui/envinfo` goes here.
@brucelawson brucelawson added the status: waiting for maintainer These issues haven't been looked at yet by a maintainer label Mar 8, 2022
@michaldudak michaldudak added accessibility a11y component: skeleton This is the name of the generic UI component, not the React module! and removed status: waiting for maintainer These issues haven't been looked at yet by a maintainer labels Mar 8, 2022
@michaldudak michaldudak changed the title [Accessibility][Skeleton] fails WCAG level A guidelines as it exposes no state to assistive technologies [Skeleton] fails WCAG level A guidelines as it exposes no state to assistive technologies Mar 8, 2022
@oliviertassinari
Copy link
Member

oliviertassinari commented Mar 8, 2022 •
edited
Loading

Duplicate of #26930? It seems to be a docs only problem.

@mnajdova mnajdova added the docs Improvements or additions to the documentation label Mar 24, 2022
@mnajdova
Copy link
Member

Agree, looks very similar to #26930. @brucelawson I am marking the issue as a duplicate. If you noticed that something is missing in #26930 would be best if you can leave a comment there, so that we have all discussion in one place. Thanks!

@mnajdova mnajdova added the duplicate This issue or pull request already exists label Mar 24, 2022
@samuelsycamore samuelsycamore closed this as not planned Won't fix, can't repro, duplicate, stale Nov 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
accessibility a11y component: skeleton This is the name of the generic UI component, not the React module! docs Improvements or additions to the documentation duplicate This issue or pull request already exists
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@brucelawson @oliviertassinari @mnajdova @michaldudak @samuelsycamore