Bug 495694 - System monitor (or not only) inactive header color with dark-header color scheme
Summary: System monitor (or not only) inactive header color with dark-header color scheme
Status: RESOLVED FIXED
Alias: None
Product: frameworks-kirigami
Classification: Frameworks and Libraries
Component: general (show other bugs)
Version: 6.7.0
Platform: Neon Linux
: NOR normal
Target Milestone: Not decided
Assignee: kdelibs bugs
URL:
Keywords: regression
Depends on:
Blocks:
 
Reported: 2024-11-01 17:37 UTC by evgeniyharchenko.dev
Modified: 2025-03-01 03:15 UTC (History)
6 users (show)

See Also:
Latest Commit:
Version Fixed In: 6.12
Sentry Crash Report:


Attachments
Header-color-using color scheme (3.35 KB, text/plain)
2024-11-01 17:37 UTC, evgeniyharchenko.dev
Details
Bug (1.06 MB, image/gif)
2024-11-01 17:38 UTC, evgeniyharchenko.dev
Details

Note You need to log in before you can comment on or make changes to this 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