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
USWDS-Site: Update incorrect success-dark color token values #1939
Conversation
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.
Everything checks out! I checked the hex values for fun and it updated appropriately as well.
Should we do a sweep of other token values to make sure everything is matched up?
@mahoneycm I did an initial sweep of the color values for tokens and settings, which is how I caught |
kind: color | ||
- name: success-darker | ||
subsection: state color token | ||
usage: Use grades 70-80. | ||
var: $theme-color-success-darker | ||
default: "'green-cool-60'" | ||
default: "'green-cool-60v'" | ||
kind: color | ||
|
||
- name: Info color family |
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.
Ran a quick test on all color values and this all looks correct now. Thanks for fixing!
Summary
success-dark
fromgreen-cool-50
togreen-cool-50v
andsuccess-darker
fromgreen-cool-60
togreen-cool-60v
to match current USWDS value.success-dark
to vivid variant uswds#4183$theme-color-accent-warm-darker
fromorange-60v
toorange-60
to match current USWDS value.Related issue
Closes #1926
Preview links
Test and review
Outstanding question
There is a mismatch between the default value and the recommended color grades. I have opened up issue #1941 to assess if we should update either the recommended values or the defaults.