Skip to content

fix: improve Gen1 Legacy badge color contrast for accessibility#8546

Merged
ahmedhamouda78 merged 1 commit intoaws-amplify:mainfrom
ahmedhamouda78:fix/gen1-badge-contrast
Apr 1, 2026
Merged

fix: improve Gen1 Legacy badge color contrast for accessibility#8546
ahmedhamouda78 merged 1 commit intoaws-amplify:mainfrom
ahmedhamouda78:fix/gen1-badge-contrast

Conversation

@ahmedhamouda78
Copy link
Copy Markdown
Member

Description of changes:

Fix the Gen1 "Legacy" badge (.gen1-page-banner__badge) color contrast to meet WCAG 2.2 AA requirements.

  • Before: background-color: #d97706 with white text → 3.18:1 contrast ratio (fails 4.5:1 minimum)
  • After: background-color: #b45309 with white text → 5.04:1 contrast ratio (passes)

This is a pre-existing accessibility violation that causes the accessibility scan to fail on all Gen1 pages, including new ones added in #8540.

Checks

  • Does this PR conform to the styleguide?
  • Does this PR include filetypes other than markdown or images? Yes — SCSS styling fix, no unit tests needed.
  • Are any files being deleted with this PR? No.
  • Are all links in MDX files using the MDX link syntax rather than HTML link syntax? N/A

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Change .gen1-page-banner__badge background-color from #d97706 to #b45309
to meet WCAG 4.5:1 contrast ratio requirement (3.18:1 → 5.04:1).
@ahmedhamouda78 ahmedhamouda78 requested a review from a team as a code owner April 1, 2026 13:54
@ahmedhamouda78 ahmedhamouda78 added the run-tests Label to run tests on PRs from a fork label Apr 1, 2026
@ahmedhamouda78 ahmedhamouda78 enabled auto-merge (squash) April 1, 2026 14:34
@ahmedhamouda78 ahmedhamouda78 merged commit 47b7dd4 into aws-amplify:main Apr 1, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

run-tests Label to run tests on PRs from a fork

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants