-
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
feat(FileUploader): FileUploader size prop update #8462
feat(FileUploader): FileUploader size prop update #8462
Conversation
Deploy preview for carbon-elements ready! Built with commit 12015cf |
Deploy preview for carbon-components-react failed. Built without sensitive environment variables with commit 12015cf https://app.netlify.com/sites/carbon-components-react/deploys/609c6e040878020007476f88 |
2ea7806
to
0c6da29
Compare
3934867
to
a65eff1
Compare
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
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.
lgtm!
2d560e3
to
31f25f7
Compare
ping @andreancardona to review when you get a chance, should be the last PR for the size updates |
bump @andreancardona when you get a sec! |
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!
Refs #8459
Refs #8369
Updates the
FileUploader
and subcomponents to use the standardized size values found across the system.Branched off of #8459 so will need to wait for that to be merged.
Changelog
New
field
maps tomd
small
maps tosm
Testing / Reviewing
Ensure all variants of
FileUploaderItem
andFileUploaderButton
render as expected