Uh oh!
There was an error while loading. Please reload this page.
fix(ui): remove empty space on the package page when the readme is short - #2194
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.
1 Skipped Deployment
|
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)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughThe PR updates the 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 |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
It seems to have been added in 2ad8034 when there were five rows, but then the header row was removed and so this one is no longer needed
Uh oh!
There was an error while loading. Please reload this page.
🔗 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