Skip to content

💄🔧:lift the navbar links off the ink - #1838

Merged
openinf-commit-queue[bot] merged 1 commit into
livefrom
fix/navbar-link-contrast
Aug 20, 2026
Merged

💄🔧:lift the navbar links off the ink#1838
openinf-commit-queue[bot] merged 1 commit into
livefrom
fix/navbar-link-contrast

Conversation

@DerekNonGeneric

Copy link
Copy Markdown
Member

Bootstrap's dark navbar draws its links at white 55%. Against our ink that
measures 6.04:1 — it clears AA, but only just, and at that size it reads washed
rather than quiet.

alpha   effective          vs ink
0.55    rgb(153,152,150)    6.04     <- was
0.78    rgb(205,204,204)   10.93     <- now
1.00    rgb(255,255,255)   17.46     <- hover and current page

78% is nearly double the contrast and still sits back from the page it is
pointing at.

The part that came with it

The current page was marked by being white among greys. Once the greys are
lifted to 78% that distinction nearly vanishes, and it was a weak marker to
begin with: colour alone, no second cue, and nothing announced to a screen
reader.

So the current link now takes a rule under it in $primary, and the
aria-current="page" that should always have gone alongside the active
class. Marked twice, and one of the two is not visual.

Scope

Both rules are .navbar-dark, which is the fixed top bar and nothing else. The
mobile sidebar is a plain .navbar on $secondary, so its links were never
part of this, and it already marks the current page with a border down its
left.

aria-current is added in both navs, since the sidebar deserves it equally.

Checks

nps test passes, all 17 verifiers. vnu clean on /, /about/ and
/news/. Confirmed in the built output that active and aria-current land
on Home from / and About from /about/.

How it came up

Reviewing the home page in #1833. It is site-wide, so it is here on its own.

@netlify

netlify Bot commented Aug 20, 2026

Copy link
Copy Markdown

Deploy Preview for gh-pages-openinf ready!

Name Link
🔨 Latest commit b2c2db8
🔍 Latest deploy log https://app.netlify.com/projects/gh-pages-openinf/deploys/6a878b47d1a6d8000899db43
😎 Deploy Preview https://deploy-preview-1838--gh-pages-openinf.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Bootstrap's dark navbar draws its links at white 55%, which on our ink
is 6.04:1. It clears AA and still reads washed. 78% is 10.93:1.

That leaves the current page without a marker, since it was white among
greys. It takes a rule in the brand teal instead, sitting on the bar's
bottom edge, and the `aria-current` that belongs beside the class.

The sidebar is untouched: not `navbar-dark`, and already marked down
its left.

Signed-off-by: Derek Lewis <DerekNonGeneric@inf.is>
Assisted-by: Claude-Code:claude-opus-5
@DerekNonGeneric
DerekNonGeneric force-pushed the fix/navbar-link-contrast branch from ae01246 to b2c2db8 Compare August 20, 2026 23:18
@OpenINFbot OpenINFbot added the commit-queue Land this pull request when its checks pass label Aug 20, 2026
@openinf-commit-queue
openinf-commit-queue Bot merged commit 6b1d250 into live Aug 20, 2026
15 checks passed
@openinf-commit-queue openinf-commit-queue Bot removed the commit-queue Land this pull request when its checks pass label Aug 20, 2026
@OpenINFbot
OpenINFbot deleted the fix/navbar-link-contrast branch August 20, 2026 23:21
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