Skip to content

Fix accessibility status page#3058

Open
trmccormick wants to merge 2 commits into
mainfrom
fix-accessibility-status-page
Open

Fix accessibility status page#3058
trmccormick wants to merge 2 commits into
mainfrom
fix-accessibility-status-page

Conversation

@trmccormick
Copy link
Copy Markdown
Collaborator

Fixes #3055

Improve breadcrumb contrast on System Status page

This pull request addresses an accessibility issue on the System Status page (/status) where breadcrumb link colors had insufficient contrast (4.2:1) below the WCAG AA requirement of 4.5:1. The links were using a muted gray (#9a9da0) that didn't meet the minimum ratio against light backgrounds.

The fix updates the breadcrumb link color to a darker shade (#212529) for better readability and compliance. This was verified by inspecting the computed styles after asset recompilation, confirming the color change applies correctly.

Changes proposed in this pull request:

Add CSS rule .breadcrumb-item a { color: #212529; } to hyku.scss for improved contrast.
@samvera/hyku-code-reviewers

…trast

Change breadcrumb link color to #212529 for better contrast (≥4.5:1).

Resolves issue #3055.
Update the CSS selector to target breadcrumb links more specifically.
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 13, 2026

Test Results

    3 files  +    3      3 suites  +3   14m 55s ⏱️ + 14m 55s
2 368 tests +2 368  2 312 ✅ +2 312  56 💤 +56  0 ❌ ±0 
2 395 runs  +2 395  2 337 ✅ +2 337  58 💤 +58  0 ❌ ±0 

Results for commit 1b36236. ± Comparison against base commit 3c9031e.

♻️ This comment has been updated with latest results.

@trmccormick trmccormick self-assigned this May 13, 2026
@trmccormick trmccormick added minor-ver for release notes patch-ver for release notes accessibility and removed minor-ver for release notes labels May 13, 2026
Copy link
Copy Markdown
Collaborator

@laritakr laritakr left a comment

Choose a reason for hiding this comment

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

This change is in the right place, but seems like it would change status site-wide across the board, not just the status page. Please verify your intent, as the PR title doesn't match this change. If you are only trying to target that one place, the selectors need to be tightened.

For each of these changes, I'd like to see before & after screenshots as verification.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Ensure that text and background colors have enough contrast on System Status page

2 participants