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

Feature request - light mode #15

Closed
dakennguyen opened this issue Dec 4, 2021 · 8 comments · Fixed by #16
Closed

Feature request - light mode #15

dakennguyen opened this issue Dec 4, 2021 · 8 comments · Fixed by #16

Comments

@dakennguyen
Copy link
Contributor

Hi, I love the combination of one and nord theme. Nord theme doesn't support light mode, but one theme has light mode. Maybe this is too much to ask but what do you think about having a light mode for onenord? At first, we can build it from onelight's colors, and then create our own light version of nord later where we see fit.

If you are interested, I'm willing to give a hand if you need any help 😄

@rmehri01
Copy link
Owner

rmehri01 commented Dec 4, 2021

Hi again! I think that's a good idea, I'll try out some colors and see if I can get something that fits well. Theoretically it shouldn't be too hard to do since I just need to make another color palette but I'd appreciate your help with testing it out later 😄

@dakennguyen
Copy link
Contributor Author

awesome, thank you!! I'll wait for your update then

@rmehri01
Copy link
Owner

rmehri01 commented Dec 4, 2021

Alright, I just changed some colors around and here is what I have so far. I think it's not too bad but some of the contrast seems a bit off. I don't have much experience with light mode themes so what do you think:

Screen Shot 2021-12-04 at 11 27 53 AM

Screen Shot 2021-12-04 at 11 28 03 AM

@dakennguyen
Copy link
Contributor Author

dakennguyen commented Dec 5, 2021

I think it looks nice!!! But not sure if the yellow is too bright or not 🤔 Would you mind pushing the branch so I can further test it on some of my languages?

Ah, also I think we could try to gray out the background a little bit to see how it goes (like these https://levelup.gitconnected.com/10-pretty-light-themes-for-vs-code-80dbf6405f39). One theme background is black so the opposite is white, while our background is not entirely black so gray background for the opposite maybe also make sense!

@rmehri01
Copy link
Owner

rmehri01 commented Dec 5, 2021

Would you mind pushing the branch so I can further test it on some of my languages?

Sure, let me clean it up a bit first and then I'll make a pr for it and tag you.

Ah, also I think we could try to gray out the background a little bit to see how it goes

Yeah I think that might be a good idea, I'll try it out after!

@VKondakoff
Copy link

Is there a way to switch themes "on the fly"? (I'm thinking about switching themes together with the system light/dark mode auto-switching).

Using :set background=light command does not switch the theme and setting vim.o.background in init.lua works only after restarting nvim. How can I switch Onenord theme without restarting nvim?

Thank you!

@rmehri01
Copy link
Owner

rmehri01 commented Dec 8, 2021

Hi! I've just been using :colorscheme onenord and :colorscheme onenordlight, but I think your way is better since there's only 2 themes. I just pushed a change for it!

@VKondakoff
Copy link

Thanks, works like a charm! 👍

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 a pull request may close this issue.

3 participants