-
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
chore(button): size audit #10532
chore(button): size audit #10532
Conversation
✔️ Deploy Preview for carbon-react-next ready! 🔨 Explore the source changes: ff85c37 🔍 Inspect the deploy log: https://app.netlify.com/sites/carbon-react-next/deploys/61f977c584bd6e0008b74ab6 😎 Browse the preview: https://deploy-preview-10532--carbon-react-next.netlify.app |
✔️ Deploy Preview for carbon-components-react ready! 🔨 Explore the source changes: ff85c37 🔍 Inspect the deploy log: https://app.netlify.com/sites/carbon-components-react/deploys/61f977c59e710b0008b21887 😎 Browse the preview: https://deploy-preview-10532--carbon-components-react.netlify.app/ |
✔️ Deploy Preview for carbon-elements ready! 🔨 Explore the source changes: ff85c37 🔍 Inspect the deploy log: https://app.netlify.com/sites/carbon-elements/deploys/61f977c5f534600007d9fa01 😎 Browse the preview: https://deploy-preview-10532--carbon-elements.netlify.app |
Is the v11 icon button story supposed to be broken? It's missing the icon. |
Yeah, i noticed that before I even started working on this. Up to ya'll if we want to get this in and then deal with the icon button or get it bundled in this PR. |
Cancel that. I think it's from when i merged something from main |
REF 10295
defaultProps
object and used default params REF...other
to...rest
for consistency with the rest of the codebase.Testing