Skip to content

[checkbox] Set aria-checked=mixed when indeterminate#48147

Merged
mj12albert merged 2 commits into
mui:masterfrom
mj12albert:fix/indeterminate-aria-checked
Apr 14, 2026
Merged

[checkbox] Set aria-checked=mixed when indeterminate#48147
mj12albert merged 2 commits into
mui:masterfrom
mj12albert:fix/indeterminate-aria-checked

Conversation

@mj12albert

@mj12albert mj12albert commented Mar 30, 2026

Copy link
Copy Markdown
Member

Fixes #20476

Indeterminate checkboxes don't have any specific logic, it's completely user-controlled so we can simply add the attribute here

@mj12albert mj12albert added accessibility a11y scope: checkbox Changes related to the checkbox. type: enhancement It’s an improvement, but we can’t make up our mind whether it's a bug fix or a new feature. labels Mar 30, 2026
@mui-bot

mui-bot commented Mar 30, 2026

Copy link
Copy Markdown

Netlify deploy preview

https://deploy-preview-48147--material-ui.netlify.app/

Bundle size report

Bundle Parsed size Gzip size
@mui/material 🔺+32B(+0.01%) 🔺+22B(+0.02%)
@mui/lab 0B(0.00%) 0B(0.00%)
@mui/system 0B(0.00%) 0B(0.00%)
@mui/utils 0B(0.00%) 0B(0.00%)

Details of bundle changes

Generated by 🚫 dangerJS against 55c4b2d

@mj12albert mj12albert force-pushed the fix/indeterminate-aria-checked branch from 5256a21 to 816cbcb Compare March 30, 2026 13:13
@mj12albert mj12albert force-pushed the fix/indeterminate-aria-checked branch from 816cbcb to 55c4b2d Compare March 30, 2026 13:30
@mj12albert mj12albert marked this pull request as ready for review March 30, 2026 13:37
@zannager zannager requested a review from sai6855 March 31, 2026 15:32
@mj12albert mj12albert added v7.x needs cherry-pick The PR should be cherry-picked to master after merge. labels Mar 31, 2026
@silviuaavram

silviuaavram commented Apr 8, 2026

Copy link
Copy Markdown
Member

Do we need to set the aria attribute? Since we are using a native input type checkbox, I thought we don't need the aria-indeterminate.

For example, I tested with VO and Chrome, and I see no difference in reading. Both using and not using aria-indeterminate="mixed" and I get the same unchecked reading.

@mj12albert

Copy link
Copy Markdown
Member Author

Do we need to set the aria attribute? Since we are using a native input type checkbox, I thought we don't need the aria-indeterminate.

For example, I tested with VO and Chrome, and I see no difference in reading. Both using and not using aria-indeterminate="mixed" and I get the same unchecked reading.

@silviuaavram Support is spotty, but proper implementations announce something like "partially checked", so it's a small nice to have I guess

AFAIK it works with JAWS+Chrome, Win Narrator+Edge

@mj12albert mj12albert enabled auto-merge (squash) April 14, 2026 15:13
@code-infra-dashboard

Copy link
Copy Markdown

Bundle size

Bundle Parsed size Gzip size
@mui/material 🔺+32B(+0.01%) 🔺+17B(+0.01%)
@mui/lab 0B(0.00%) 0B(0.00%)
@mui/private-theming 0B(0.00%) 0B(0.00%)
@mui/system 0B(0.00%) 0B(0.00%)
@mui/utils 0B(0.00%) 0B(0.00%)

Details of bundle changes

Deploy preview

https://deploy-preview-48147--material-ui.netlify.app/


Check out the code infra dashboard for more information about this PR.

@mj12albert mj12albert merged commit a8fa9d7 into mui:master Apr 14, 2026
19 checks passed
@github-actions

Copy link
Copy Markdown

Cherry-pick PRs will be created targeting branches: v7.x

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

Labels

accessibility a11y needs cherry-pick The PR should be cherry-picked to master after merge. scope: checkbox Changes related to the checkbox. type: enhancement It’s an improvement, but we can’t make up our mind whether it's a bug fix or a new feature. v7.x

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[checkbox] Set aria-checked="mixed" when indeterminate

3 participants