Skip to content

Conversation

@DDeenis
Copy link
Contributor

@DDeenis DDeenis commented Jan 30, 2026

Makes the versions list scrollable, so the user doesn't have to scroll the whole page to see a long list of versions.

Before:
image

After:
image

"Other versions" list is now also scrollable:
image

@vercel
Copy link

vercel bot commented Jan 30, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
npmx.dev Ready Ready Preview, Comment Feb 1, 2026 0:19am
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
docs.npmx.dev Ignored Ignored Preview Feb 1, 2026 0:19am
npmx-lunaria Ignored Ignored Feb 1, 2026 0:19am

Request Review

@DDeenis DDeenis changed the title Make versions list scrollable if there are too many elements style: make versions list scrollable if there are too many elements Jan 30, 2026
@danielroe
Copy link
Member

danielroe commented Jan 30, 2026

I'm not sure about this...

for a start, maybe skip this behaviour on mobile?

and I guess, on desktop we also need some kind of indication that there's content within to scroll?

@DDeenis
Copy link
Contributor Author

DDeenis commented Jan 30, 2026

@danielroe sure, the mobile change makes sense. About indication though, isn't the scrollbar enough? We can make the scroll container a bit bigger/smaller so a subtle cropped item could be visible, indicating that there is more to scroll
image

I won't push hard on this change though, it just was really frustrating for me that I have to scroll the whole page to see the full list of versions

@danielroe
Copy link
Member

danielroe commented Jan 30, 2026

you won't see the scrollbar on MacOS/iOS unless the user starts to scroll it

…e scroll container a little bigger to show that there are more items
@DDeenis
Copy link
Contributor Author

DDeenis commented Jan 30, 2026

I see. Good to know - that is a problem.
I've made the requested change (mobile), and slightly increased the container height to partly show the last element.
Will leave a decision up to you @danielroe - feel free to close if you feel that this change is unnecessary

@serhalp serhalp added the idea Suggestion or idea, not yet vetted/decided label Jan 31, 2026
- Combined scrollable versions styling with main's padding
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

idea Suggestion or idea, not yet vetted/decided

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants