Skip to content

fix(a11y): add main landmark, unique nav labels, and fix color contrast#7894

Closed
Rishabh037 wants to merge 3 commits intowebpack:mainfrom
Rishabh037:fix/a11y-color-contrast
Closed

fix(a11y): add main landmark, unique nav labels, and fix color contrast#7894
Rishabh037 wants to merge 3 commits intowebpack:mainfrom
Rishabh037:fix/a11y-color-contrast

Conversation

@Rishabh037
Copy link

fix(a11y): improve footer link accessibility (WCAG 2.1 AA compliance)

Summary

This PR resolves accessibility issues reported by axe DevTools on the documentation homepage related to footer links.

The affected links relied primarily on color for distinction and did not fully meet WCAG 2.1 AA requirements for link accessibility and contrast.


Problem

Accessibility scans reported:

  • Links must be distinguishable without relying on color
  • Insufficient visual distinction between link text and surrounding text

This can impact:

  • Screen reader users
  • Users with low vision
  • Users with color vision deficiencies

Changes Made

  • Added text-decoration: underline to footer paragraph links
  • Ensured visible hover and focus states
  • Verified improved accessibility compliance using axe DevTools
  • Scoped changes only to footer links to avoid global styling impact

Before / After

🔴 Before

Screenshot 2026-03-01 at 7 13 33 PM )

🟢 After

Screenshot 2026-03-01 at 7 34 20 PM

@vercel
Copy link

vercel bot commented Mar 1, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
webpack-js-org Ready Ready Preview, Comment Mar 1, 2026 3:27pm

Request Review

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Mar 1, 2026

CLA Signed

The committers listed above are authorized under a signed CLA.

"prettier": "^3.8.1",
"react-refresh": "^0.18.0",
"react-test-renderer": "^17.0.2",
"react-test-renderer": "17.0.2",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please revert it

<Route path="*" element={<PageNotFound />} />
</Route>
</Routes>
</main>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We already have the main, it is not a good to have multiple mains

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.

2 participants