ewal / kitty-everforest

Everforest themes for the Kitty terminal
2 stars 0 forks source link

Color mismatches compared to upstream palette #3

Closed antoineco closed 2 years ago

antoineco commented 2 years ago

While looking into the collection of upstream color themes for Kitty, I noticed that somebody had already contributed variations of the Everforest color scheme. So first of all, thanks a lot for this! πŸ™Œ

I noticed that a few of the 16 base colors were misused though. Here are a few examples, illustrated in the screenshot(s) below:

I have "fixed" versions of all 6 variants locally, but before submitting a PR I wanted to share the result with you, and ask whether you would be willing to accept those changes.

before

before before

after

after after

ewal commented 2 years ago

Very nice! Please go ahead and submit the pr :+1: :tada:

ewal commented 2 years ago

@antoineco, thanks again for your contribution, truly appreciated. There are however one thing with the fixed color palette that I believe degrades the end user experience.

With the applied fixes some labels becomes hard to read. I took the liberty to adjust certain colors when I made the themes as seen in the screen shots below, specifically the color used for the git branch label so the contrast would be more profound. I'd prefer to use the strict colors everywhere but it isn't as friendly for the eyes. What's your take on this?

Previously:

old

Updated: image

antoineco commented 2 years ago

Interesting point.

It seems like you're using the pure prompt for Zsh? By default, Pure uses the color 242 for the git:branch component.

On both of the machines I use (macOS with Kitty, Windows 11 with Windows Terminal), the Pure prompt looks like this with the colors submitted in my previous PR:

image

Questions

ewal commented 2 years ago

Nice catch and I believe you're right about Pure. I'm currently using Fish shell with Pure in Linux and on Mac. I'll investigate this further.

ewal commented 2 years ago

This is the output I got from the script you provided:

image

antoineco commented 2 years ago

Ah, the Fish port of Pure uses #brightblack as foreground color for the Git branch: https://github.com/pure-fish/pure/blob/2d07e74567e9190c82ae66c37c34ca86850cd9ac/conf.d/pure.fish#L6

I tried applying the same color in (Zsh) Pure, and you're right. I think the current #brightblack looks OK for autocomplete suggestions, but it's too dark for prompt components (see screenshot below for a comparison of 242 vs. #black vs. #brightblack):

image

I still believe a shell prompt should use better colors than #(bright)white or #(bright)black as foreground color to display information β€” unless it was specifically designed for either a dark or light theme β€” but I also agree that a better contrast could be a sensible default.

ewal commented 2 years ago

I learnt something new today, thank you :beers: I fixed my shell theme by modifying a couple of Pure vars. Lesson learnt from this is to run a non modified terminal if ever creating another theme again :slightly_smiling_face: