Skip to content

Improve accessibility of bad.html — Ramp verified WCAG fixes#7

Closed
yangzhang75 wants to merge 1 commit into
mainfrom
ramp/fix-bad-html-1781938353616
Closed

Improve accessibility of bad.html — Ramp verified WCAG fixes#7
yangzhang75 wants to merge 1 commit into
mainfrom
ramp/fix-bad-html-1781938353616

Conversation

@yangzhang75

Copy link
Copy Markdown
Owner

What Ramp did

Ramp audited packages/harness/fixtures/bad.html, repaired the high-confidence WCAG violations, and verified each fix with axe-core before opening this PR. We don't ship a report — we ship the fix.

Compliance score

Before 60/100 → After 96/100 (+36)

critical serious moderate total
before 2 1 2 5
after 0 0 1 1

Fixes

WCAG Issue Result
1.1.1 Non-text Content missing alt text ✅ fixed + axe-verified
4.1.2 Name, Role, Value icon button accessible names ✅ fixed + axe-verified
1.4.3 Contrast (Minimum) low color contrast ✅ fixed + axe-verified
1.3.1 Info and Relationships missing landmarks ⚠️ failed

axe-core: before vs after

  • before: button-name(critical), color-contrast(serious), image-alt(critical), landmark-one-main(moderate), region(moderate)
  • after: region(moderate)

What a screen reader hears: before vs after

Before

RootWebArea: Ramp demo — intentionally inaccessible page
heading: Acme Dashboard
image
button
button
paragraph
StaticText: Your subscription renews on the 1st of next month.
form
textbox: Email
generic
button: Subscribe

After

RootWebArea: Ramp demo — intentionally inaccessible page
heading: Acme Dashboard
main
image: Acme company logo
button: Search
button: Add
paragraph
StaticText: Your subscription renews on the 1st of next month.
form
textbox: Email
generic
button: Subscribe

Validation

Each fix re-run through axe-core: targeted violation resolved, no new violations introduced. Static HTML — no build step.


🤖 Generated by Ramp — accessibility audit → fix → verify → PR. (Demo PR against an intentionally-broken fixture; not for merge.)

@yangzhang75 yangzhang75 deleted the ramp/fix-bad-html-1781938353616 branch June 21, 2026 05:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant