jnsh / arc-theme

A flat theme with transparent elements (actively maintained fork)
GNU General Public License v3.0
900 stars 77 forks source link

Text contrast #153

Closed lnicola closed 3 years ago

lnicola commented 3 years ago

Hello!

Would it be possible to increase the text contrast It's partly a matter of taste, of course, but text in Adwaita looks much better to me. Look at the labels below:

image

image

This was prompted by an inadvertent change in Firefox. Below, the left rendering is a bug, while the right is how it's supposed to look like with Arc. I think the left version looks much better.

image

Here is Firefox with Adwaita:

image

lnicola commented 3 years ago

On my system, the darker part of the horizontal bar of the "T" in "Themes" is #5c616c with Arc and #000000 with Adwaita. The darker part of the "l" in "Applications" is #7b7f88 vs. #323232. The background is #f5f6f7. That's objectively pretty poor contrast:

image

lnicola commented 3 years ago

Stealing two colors from Adwaita seems to "fix" this:

$text_color: if($variant == 'light', black, white);
$fg_color: if($variant == 'light', #2e3436, #eeeeec);
jnsh commented 3 years ago

This has been discussed ad nauseam, and won't be changed. See https://github.com/jnsh/arc-theme/wiki/Common-problems#text-against-light-background-is-too-lightlow-contrast

Changing the $text_color and $fg_color variables for SASS based themes works, but the text won't be changed for the rest of the themes. The above link has a suggested sed command that can be used to change the text color in all source files, and therefore all themes.

lnicola commented 3 years ago

Ah, sorry. I checked the open issues, but not the closed ones.

I can understand your stance on not making significant changes to the theme. I still think it's a desirable change, but it's fair enough if you don't want that.