Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #1166.
Regarding emulation of day mode (especially on non-Linux platforms):
Taking into account the possibility that the background color is not set in the
.card
styling, I think we need to set the background color of the<body>
element to theQPalette.Window
color value of the system default palette in order to emulate it correctly. However, on Windows (and probably on macOS as well), once thefusion
QStyle is applied to enable night mode, the palette retrieved bystandardPalette()
will be different from that in day mode.On Windows:
This is probably because the default QStyle on Windows and macOS is not
fusion
. So in this PR, the system default palette is stored intheme_manager.default_palette
before thefusion
style is applied, so that theQPalette.Window
color value in day mode can be used in night mode on any platform.