-
-
Notifications
You must be signed in to change notification settings - Fork 2k
Automatically select light/dark theme based on color-palette-update-notifications
#4366
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
Comments
Lazygit doesn't have a concept of light or dark themes; it is your terminal emulator that does. And as you say, many of them have support for this already or are adding support for it, so I'm not sure what you think is left for lazygit to do here. |
Currently, lazygit only allows you to have one theme configured: Line 112 in 19ac926
You can however override this theme by doing something like this on startup: Line 746 in 19ac926
but that is an inferior solution, as it means that if you launch Lazygit with your "light theme configuration", and then switch your OS to dark mode, any instance of lazygit that was spawned before the switch will be stuck in the wrong theme. The ideal solution would be to have two themes in the config, a dark_theme and a light_theme, and implement the |
I see. So the main feature request is not so much to support automatic switching of light/dark themes, but to add a distinction between light and dark themes in the first place; that's probably the bigger challenge, because it involves some design work (for example, to make it so that the distinction is only relevant for people who care about it, but keep it convenient for those who don't. Personally I don't, and I would like to keep setting my colors unconditionally without having to care about putting them in the right sub-theme). Once we have that, adding the automatic switching shouldn't be a big deal. |
Another quirk is that the theme information should be ideally passed down to the diff engine, e.g. |
Is your feature request related to a problem? Please describe.
Describe the solution you'd like
Adopt the
color-palette-update-notifications
standard.Describe alternatives you've considered
All other solutions are kinda bad. It's good that we finally have a standard for this.
Additional context
Already supported by Ghostty, Kitty, Neovim, Tmux.
WezTerm, Zellij, and Helix are adding support at the moment.
The text was updated successfully, but these errors were encountered: