Skip to content
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

add everforest_light, improve everforest_dark string color #1412

Merged
merged 1 commit into from
Jan 3, 2022
Merged

add everforest_light, improve everforest_dark string color #1412

merged 1 commit into from
Jan 3, 2022

Conversation

WindSoilder
Copy link
Contributor

@WindSoilder WindSoilder commented Dec 31, 2021

I'm becoming a new fans of everforest theme....

Adding a new everforest light theme, and make everforest string color to green:
图片

It's just a copy of everforest_dark.toml with different color palette

@archseer
Copy link
Member

\cc @CptPotato since you ported the original theme :)

@CptPotato
Copy link
Contributor

CptPotato commented Dec 31, 2021

Thanks for the ping 👍

I'm not a user of light themes so I can't say how well it compares to others. However, it does look good to me in terms of contrast and readability.

The string color is a good point. I chose to deviate from the original theme and change it to a light grey. This was because both functions and strings shared the same color and lacked visual separation in my opinion. It was very noticable with "string heavy" code.

That being said, changing the color back to green makes the dark theme a more "correct" port of the original and I'm fine with that. I am currently using a customized version (increased contrast) of this theme, so I get to keep my preferred look anyway :)

@archseer
Copy link
Member

archseer commented Jan 3, 2022

Okay 👍🏻 the custom theme looks nice, you can upstream it or link to the gist from the wiki if you want?

@archseer archseer merged commit 609f736 into helix-editor:master Jan 3, 2022
@CptPotato
Copy link
Contributor

Okay 👍🏻 the custom theme looks nice, you can upstream it or link to the gist from the wiki if you want?

That's a possibility.
I also thought about creating a separate repo where I keep my themes in case I end up customizing or porting more. I might give sonokai a shot next (it's a lowered contrast version of monokai pro which is already included in helix).

A separate repo would also prevent the additional maintenance burden and bloat, even though it means that themes are a bit harder to discover.

@WindSoilder WindSoilder deleted the everforest_improve branch January 3, 2022 23:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants