Uh oh!
There was an error while loading. Please reload this page.
feat: versions page improvements - #2844
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 info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughSummary by CodeRabbit
WalkthroughTemplate and class-based styling were refactored across Current Tags and Version History: Latest and Other tags cards were restyled, the Version History header now contains the filter, virtualised group headers and rows were restyled, and SSR-fallback rendering was updated and made null-safe. ChangesVersions page styling and layout refactor
Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 2 | ❌ 2❌ Failed checks (1 warning, 1 inconclusive)
✅ Passed checks (2 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❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@app/pages/package/`[[org]]/[name]/versions.vue:
- Around line 448-452: Remove the redundant bottom margin on the heading: in the
component containing the parent div with class "flex items-center
justify-between gap-2 mb-3" remove the "mb-3" utility from the child h2 element
(the <h2 class="text-sm text-fg-subtle uppercase mb-3"> that displays {{
$t('package.versions.page_title') }} and the version count) so only the
container controls the spacing.
- Line 541: Replace direct array index accesses of item.versions[0] with a
type-safe guarded access using optional chaining and nullish coalescing (e.g.
item.versions?.[0] ?? '') in the versions.vue component; update the occurrences
referenced (the span with :title="item.versions[0]" and the two uses at lines
noted in the review) to match the SSR fallback pattern already used around lines
709-710 so the UI never reads an out-of-range index.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 9c8381bc-a993-45c4-9f78-2c42f8aa0b9a
📒 Files selected for processing (1)
app/pages/package/[[org]]/[name]/versions.vue
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
trueberryless
commented
Jun 3, 2026
Could you please add a before/after screenshot? |
alexdln
commented
Jun 3, 2026
@trueberryless sure, done |
trueberryless
commented
Jun 3, 2026
@trueberryless I improved the sizes and made it so that on mobile devices it'll just display the badge for deprecated (with sr-only text). I think this will be enough to avoid big changes in the design, and perhaps we can try rethinking the grid for mobile devices (using two rows, shortening the numbers, etc.) ![]() |
trueberryless
left a comment
There was a problem hiding this comment.
Amazing. Only showing the badge is a very clean responsive solution. I like it 👍
Uh oh!
There was an error while loading. Please reload this page.



🔗 Linked issue
Resolves#2796
🧭 Context
Improvements for versions page:
Screenshots
versions field before/after - https://npmx.dev/package/@types/node/versionsMobile before/after - https://npmx.dev/package/@types/node/versions

