-
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(tokens): update color tokens #5240
fix(tokens): update color tokens #5240
Conversation
Deploy preview for the-carbon-components ready! Built with commit 52d7c74 https://deploy-preview-5240--the-carbon-components.netlify.com |
Deploy preview for carbon-elements ready! Built with commit 6e351e1 |
Deploy preview for carbon-components-react ready! Built with commit 6e351e1 https://deploy-preview-5240--carbon-components-react.netlify.com |
I can't preview the changes but the components not mentioned that also have invalid states are:
|
@aagonzales Deploy preview is now available in case you want to take a look. |
in the spec I think gonzo has a black adornment on the error icon (i.e. the exclamation mark is black in the dark themes for added contrast. is this right @aagonzales ? spec |
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.
I can't see the themes to check Jeannie's concern about the color of the ! but that could be a separate issues anyways.
These all look good now.
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 👍 - Thanks @tw15egan!
Closes #5230
Closes #5024
Updates any text using
$support-01
to$text-error
. Found by searching forcolor: $support-01
in the repo. Also includes small fix forcheckbox
Changelog
Changed
$ui-05
instead of$icon-01
$text-error
instead of$support-01