Skip to content

Keep the getbb.app header on one line at 721–860px - #5325

Merged
brsbl merged 2 commits into
mainfrom
web/nav-no-wrap
Oct 10, 2026
Merged

brsbl merged 2 commits into
mainfrom
web/nav-no-wrap

Conversation

@brsbl

@brsbl brsbl commented Oct 10, 2026 •

Copy link
Copy Markdown
Collaborator

Human comments

What was wrong

From 721 to 754px wide, the getbb.app header didn't fit on one line. "Sign in" and "Download for macOS" wrapped onto two lines. The header only tightened its spacing at 720px and below, and the full row needs about 755px at the wider spacing.

What changed

  • At 860px and below, the header links sit 16px apart instead of 26px.
  • Header links and the download button never wrap their text.
  • The download button stays visible down to 721px, as before.

How you verified

  • Chrome for Testing (Browser Automation), home page, at 721, 730, 740, 750, 760, 780, 800, 820, 840, 860, 861, 900, and 1000px wide.
  • Before: the header wrapped at 721 to 754px.
  • After: no wrapping or horizontal overflow at any of those widths, with the download button showing at each.
  • Remote CI: pending.

Screenshots

Before: 59a1fcc29f (main) · After: 7b9b941476 · Captured at 2×

Screen / state Viewport Before After
Home page header 750×600 Before: Sign in and Download for macOS wrap onto two lines After: every link and the download button fit on one line

BB-Thread: 🔥 𝒃𝒃 landing pages & guides

🤖 Generated with Claude Code

AGENT GENERATED

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 10, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-10T06:12:12.791113Z d37c217 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: d37c2170d0

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread apps/web/src/landing/landing.css Outdated
Comment on lines +3142 to +3143
/* The full download CTA is repeated immediately below in the hero. Hide the
nav copy on narrow screens so the utility links fit on one line. */

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Remove the relocated CSS comment

This media-query explanation is an ordinary code comment, but the repository guidelines categorically forbid code comments except semantic tool directives and Plugin SDK declaration comments; remove it rather than relocating it with the rule.

AGENTS.md reference: AGENTS.md:L9-L11

Useful? React with 👍 / 👎.

…ter gaps instead

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@brsbl brsbl changed the title Keep the getbb.app header on one line at 721–800px Keep the getbb.app header on one line at 721–860px Oct 10, 2026
@brsbl
brsbl merged commit 3912f4a into main Oct 10, 2026
19 checks passed
@brsbl
brsbl deleted the web/nav-no-wrap branch October 10, 2026 19:33
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