You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Reworked the peer dependency layout to use justify-between for better space distribution.
Replaced the optional text badge with an information icon to make the UI more compact, freeing up space to display versions. Even on desktop, dependencies were hard/impossible to read, this improves that significantly.
This change doesn’t strictly depend on replacing the “optional” badge to the icon version, if you prefer keeping the text, we can do so, with slightly less space for versions, but still a big improvement over the current layout.
hmm, yeah, I was thinking about adding a scroll, but It didn't make much sense since it could conflict with click events, the thing is that there are 2 unfixable issues: versions list can be enormous and that will add up with package name (e.g. @organization/package-name and versions 0.0.0 || 1.0.0 || 2.0.0 || 3.0.0... (also why I made optional as icon instead of text) unless you guys are ok with multi line dependencies so we can show versions on 2 lines if needed.
I'm happy to have a more consistent format. justify between seems fine but some people have said it's difficult to trace across so it might be harder work for the eye
truncation seems essential, perhaps with the ability to expand? some peer dep constraints are very long.
(I guess we could wrap ... but often it's not that important to see at a glance every single version for long version strings) https://discord.com/channels/1464542801676206113/1464926468751753269/1467311813191274627
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #532, fixes #550
Reworked the peer dependency layout to use justify-between for better space distribution.
Replaced the optional text badge with an information icon to make the UI more compact, freeing up space to display versions. Even on desktop, dependencies were hard/impossible to read, this improves that significantly.
This change doesn’t strictly depend on replacing the “optional” badge to the icon version, if you prefer keeping the text, we can do so, with slightly less space for versions, but still a big improvement over the current layout.