fix(ui): prevent empty space in package grid when readme is short#2194
fix(ui): prevent empty space in package grid when readme is short#2194howwohmm wants to merge 1 commit intonpmx-dev:mainfrom
Conversation
The 1280px+ breakpoint had no explicit grid-template-rows, so extra sidebar height was distributed across all rows. Adding auto auto auto 1fr pushes extra space into the readme row instead of the install/vulns area. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe pull request adds an explicit Suggested reviewers
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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 Tip CodeRabbit can use Trivy to scan for security misconfigurations and secrets in Infrastructure as Code files.Add a .trivyignore file to your project to customize which findings Trivy reports. |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
🔗 Linked issue
Fixes #2190
🧭 Context
On packages with short README sections (e.g.
/package/next,/package/@angular/core), the left-column blocks (Get started, vulns) had large empty gaps because the grid rows stretched to match the taller sidebar.📚 Description
The 1280px+ desktop breakpoint was missing
grid-template-rows. Without it, extra height from the sidebar spanning all rows gets distributed evenly across every row — inflating install/vulns cells.Adding
grid-template-rows: auto auto auto 1frpushes all extra space into the readme row (the last one), keeping details/install/vulns compact at the top.The 1024px breakpoint already had this pattern (
auto auto auto auto 1fr).What it doesn't change