fix: improved responsiveness for smaller screens#422
fix: improved responsiveness for smaller screens#422Flo0806 wants to merge 3 commits intonpmx-dev:mainfrom
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
Flo0806
force-pushed
the
fix/detail-mobile-issue
branch
from
adb7fff to
94a14d3
Compare
|
Personally it feels like the docs/code buttons are too close to the chips. I wonder if keeping the version next to the package name would separate them a bit more? 
I guess there's only a ton of space because I am looking at vue and not something with a longer name. |
|
the thing is: what to do with longer package names? handle it separately or use one way. What is your idea? |
|
👁️ I'm adding a 'compare to... |
|
My 2 cents is that, since the package name has such a variable length, it might be worth letting it stand alone and re-arranging other pieces of the UI away from the name |
- Use main's improved mobile responsiveness layout - Keep mobile-only docs/code/compare links - Use updated PackageMetricsBadges component
|
There are a couple of things to check as well, in my opinion.
|
|
@danielroe Are you on it just now? Otherwise I'll fix this finally. |
|
IMO the version should remain where it is, and these 'chips' should behave as a unit, and either all go to the next line, or all stay on the previous line |
|
@danielroe, as you can see from this comment on the issue, after a recent update, they already kinda behave like that #410 (comment) |
|
perfect - I think that works. |
# Conflicts: # app/pages/[...package].vue
|
thank you as always @Flo0806 - sorry a fix got merged first 🙏 |
5 participants
fixes: #410
try new positions for chips and tabs for better responsiveness on smaller screens (detail page)