-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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 CustomSelectControl component dropdown arrow spacing #22925
Conversation
Size Change: +383 B (0%) Total Size: 1.12 MB
ℹ️ View Unchanged
|
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.
Looks good, thanks for working this into a separate PR.
Let's update to use the $icon-size variable then merge.
Co-authored-by: Daniel Richards <daniel.richards@automattic.com>
Co-authored-by: Daniel Richards <daniel.richards@automattic.com>
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.
Notes seem addressed now
This PR fixes an issue identified in #18869 (comment) whereby if the selected option in a
CustomSelectControl
component is a particularly long string then if overlays the down arrow icon.This PR adds sufficient padding to the RHS of the control to ensure the icon has room to display (see fix below).
How has this been tested?
Testing Instructions
On Master
CustomSelectControl
..On this PR
npm run dev
.npm run storybook:dev
.CustomSelectControl
stories in Storybook. Chose the "Long labels" story.Types of changes
Bug fix (non-breaking change which fixes an issue).
Checklist: