-
Notifications
You must be signed in to change notification settings - Fork 2.2k
Add new setting to enable/disable dark mode in main menu #16787
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
base: master
Are you sure you want to change the base?
Conversation
|
how complicated would it be to have an auto default option that follows the systems preferences? - most software/websites do this. looks like sdl has a method for this: https://wiki.libsdl.org/SDL3/SDL_SystemTheme (but it is sdl3) |
|
Not hard, but won't implement since we don't use SDL3 by default anywhere at all. |
|
It's a small but solid improvement, and I'm in favor of a feature like this. Personally, I would prefer a night sky with stars, but I am unaware how technically feasible that is. |
IMO this looks much better with stars. Our fork already has this, steal the code from here: IonicCheese/erbium@00dbbf2 (credit to @IonicCheese and @CrazyladMT for the stars) |
Drawing 2D rects without batching is not that efficient and IMO it would be better to extract and share the stars with the in-game sky code anyway. Either way: I agree stars would be nice but let's get this incremental improvement merged first. |
Im not good with Irrlicht/C++ lol, i did what worked ¯_(ツ)_/¯ |
Co-authored-by: Zenon Seth <[email protected]>

adopted #16239
fixes #11308 (though no stars)
How to test