Skip to content

Keep Docs in the site header on phones - #1093

Merged
nedtwigg merged 1 commit into
mainfrom
docs-in-header
Oct 9, 2026
Merged

nedtwigg merged 1 commit into
mainfrom
docs-in-header

Conversation

@nedtwigg

@nedtwigg nedtwigg commented Oct 9, 2026

Copy link
Copy Markdown
Member

At wide widths the header shows Playground | Download | Docs | GitHub. Narrow widths used to hide both Download and Docs; now only Download drops, so Playground | Docs | GitHub stay on phones.

  • website/src/lib/site-nav.ts: Docs no longer sets hideOnMobile.
  • website/src/components/SiteHeader.tsx: below sm (640px) the nav uses text-base and gap-4, and below 360px text-sm and gap-3, so the three links fit. At 360px the old sizing ran GitHub 15px past the right gutter. Layout at 640px and up is unchanged.

Checked in the browser at 320, 360, 375, 640, and 1280px: every link ends inside the 16px gutter, and the header never overflows. (At 320px the homepage itself is 348px wide because of other content, not the header.)

🤖 Generated with Claude Code

Narrow widths now drop only Download, keeping Playground, Docs, and GitHub.
The nav steps down its text size and spacing below 640px and again below
360px so the three links fit down to a 320px screen.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying mouseterm with  Cloudflare Pages  Cloudflare Pages

Latest commit: 5143100
Status: ✅  Deploy successful!
Preview URL: https://0e042165.mouseterm.pages.dev
Branch Preview URL: https://docs-in-header.mouseterm.pages.dev

View logs

@nedtwigg
nedtwigg merged commit 3a4ed36 into main Oct 9, 2026
13 checks passed
@nedtwigg
nedtwigg deleted the docs-in-header branch October 9, 2026 23:26

This branch is waiting to be deployed

1 waiting deployment
hosted-preview — 5143100f Waiting Oct 9, 2026 by nedtwigg via cleanup #1277
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