-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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(search): design tweaks to search component #6584
fix(search): design tweaks to search component #6584
Conversation
Deploy preview for carbon-elements ready! Built with commit b9c4b70 |
Deploy preview for carbon-components-react ready! Built with commit b9c4b70 https://deploy-preview-6584--carbon-components-react.netlify.app |
Deploy preview for carbon-elements ready! Built with commit 9e4a776 |
Deploy preview for carbon-components-react ready! Built with commit 9e4a776 https://deploy-preview-6584--carbon-components-react.netlify.app |
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 great! thank you 👍🏻
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 to me
Closes #6578
Changes close + search icon to
16px
in all size variantsAdjusts padding based on size variant
Changelog
Changed
body-short-01
Removed
Close20
is removed from theSearch
componentTesting / Reviewing
Ensure
Search
renders properly for each size variant