-
Notifications
You must be signed in to change notification settings - Fork 203
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
Update the color of text-secondary
#5151
Comments
@fcoveram this change would make the secondary text as dark as the primary in the light theme. Is that intentional? I thought we would be picking something darker but not as much as the primary. |
I tried your suggestion @fcoveram and to be honest, I did not observe major differences in contrast between |
I tried both colors and |
I agree, @fcoveram, |
Let's go with |
Problem
During the last monthly call (Nov 6th), we talked about the color feeling of
text-secondary
in content-heavy pages in light theme and how the lighter shade feels too soft for reading, as part of #5148We decided to make the
text-secondary
color darker in light theme while keeping the lighter version in dark theme.Description
Change
text-secondary
fromgray-9
togray-12
in light theme. The outcome uses the same gray shade in bothtext
andtext-secondary
.The Figma's Design Library variables were updated with this change.
The text was updated successfully, but these errors were encountered: