Bug 452091 - KColorSchemeManager: Selecting default color scheme doesn't respect dark mode on Windows
Summary: KColorSchemeManager: Selecting default color scheme doesn't respect dark mode...
Status: RESOLVED FIXED
Alias: None
Product: frameworks-kconfigwidgets
Classification: Frameworks and Libraries
Component: general (show other bugs)
Version: 5.92.0
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: kdelibs bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-03-30 22:31 UTC by Nicolas Fella
Modified: 2022-05-02 15:51 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Nicolas Fella 2022-03-30 22:31:54 UTC
STEPS TO REPRODUCE
1. In Windows system settings configure dark colors for apps
2. Open Kate. It has dark colors
3. Select Settings > Color Scheme > Breeze Light. Now colors are light
4. Select Settings > Color Scheme > Default

OBSERVED RESULT
Kate has light colors

EXPECTED RESULT
Kate has dark colors

SOFTWARE/OS VERSIONS
Windows: 10
KDE Frameworks Version: 5.92
Qt Version: 5.15

ADDITIONAL INFORMATION
After closing and reopening Kate colors are dark as expected
Comment 1 Nate Graham 2022-03-31 21:53:28 UTC
Fixed by Thomas Friedrichsmeier in Frameworks 5.93 with https://invent.kde.org/frameworks/kconfigwidgets/-/commit/17b367ddc5c7e0b13703a247a42ff858d92e9780!
Comment 2 Nicolas Fella 2022-03-31 22:10:46 UTC
No, that's a different thing
Comment 3 Nicolas Fella 2022-05-02 15:51:37 UTC
Git commit 9f7b8a902d9654520f4fac5c870d5b1174c893ff by Nicolas Fella.
Committed on 29/04/2022 at 23:15.
Pushed by nicolasfella into branch 'master'.

[kcolorschememanager] Rework and improve auto theme switching

Extract the automatically determined color scheme into a function

Avoid applying the scheme twice on startup

Fix activating the default scheme when system is dark

M  +54   -29   src/kcolorschememanager.cpp
M  +0    -2    src/kcolorschememanager.h
M  +16   -3    src/kcolorschememanager_p.h
M  +4    -1    src/windowsmessagesnotifier.cpp

https://invent.kde.org/frameworks/kconfigwidgets/commit/9f7b8a902d9654520f4fac5c870d5b1174c893ff