Fix mobile horizontal overflow on docs, download, and header - #12
Conversation
- 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.
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThe CSS now wraps long inline code, permits content to shrink within grids, constrains grid tracks with ChangesResponsive CSS behavior
Estimated code review effort: 2 (Simple) | ~10 minutes ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
|
|
ⓘ Qodo reviews are paused because your trial has ended. Ask your workspace admin to add credits to resume reviews. Manage billing |



Summary
.docs-layoutand.download-gridused bare1frgrid tracks withoutminmax(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 sameminmax(0, 1fr)+min-width: 0pattern already used by.article-layout/.proseon the guide pages, across all grid definitions (.grid,.grid-2,.footer-grid,.docs-layout,.download-grid) and their responsive overrides.<code>tokens with no natural break point (e.g.rtmps://stream.example.com:1936/liveinside a guide's<li>) could poke out of their container since inline code had no wrap behavior. Addedoverflow-wrap: break-wordto inlinecodein both stylesheets..nav-cta, consistent with.nav-links— the mobile hamburger menu's "Quickstart" link covers the same destination.Test plan
/,/docs/,/download/,/quickstart/,/guides/*,/legal/) with Playwright at 320px and 375px viewports —document.documentElement.scrollWidthnow equals the viewport width on every page (previously docs/download overflowed to 1134px/609px, and one guide page overflowed by ~40px).Generated by Claude Code
Need help on this PR? Tag
@codesmith-botwith what you need. Autofix is disabled.Summary by CodeRabbit