Bug 495694

Summary: System monitor (or not only) inactive header color with dark-header color scheme
Product: [Frameworks and Libraries] frameworks-kirigami Reporter: evgeniyharchenko.dev
Component: generalAssignee: kdelibs bugs <kdelibs-bugs>
Status: RESOLVED FIXED    
Severity: normal CC: ahiemstra, kdedev, kdelibs-bugs, nate, notmart, postix
Priority: NOR Keywords: regression
Version: 6.7.0   
Target Milestone: Not decided   
Platform: Neon   
OS: Linux   
See Also: https://bugs.kde.org/show_bug.cgi?id=495462
Latest Commit: Version Fixed In: 6.12
Sentry Crash Report:
Attachments: Header-color-using color scheme
Bug

Description evgeniyharchenko.dev 2024-11-01 17:37:20 UTC
Created attachment 175453 [details]
Header-color-using color scheme

SUMMARY
System monitor (or not only) inactive header has wrong colors when using color scheme with [Colors:Header] section. This is a regression. E.g. Plasma 6.1.5 (KDE Frameworks 6.6.0, Qt 6.7.2) does not have this problem. Unfortunately, I can't say exactly when it appeared.

STEPS TO REPRODUCE
1. Download attached color scheme and apply it.
2. Open System Monitor.
3. Click on Edit Page button

OBSERVED RESULT
Header becomes light (wrong colors).

EXPECTED RESULT
Header has normal dark color with inactive-gray buttons on it.

SOFTWARE/OS VERSIONS
KDE Neon 6.2 (X11)
KDE Plasma Version: 6.2.2
KDE Frameworks Version: 6.7.0
Qt Version: 6.8.0
Nvidia driver: 550.120
Comment 1 evgeniyharchenko.dev 2024-11-01 17:38:12 UTC
Created attachment 175454 [details]
Bug

You can look at attached gif for better understanding.
Comment 2 Nate Graham 2024-11-04 19:20:08 UTC
Can reproduce on Wayland. This seems like it could be related to Bug 495462.
Comment 3 Bug Janitor Service 2025-02-24 13:07:51 UTC
A possibly relevant merge request was started @ https://invent.kde.org/frameworks/kirigami/-/merge_requests/1731
Comment 4 Christoph Cullmann 2025-02-25 22:08:14 UTC
Git commit eba0bd6627e7a7a59bd7f3b0d65d85bb40136351 by Christoph Cullmann, on behalf of Arjen Hiemstra.
Committed on 25/02/2025 at 22:04.
Pushed by cullmann into branch 'master'.

platform: Use the old color set for disabled colors in PlatformTheme

If we are temporarily overriding the inherit state because the item is
disabled, copy over the old color set to the new data, to avoid using a
different color set all of a sudden.

M  +8    -0    src/platform/platformtheme.cpp

https://invent.kde.org/frameworks/kirigami/-/commit/eba0bd6627e7a7a59bd7f3b0d65d85bb40136351