Skip to content

🏗️🔧:bring the navbar into the palette - #1836

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

🏗️🔧:bring the navbar into the palette#1836
openinf-commit-queue[bot] merged 1 commit into
livefrom
fix/navbar-off-palette

Conversation

@DerekNonGeneric

Copy link
Copy Markdown
Member

The navbar carries Bootstrap's bg-dark, which meant Bootstrap's own #212529
— a colour this palette never chose. _sublime-theme.scss names four colours,
an ink and a parchment, and that grey is none of them.

Against parchment it does not show. Put a dark section directly under the
navbar, as the home page in #1833 does, and the seam is obvious:

surface colour cast
navbar, bg-dark #212529 cool blue-grey
masthead, $body-color #1d1916 warm near-black

Two darks of near-equal lightness with opposite hue casts, meeting on an edge.
It reads as something gone wrong rather than as a decision.

The change

$dark: $body-color;

Our dark is the ink, so the palette says so. Bootstrap derives --bs-dark-rgb
and bg-dark from $dark, so the navbar follows without touching any markup
and without an !important to fight the utility.

Scope

bg-dark is the only $dark-derived class any template uses, and it appears
once:

_includes/header.liquid:3:bg-dark

No text-dark, btn-dark, border-dark or table-dark anywhere. So the
navbar is the entire visible effect, on every page. Against parchment the
difference is slight; under the home page's masthead it is the point.

Checks

nps test passes, all 17 verifiers. vnu clean.

Note

#1833 wants this. The home page is what exposes the mismatch, and it will look
seamed until this lands.

The navbar is `bg-dark`, and until now that meant Bootstrap's #212529,
a colour this palette never chose. Against parchment nobody could tell.
Put a dark section directly under it, as the home page does, and a cool
grey sits on our warm ink and reads as a mistake rather than a choice.

Our dark is the ink, so `$dark` says so. `bg-dark` is the only thing
derived from it that any template uses, so the navbar is the whole
visible effect.

Signed-off-by: Derek Lewis <DerekNonGeneric@inf.is>
Assisted-by: Claude-Code:claude-opus-5
@netlify

netlify Bot commented Aug 20, 2026

Copy link
Copy Markdown

Deploy Preview for gh-pages-openinf ready!

Name Link
🔨 Latest commit ea1721c
🔍 Latest deploy log https://app.netlify.com/projects/gh-pages-openinf/deploys/6a873133fb4ed1000824c5c8
😎 Deploy Preview https://deploy-preview-1836--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.

@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 ccda078 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-off-palette branch August 20, 2026 18:10
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