-
Notifications
You must be signed in to change notification settings - Fork 180
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
Kanagawa light mode #13
Comments
yes my friend. I have recently added the support for themes, so expect more to come :) |
Are you working on this? If not, I would give it a shot based on the color palette you use for the dark theme. |
Hi! Yes it's been some time. I am not currently working on this, but I have playing with kanagawa.themes in my todo list. If you want to experiment with this, I'd suggest adding a new theme. The names in the default themes are the ones required to setup the highlights |
Yeah, I've played around a bit earlier. Creating a light theme with the existing palette is proving a bit tricky though, because lot's of the colors in there are lacking a bit of contrast on a bright background. I'll play around some more; once I get something I'm happy with I'll leave you a pull request. |
Any progress on this? I've found myself wanting it more and more as of late :) |
I'm not sure what I "envisioned", but I sure gave it a good try. My original idea was to keep the original palette, and just use those colours to make a light theme, but I had many issues with contrast. Edit: Also, I tried to make this have high contrast, so you can actually read stuff in brightly lit situations. Tell me what you think @rebelot, I might pick this back up |
That isn't bad at all! After some experiments my idea is the following:
Now I'm just wondering if it's worth it to include an HSL module and handle shading on the fly or just add the new colors (maybe with a bit of name refactoring) |
Sorry for the slow responses, pretty busy atm. Im gonna give this another go and start fresh, with the same colors I already played with. Though its probably gonna be a while, as said, Im kinda busy at the moment. |
That looks pretty good! When are you thinking of releasing it? Maybe it could be on a separate branch for now, so people could opt-in to test it for a while :) Or that may be unnecessary as it's enabled/disabled by the user anyway |
Honestly, merging something is better than nothing. Even though it ain't pretty enough, I'd rather stay outside than go inside just because I don't have a light colorscheme. 😉 |
fair enough, I am working on it atm but for now it has its own slightly outdated branch https://github.com/rebelot/kanagawa.nvim/tree/light_theme |
Which option do I need to set to use it? I have that branch checked out |
add |
light_theme branch has been deleted, please update to see a first attempt on light theme on master. Expect some (many) changes in the near future. I hope you can enjoy it :) |
Is there a way to dump the color hexcodes for the light scheme? I realized that I also need to update my Kitty theme for it to be useful. |
:lua =require"kanagawa.themes".light(require"kanagawa.colors".setup({theme = "light"})) |
Thanks, got it to work: Here's the theme file I used for Fish:
I went through the original (dark) file and looked for the hex codes in the output of the command you posted, @rebelot. There are still some bugs in the Kitty theme (per the neofetch output), but it's a start. |
Shall I use this thread to discuss issues I've noticed with the light theme, or shall I create new issues for each? |
Doesn't really matter. Please address the issues you have found |
this is due to fzf color scheme, I think you should use |
Hi! |
done, looks like a god idea:) |
sorry but I can't reproduce... do you have any plugins that might interfere with colorschemes? did you hardcode kanagawa highlights anywhere? |
I just noticed you are using transparent background and dimInactive feature. For this you must also set an appropriate background for your terminal theme, as the current buffer has no background (what you see is your terminal background color) |
Ah okay, I'm pretty much a noob here, how do I see the background? What would be appropriate here? |
Either remove the transparency option or set an appropriate theme for the terminal |
That worked, by setting the transparency to false. I have a question about the dragon theme. Is it possible to try now? I set 'dragon' in the theme field, but didn't notice any change. |
@DMunkei you need to unset the background option to change to a theme different than default or light |
Hey @rebelot, I saw a lot of commotion about breaking changes, so I took the time to update and realized I should step by here and let you know that I have happily been rocking the light/lotus variant for months now. So feel free to close this ticket, unless you have more modifications in mind :) |
Hi, I am currently enjoying the dark colorscheme, just wondering if there are any plans regarding a light mode for those sunny days?
The text was updated successfully, but these errors were encountered: