[test][linear progress] Add axe tests and WCAG conformance report - #48922
[test][linear progress] Add axe tests and WCAG conformance report#48922michelengelen wants to merge 2 commits into
Conversation
Deploy previewBundle size
Check out the code infra dashboard for more information about this PR. |
2d239cc to
e4e16fa
Compare
fc0a007 to
b7f0cab
Compare
57ab9fb to
2db86bf
Compare
PR reviewThe risky move here — re-bundling the Docs (1)1. 🟡 Checkbox rationale comment attached to the progress ruleLocation: // FormControlLabelPosition is not enrolled: its only axe finding is an aria-label on a
// role-less FormGroup div (aria-prohibited-attr), a demo quirk unrelated to Checkbox.
{
test: `docs/data/material/components/progress/{${LINEARPROGRESS_A11Y_DEMOS.join(',')}}`,The comment explains a checkboxes-page decision (a Failure scenario: the comment reads as if a progress demo were excluded; the actual checkbox exclusion loses its documentation. Fix: move the comment up next to the checkbox rules — it belongs in #48918's scope. VerdictApprove after nits - the flaky-slug tradeoff is solved properly; only a comment is in the wrong place. 🤖 Review generated with Claude Code |
2db86bf to
929d929
Compare
Applies mui#48916 (comment): the shared reports table starts empty, and each component PR appends its own row. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013HgD6GEW3UxNRvC2abbTy9
Adds axe coverage and the WCAG 2.2 Level A/AA conformance report for
LinearProgress. Also un-excludes theprogressslug from the demo bundle so axe can reach these demos; screenshots stay disabled slug-wide.Known gaps: fill-versus-track below 3:1 for most palette colours in light mode (1.4.11), indefinite animation with no pause control (2.2.2), and value changes not announced (4.1.3).
Report:
LinearProgress/accessibility.mdImportant
How to review this PR
Layer 9 of 14 in a stacked series (#48915 → #48926). Its branch is built on
a11y/stack/toggle-button(#48921). GitHub cannot chain PR bases across a fork, so every PR in the series targetsmasterinstead.Review only the last commit —
2db86bf. Everything above it belongs to the layers below, and disappears as those merge.Original work by @mj12albert. Supersedes #48735, rebased onto current master.
Part of the WCAG conformance effort (#14187).