-
-
Notifications
You must be signed in to change notification settings - Fork 463
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: Better alignment of selected option within select box. #1939
fix: Better alignment of selected option within select box. #1939
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think its the position (left/right) of Icons that needs to be fixed wrt the Issue linked that is to centrally align items inside the label
That was the issue request but my suggestion was just to adjust the icon placement. @delucis was ok with this approach as well. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for tackling this @rgilsimoes! And for your patience waiting on a review.
I noticed this changes the overall spacing of the components, not only the outline positioning:
Before
After
Basically now there is more spacing to the left of the menus, causing unbalanced spacing around the vertical line for example.
I think we should aim to match the current layout and only move the outline. Do you think you could tackle that?
161f8be
to
e4c1c3d
Compare
Yes, I think that’s fine! I think the outline is understood as an overlay on the layout, and is not visible most of the time, so we should optimize the layout for the most common display style (i.e. when the outline is not visible). |
@rgilsimoes Are you planning to continue this PR? This alignment has been bugging me too 😅 Happy to contribute to this PR or create a new branch if you’d like assistance! |
@evadecker Would love the contribution! Feel free to make suggestions on this PR if that’s enough to fix it, or open a new one. Thank you 💖 |
PR up: #2062 |
Sorry for the late reply! Schedule has been tight! Thank's for continuing @evadecker ! |
Description