fix: correct package weekly download stats safe-size#1698
fix: correct package weekly download stats safe-size#1698danielroe merged 1 commit intonpmx-dev:mainfrom
Conversation
|
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 infoConfiguration used: Organization UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThis pull request modifies the WeeklyDownloadStats.vue component, specifically adjusting the vertical space allocated for the weekly sparkline visualisation. The container height is reduced from 110 pixels to 76 pixels, with corresponding responsive adjustments for motion-safe displays from 140 pixels to 92 pixels plus 0.75 rem. The change consists of a single-line modification affecting only the layout dimension with no alterations to data handling, state management, or event logic. Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 1✅ Passed checks (1 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! |
🧭 Context
There's too much space under the weekly download graph on package pages. The problem is that when we were adjusting the dimensions to fix the CLS, we set the height of the entire dropdown (including the heading) to the content. Updated it to fit the content dimensions only
before/after