Skip to content

fix(header): solid distinct navbar surface for light-mode readability#11308

Draft
jd wants to merge 1 commit intodevs/jd/worktree-design/map-semantic-tokens-primitive-layer--efb776d7from
devs/jd/worktree-design/tighten-navbar-opacity-link-readability--2acd91b6
Draft

fix(header): solid distinct navbar surface for light-mode readability#11308
jd wants to merge 1 commit intodevs/jd/worktree-design/map-semantic-tokens-primitive-layer--efb776d7from
devs/jd/worktree-design/tighten-navbar-opacity-link-readability--2acd91b6

Conversation

@jd
Copy link
Copy Markdown
Member

@jd jd commented Apr 29, 2026

The previous navbar styling — body-bg-tinted color-mix at 95% opacity
plus a backdrop blur — collapsed against the cleaner light gray
introduced by the design-system token migration: links became hard to
read because the navbar visually merged into the body bg.

  • Map --theme-navbar-bg to --theme-bg-content (white in light mode).
    Dark mode is unaffected because --theme-bg-content aliases --theme-bg
    there.
  • Drop the color-mix transparency and the backdrop-filter blur. The
    navbar is now a solid surface; the modern feel comes from the surface
    contrast plus the divider, not from glassmorphism.
  • Add a 1px bottom border using --theme-divider for clear separation.

Depends-On: #11307

@jd
Copy link
Copy Markdown
Member Author

jd commented Apr 29, 2026

This pull request is part of a Mergify stack:

# Pull Request Link
1 feat(design): add primitive token layer (tokens.css) #11299
2 feat(design): add typography utilities (typography.css) #11300
3 feat(design): load tokens.css and typography.css #11301
4 docs(design): add DESIGN.md design system documentation #11302
5 docs(design): link DESIGN.md from AGENTS.md and CLAUDE.md #11303
6 fix(lint): exclude root *.md files from remark-lint rules #11304
7 fix(design): address Phase 1 code review findings #11305
8 refactor(design): map semantic tokens to new primitive layer #11307
9 fix(header): tighten navbar opacity for link readability #11308 👈

@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Apr 29, 2026

Merge Protections

Your pull request matches the following merge protections and will not be merged until they are valid.

🔴 ⛓️ Depends-On Requirements

Waiting for:

This rule is failing.

Requirement based on the presence of Depends-On in the body of the pull request

🔴 👀 Review Requirements

Waiting for:

  • #approved-reviews-by >= 2
This rule is failing.
  • any of:
    • #approved-reviews-by >= 2
    • author = dependabot[bot]
    • author = mergify-ci-bot

🔴 🔎 Reviews

Waiting for:

  • #review-requested = 0
This rule is failing.
  • #review-requested = 0
  • #changes-requested-reviews-by = 0
  • #review-threads-unresolved = 0

🟢 🤖 Continuous Integration

Wonderful, this rule succeeded.
  • all of:
    • check-success = build
    • check-success = lint
    • check-success = test
    • any of:
      • check-success = test-broken-links
      • label = ignore-broken-links
    • any of:
      • check-success=Cloudflare Pages
      • -head-repo-full-name~=^Mergifyio/

🟢 Enforce conventional commit

Wonderful, this rule succeeded.

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

  • title ~= ^(fix|feat|docs|style|refactor|perf|test|build|ci|chore|revert|ui)(?:\(.+\))?:

🟢 📕 PR description

Wonderful, this rule succeeded.
  • body ~= (?ms:.{48,})

@mergify mergify Bot requested a review from a team April 29, 2026 14:11
The previous navbar styling — body-bg-tinted color-mix at 95% opacity
plus a backdrop blur — collapsed against the cleaner light gray
introduced by the design-system token migration: links became hard to
read because the navbar visually merged into the body bg.

- Map --theme-navbar-bg to --theme-bg-content (white in light mode).
  Dark mode is unaffected because --theme-bg-content aliases --theme-bg
  there.
- Drop the color-mix transparency and the backdrop-filter blur. The
  navbar is now a solid surface; the modern feel comes from the surface
  contrast plus the divider, not from glassmorphism.
- Add a 1px bottom border using --theme-divider for clear separation.

Change-Id: I2acd91b68a453f89a2ecdcc92324c976d6857315
@jd jd changed the title fix(header): tighten navbar opacity for link readability fix(header): solid distinct navbar surface for light-mode readability Apr 29, 2026
@jd jd force-pushed the devs/jd/worktree-design/tighten-navbar-opacity-link-readability--2acd91b6 branch from d4a51c5 to d8b4779 Compare April 29, 2026 14:25
@jd
Copy link
Copy Markdown
Member Author

jd commented Apr 29, 2026

Revision history

# Type Changes Reason Date
1 initial d4a51c5 2026-04-29 14:25 UTC
2 content d4a51c5 → d8b4779 2026-04-29 14:25 UTC

@mergify mergify Bot had a problem deploying to Mergify Merge Protections April 29, 2026 14:25 Failure
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant