Skip to content

[test][text field] Add axe tests and WCAG conformance report - #48923

Open
michelengelen wants to merge 10 commits into
mui:masterfrom
michelengelen:a11y/stack/text-field
Open

[test][text field] Add axe tests and WCAG conformance report#48923
michelengelen wants to merge 10 commits into
mui:masterfrom
michelengelen:a11y/stack/text-field

Conversation

@michelengelen

@michelengelen michelengelen commented Aug 4, 2026

Copy link
Copy Markdown
Member

Adds axe coverage and the WCAG 2.2 Level A/AA conformance report for TextField.

Result Count
✅ Supports 25
⚠️ Partially Supports 3
❌ Does Not Support 0
➖ Not Applicable 27

Known gaps: placeholder (~2.55:1) and some focused labels below 4.5:1 (1.4.3), the outlined resting border at ~1.74:1 (1.4.11), and dynamically shown errors not announced (4.1.3).

Report: TextField/accessibility.md

Important

How to review this PR

Layer 10 of 14 in a stacked series (#48915#48926). Its branch is built on a11y/stack/linear-progress (#48922). GitHub cannot chain PR bases across a fork, so every PR in the series targets master instead.

Review only the last commit4b8f12c. Everything above it belongs to the layers below, and disappears as those merge.

Original work by @mj12albert. Supersedes #48758, rebased onto current master.

Part of the WCAG conformance effort (#14187).

@code-infra-dashboard

code-infra-dashboard Bot commented Aug 4, 2026

Copy link
Copy Markdown

Deploy preview

Bundle size

Bundle Parsed size Gzip size
@mui/material 0B(0.00%) 0B(0.00%)
@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


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

@michelengelen
michelengelen force-pushed the a11y/stack/text-field branch from 81cc196 to 68e85f7 Compare August 5, 2026 09:56
@michelengelen michelengelen added test accessibility a11y scope: text field Changes related to the text field. labels Aug 5, 2026
@michelengelen
michelengelen force-pushed the a11y/stack/text-field branch 5 times, most recently from 7af3886 to 3a66aca Compare August 6, 2026 10:43
@michelengelen
michelengelen marked this pull request as ready for review August 7, 2026 10:15
@michelengelen
michelengelen force-pushed the a11y/stack/text-field branch 2 times, most recently from ea1f968 to 9fed364 Compare August 7, 2026 13:01
@michelengelen
michelengelen force-pushed the a11y/stack/text-field branch from 9fed364 to 4b8f12c Compare August 11, 2026 12:25
@michelengelen

Copy link
Copy Markdown
Member Author

PR review

No findings.

The two deleted tests were verified as moved, not lost: "label the input" becomes 2.5.3 (same toHaveAccessibleName mechanism via role query) and "has an accessible description" becomes the 1.3.1 helper-text test. The 3.3.1 pair (aria-invalid plus linked message, and the not-invalid control case) is the strongest error-identification coverage in the stack. The Validation prose rewrite in text-fields.md is accurate and reads better.

Verdict

Approve - deleted tests are subsumed with equal or stronger assertions.


🤖 Review generated with Claude Code

@github-actions github-actions Bot added the PR: out-of-date The pull request has merge conflicts and can't be merged. label Aug 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

accessibility a11y PR: out-of-date The pull request has merge conflicts and can't be merged. scope: text field Changes related to the text field. test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants