WordPress / openverse

Openverse is a search engine for openly-licensed media. This monorepo includes all application code.
https://openverse.org
MIT License
254 stars 203 forks source link

Update the color of `text-secondary` #5151

Open fcoveram opened 2 weeks ago

fcoveram commented 2 weeks ago

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 #5148

We decided to make the text-secondary color darker in light theme while keeping the lighter version in dark theme.

Description

Change text-secondary from gray-9 to gray-12 in light theme. The outcome uses the same gray shade in both text and text-secondary.

The Figma's Design Library variables were updated with this change.

Screenshot of Figma variables of Openverse Design Library

dhruvkb commented 2 weeks ago

@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.

fcoveram commented 2 weeks ago

It is intentional but open to change.

I can try gray-10 and gray-11 in my browser inspector, and perhaps you both @obulat and @dhruvkb do the same and share your preferences here. What do you think?

dhruvkb commented 2 weeks ago

I tried your suggestion @fcoveram and to be honest, I did not observe major differences in contrast between gray-11 and gray-12, (and gray-10 felt a bit lighter and harder to read) so I'm inclined to accept your first suggestion and use gray-12.

fcoveram commented 2 weeks ago

I tried both colors and gray-12 keeps feeling better than lighter versions.

obulat commented 2 weeks ago

I agree, @fcoveram, gray-12 feels more readable than the lighter colors.

fcoveram commented 2 weeks ago

Let's go with gray-12 then. We can always change it in the future if needed.