-
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(styles): replace old spacing variables with new ones #4889
fix(styles): replace old spacing variables with new ones #4889
Conversation
Deploy preview for carbon-components-react failed. Built with commit c9fa760 https://app.netlify.com/sites/carbon-components-react/deploys/5df8077dde690b00092d2353 |
Deploy preview for the-carbon-components ready! Built with commit c9fa760 https://deploy-preview-4889--the-carbon-components.netlify.com |
Deploy preview for carbon-elements ready! Built with commit c9fa760 |
Note to design review, these are the components that had their spacing tokens updated. Visually they should be the same unless the token used in the past was a mistake 👍
|
I noticed there were a couple places where the old spacing variables were being used, so I replaced them with the new ones 👍
Based on the migration guide here: https://github.com/carbon-design-system/carbon/blob/master/docs/migration/10.x-layout.md#migrating
Changelog
Changed