Skip to content

Fix navbar: one row on desktop, menu dropdown below lg - #60

Merged
ralyodio merged 1 commit into
masterfrom
fix-navbar
Oct 21, 2026
Merged

ralyodio merged 1 commit into
masterfrom
fix-navbar

Conversation

@ralyodio

Copy link
Copy Markdown
Contributor

The header crammed 8 links into a max-w-3xl flex-wrap row: the logo wrapped to "$ / c0mpute", "status" orphaned onto a second row at 1280px, and mobile got a ragged 3-line block.

  • Header widened to max-w-5xl; logo + links whitespace-nowrap, one row at lg+
  • Below lg the links fold into a <details> menu (works without JS), closed on client navigation
  • Links moved to components/site-nav.tsx

Verified locally at 1280/1024/768/390 plus a CDP click test (menu opens, 8 links visible, closes after navigating to /nodes). tsc --noEmit and next build pass.

🤖 Generated with Claude Code

The header crammed 8 links into a max-w-3xl flex-wrap row, so the logo wrapped to two lines, 'status' orphaned onto a second row on desktop and mobile got a ragged three-line block. Widen the header to max-w-5xl, keep logo and links on one line, and fold the links into a <details> menu below lg that closes on client navigation.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@ralyodio
ralyodio merged commit c27b443 into master Oct 21, 2026
7 checks passed
@github-actions

Copy link
Copy Markdown

ThreatCrush Security Scan

9 finding(s)

MEDIUM: 9

Severity Rule Location
MEDIUM js-unescaped-html-sink apps/web/src/app/blog/[slug]/page.tsx:53
MEDIUM js-unescaped-html-sink apps/web/src/app/layout.tsx:89
MEDIUM js-unescaped-html-sink apps/web/src/components/pit-footer.tsx:17
MEDIUM sh-remote-script-execution scripts/dev-setup.sh:25
MEDIUM sh-eval-expansion scripts/dev-setup.sh:35
MEDIUM sh-remote-script-execution scripts/install.sh:162
MEDIUM sh-remote-script-execution scripts/install.sh:187
MEDIUM sh-remote-script-execution scripts/install.sh:305
MEDIUM sh-remote-script-execution scripts/install.sh:322

Snippets are redacted; ThreatCrush never prints matched credential material.

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.

1 participant