docs: lay hero tiles out in even columns when the hero stacks - #1308
Conversation
Wrapping left ragged trailing rows once the hero centers its column. A two-column grid keeps the tiles aligned, dropping to one column on phones where a label and its pill cannot share a row. Co-authored-by: jdx <jdx@users.noreply.github.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Central YAML (base), Organization UI (inherited) Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 4 included reviews per hour; 2 remain after this review. 📝 WalkthroughWalkthroughThe mobile hero layout now supports shrinking text, independently centered actions, and a capped responsive tile grid. The grid centers tile content and allows an odd final tile to span both columns. ChangesResponsive hero layout
Estimated code review effort: 2 (Simple) | ~10 minutes Merge Risk: ⚪ Minimal · up to This localized CSS change adjusts hero tile layout at narrower widths, with no actionable merge-blocking risk remaining beyond normal checks and review. Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
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 |
The home page hero tiles wrap in a flexbox, so once the hero stacks and centers its column the rows come out ragged — three frameworks then a lone JavaScript, four tools then a lone
usage-cli.Below 960px the tile groups now use a grid instead. Columns are even, so every row lines up at any tile count, and an odd last tile spans the row rather than sitting alone in one column.
Details
Rustplus itsexperimentalpill, measured at 201px), so phones fall back to one column instead of squeezing a label against its pill. Two columns from about 450px up, one below..usage-hero-textgetsalign-self: stretchandmin-width: 0. The stacked hero centers its column, which makes the text block shrink-to-fit; without this the grid widens it past the viewport andoverflow-x: clipon the hero cuts the tiles off at both edges.Verification
Built the site and measured tiles per row over CDP at each width. Before:
[[3,1],[4,1]]at 900px and 600px. After:[[2,2],[2,2,1]]at 900px and 600px,[[1,1,1,1],[1,1,1,1,1]]at 430px. No horizontal document overflow at 320, 375, 430, 480, 600, 900, or 961px.Before, at 900px:
Hero tiles wrapping into ragged rows of three and four
After, at 900px:
Hero tiles in even two-column rows with usage-cli spanning the last row
After, at 430px:
Hero tiles in a single column on a phone-width viewport
To show artifacts inline, enable in settings.
Summary by CodeRabbit