Skip to content

Remove homepage side gutters on mobile#511

Merged
ghesp merged 1 commit into
mainfrom
fix/homepage-mobile-full-width
Jul 3, 2026
Merged

Remove homepage side gutters on mobile#511
ghesp merged 1 commit into
mainfrom
fix/homepage-mobile-full-width

Conversation

@claude

@claude claude Bot commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Requested by Georgina Hesp · Slack thread

Summary

Before: On mobile, the homepage content sits inside a bordered scaffold with ~16px empty gutters on each side, wasting horizontal space.

After: On mobile (<768px) the side scaffold border and horizontal padding are dropped so content uses the full viewport width. Desktop is unchanged — the border-x scaffold and px-4 gutter still apply at md: and up.

How: In app/src/components/homepage/index.tsx, the outer container's px-4 becomes px-0 md:px-4 and the scaffold wrapper's border-x becomes md:border-x. Homepage-only; no other pages or layouts touched.

Scope

  • app/ (hosted site, MCP, Ask AI)
  • packages/cli/
  • packages/mdx-bundler/
  • docs/ (product documentation)
  • Repo / CI / other

Type of change

  • Bug fix
  • New feature
  • Documentation
  • Refactor / chore

Test plan

CSS-class-only change on the marketing homepage. Verify at <768px that the homepage content spans the full viewport width (no left/right border scaffold, no side padding), and at >=768px that the border-x scaffold and px-4 gutter still render as before.

  • bun run check passes locally
  • Tested locally (bun dev, CLI command, or other relevant command)
  • Updated docs/ (if user-facing)
  • Verified on a docs.page URL or local preview (if rendering/routing changed)

Notes for reviewers

No breaking changes. Homepage-only; md: (768px) is the mobile cutoff.


Generated by Claude Code

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01VfnLCEWyr4yiYy7NZiha3A
@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@railway-app railway-app Bot temporarily deployed to docs.page / docs.page-pr-511 July 3, 2026 09:18 Destroyed
@ghesp ghesp marked this pull request as ready for review July 3, 2026 09:25
@ghesp ghesp merged commit 1b0cf25 into main Jul 3, 2026
3 checks passed
@ghesp ghesp deleted the fix/homepage-mobile-full-width branch July 3, 2026 09:26
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.

3 participants