feat: use a horizontal bar chart for the version distribution - #3281
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository: npmx-dev/npmx.dev/.coderabbit.yaml Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
Included review availability: Your plan provides up to 8 included reviews per hour; 5 remain after this review. 📝 SummarySummary by CodeRabbit
WalkthroughThe version distribution chart now uses horizontal bars. A settings toggle can order bars by downloads. Chart rendering, export, watermark, tooltip, legend, alt-text handling, translations, tests, and the ChangesVersion distribution chart
Sequence Diagram(s)sequenceDiagram
participant PackageStats
participant VersionDistribution
participant VueUiHorizontalBar
PackageStats->>VersionDistribution: provide version distribution data
VersionDistribution->>VersionDistribution: build and optionally sort barDataset
VersionDistribution->>VueUiHorizontalBar: render horizontal bars and barConfig
VueUiHorizontalBar-->>VersionDistribution: render tooltip, legend, or watermark slots
Priority: ➖ Normal Severity of issue fixed: Medium Merge Risk: ⚪ Minimal · up to The stats page now uses horizontal version bars with optional download ordering; no concrete production-impacting issue is identified, so the change is mergeable with normal checks. 🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✨ 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 |
Lunaria Status Overview🌕 This pull request will trigger status changes. Learn moreBy default, every PR changing files present in the Lunaria configuration's You can change this by adding one of the keywords present in the Tracked Files
Warnings reference
|
Codecov Report❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
e18e dependency analysisNo dependency warnings found. |
🔗 Linked issue
Resolves #3280
🧭 Context
In the stats page, the current version distribution chart uses vertical bars, which was ok when the chart used to be inside a dialog and the vertical real estate was constrained. However this layout comes with serious limitations (the data is hard to read, and bars become very narrow when minors are laid out for an old package).
Since the stats page now offers vertical real estate, it is time to migrate this chart to a horizontal bar chart, which also makes it easy to display data labels next to the bars, and allows to display n number of datapoints without sacrificing on readability.
📚 Description
VueUiHorizontalBarcomponent used here