Skip to content

Fix mobile horizontal overflow on docs, download, and header - #12

Merged
AlexanderWagnerDev merged 1 commit into
mainfrom
claude/docs-mobile-optimization-oj15ya
Aug 14, 2026
Merged

Fix mobile horizontal overflow on docs, download, and header#12
AlexanderWagnerDev merged 1 commit into
mainfrom
claude/docs-mobile-optimization-oj15ya

Conversation

@AlexanderWagnerDev

@AlexanderWagnerDev AlexanderWagnerDev commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Summary

  • The docs and download pages horizontally overflowed the mobile viewport (docs scrolled to ~3x screen width at 375px). Root cause: .docs-layout and .download-grid used bare 1fr grid tracks without minmax(0, 1fr), so long <pre>/table content forced the grid item — and with it the whole page — wider than the viewport (classic CSS Grid blowout). Fixed by applying the same minmax(0, 1fr) + min-width: 0 pattern already used by .article-layout/.prose on the guide pages, across all grid definitions (.grid, .grid-2, .footer-grid, .docs-layout, .download-grid) and their responsive overrides.
  • Long inline <code> tokens with no natural break point (e.g. rtmps://stream.example.com:1936/live inside a guide's <li>) could poke out of their container since inline code had no wrap behavior. Added overflow-wrap: break-word to inline code in both stylesheets.
  • The header's "Run with Docker" button wrapped onto two lines and looked cramped below ~370px viewport width. It's now hidden under the existing 680px mobile breakpoint alongside the rest of .nav-cta, consistent with .nav-links — the mobile hamburger menu's "Quickstart" link covers the same destination.

Test plan

  • Ran a local PHP server and checked every page (/, /docs/, /download/, /quickstart/, /guides/*, /legal/) with Playwright at 320px and 375px viewports — document.documentElement.scrollWidth now equals the viewport width on every page (previously docs/download overflowed to 1134px/609px, and one guide page overflowed by ~40px).
  • Screenshotted the docs page, download page, and the mobile nav menu at 375px to confirm layout and that the mobile menu still provides equivalent navigation.

Generated by Claude Code


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

Summary by CodeRabbit

  • Bug Fixes
    • Improved text wrapping for inline code and other content, preventing overflow in narrow layouts.
    • Enhanced responsive grid behavior to keep content within its containers.
    • Updated mobile navigation so call-to-action controls are fully hidden on small screens.

- docs-layout and download-grid used fixed 1fr grid tracks without
  minmax(0, 1fr), letting long code/table content blow the grid item
  past the viewport (docs and download pages scrolled ~3x screen width
  on mobile). Applied the same minmax(0, 1fr) + min-width:0 pattern
  already used by article-layout to all grid definitions.
- Long inline <code> tokens (e.g. rtmps://stream.example.com:1936/live)
  could overflow their paragraph/list item since inline code had no
  wrap behavior; added overflow-wrap: break-word.
- The header's "Run with Docker" button wrapped to two lines below
  ~370px viewport width; now hidden under 680px alongside the rest of
  nav-cta, matching nav-links (Quickstart in the mobile menu covers
  the same destination).

Verified with Playwright at 320/375px across all pages: no page
overflows the viewport anymore.
@coderabbitai

coderabbitai Bot commented Aug 14, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 0e50c50d-3b47-4c99-8427-13fad841f37b

📥 Commits

Reviewing files that changed from the base of the PR and between ece46df and 4ea1ae1.

📒 Files selected for processing (2)
  • assets/css/content.css
  • assets/css/style.css

📝 Walkthrough

Walkthrough

The CSS now wraps long inline code, permits content to shrink within grids, constrains grid tracks with minmax(0, ...), and hides mobile navigation links and calls to action.

Changes

Responsive CSS behavior

Layer / File(s) Summary
Content wrapping safeguards
assets/css/content.css, assets/css/style.css
Inline code can wrap long content. Documentation content and download cards can shrink within their grid containers.
Responsive grid and navigation sizing
assets/css/style.css
Feature, footer, documentation, download, and responsive grids use zero-minimum tracks. Narrow layouts hide .nav-links and .nav-cta while keeping the navigation toggle visible.

Estimated code review effort: 2 (Simple) | ~10 minutes

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch claude/docs-mobile-optimization-oj15ya

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@sonarqubecloud

Copy link
Copy Markdown

@AlexanderWagnerDev
AlexanderWagnerDev marked this pull request as ready for review August 14, 2026 23:53
@qodo-code-review

Copy link
Copy Markdown

ⓘ Qodo reviews are paused because your trial has ended. Ask your workspace admin to add credits to resume reviews. Manage billing

@AlexanderWagnerDev
AlexanderWagnerDev merged commit b6d12e5 into main Aug 14, 2026
6 of 7 checks passed
@AlexanderWagnerDev
AlexanderWagnerDev deleted the claude/docs-mobile-optimization-oj15ya branch August 14, 2026 23:54
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