close
Skip to content

fix(a11y): use --link-blue for secondary login button (contrast WCAG 1.4.3) - #13030

Merged
lokesh merged 1 commit into
masterfrom
a11y/login-links-secondary-contrast
Jul 3, 2026
Merged

fix(a11y): use --link-blue for secondary login button (contrast WCAG 1.4.3)#13030
lokesh merged 1 commit into
masterfrom
a11y/login-links-secondary-contrast

Conversation

@mekarpeles

Copy link
Copy Markdown
Member

Summary

  • .login-links__secondary renders on the --light-beige app-drawer background.
  • --primary-blue (hsl(202, 96%, 37%)) against --light-beige (hsl(48, 29%, 93%)) = 4.27:1 — fails WCAG 1.4.3 AA (4.5:1 required).
  • Fix: use --link-blue (hsl(202, 96%, 28%)) — 6.51:1 against --light-beige and 7.41:1 against white (hover state). Both pass AA.

The primary button already uses --link-blue in its hover state (visual intent unchanged — same family, just slightly darker at rest).

Test plan

  • Open the hamburger menu (header app-drawer) on any page — "Sign In" (primary) and "Sign Up" (secondary) buttons should both look correct.
  • Inspect secondary button at rest: color: var(--link-blue), border: 2px solid var(--link-blue).
  • Run axe on any page with the header open — no color-contrast violations on .login-links__secondary.

Part of WCAG 2.1 AA a11y remediation — tracked in #13009

…1.4.3)

.login-links__secondary renders on the --light-beige app-drawer background.
--primary-blue (hsl 202 96% 37%) against --light-beige (hsl 48 29% 93%)
yields 4.27:1 — below the 4.5:1 AA threshold. --link-blue (hsl 202 96% 28%)
gives 6.51:1 against --light-beige and 7.41:1 against white (hover state).

Fixes the 11 login-links__secondary violations tracked in #13009.
@mekarpeles mekarpeles added the Theme: Accessibility Work related to disability accessibility. [managed] label Jun 24, 2026
@lokesh
lokesh merged commit 2be7db8 into master Jul 3, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Theme: Accessibility Work related to disability accessibility. [managed]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants