Bug 482664

Summary: System Monitor Sensor widget has wrong colors in colour grid mode
Product: [Frameworks and Libraries] frameworks-kquickcharts Reporter: iansens
Component: generalAssignee: Arjen Hiemstra <ahiemstra>
Status: RESOLVED FIXED    
Severity: normal CC: ahiemstra, iansens, kde, kdelibs-bugs, liangrui.ch, nate, notmart
Priority: NOR Keywords: qt6
Version: 6.0.0   
Target Milestone: ---   
Platform: Arch Linux   
OS: Linux   
Latest Commit: Version Fixed In: 6.0.3
Attachments: Video demo showing bug with both light and dark themes

Description iansens 2024-03-07 08:53:22 UTC
Created attachment 166552 [details]
Video demo showing bug with both light and dark themes

SUMMARY
***
It looks like the System Monitor Sensor desktop widget has inverted colors for light/dark themes.
***


STEPS TO REPRODUCE
1. Add a System Monitor Sensor widget to the desktop
2. Set display style to "Colour Grid"
3. Set sensor to [Group] CPU -> Clock Frequency

OBSERVED RESULT
Colors are mismatched with the rest of the theme

EXPECTED RESULT
Colors should follow the selected theme


SOFTWARE/OS VERSIONS
Linux/KDE Plasma: (available in About System)
KDE Plasma Version: 6.0.1
KDE Frameworks Version: 6.0.0
Qt Version: 6.6.2

ADDITIONAL INFORMATION
Comment 1 Nate Graham 2024-03-07 21:24:23 UTC
Can reproduce.
Comment 2 Nate Graham 2024-03-10 20:10:00 UTC
*** Bug 482877 has been marked as a duplicate of this bug. ***
Comment 3 Arjen Hiemstra 2024-03-18 11:57:17 UTC
Git commit 331a81803a83eeb983f4abdd8f9de687e783b125 by Arjen Hiemstra.
Committed on 18/03/2024 at 11:54.
Pushed by ahiemstra into branch 'master'.

colorgrid: Use the same background color as pie/bar charts

This should result in the colors working better for different color
themes.

M  +1    -1    faces/facepackages/colorgrid/contents/ui/SensorRect.qml

https://invent.kde.org/plasma/libksysguard/-/commit/331a81803a83eeb983f4abdd8f9de687e783b125
Comment 4 Arjen Hiemstra 2024-03-20 10:46:38 UTC
Git commit 8b9ace2a5c9e8c26af13ee67476a049f7f26d159 by Arjen Hiemstra.
Committed on 20/03/2024 at 10:44.
Pushed by ahiemstra into branch 'Plasma/6.0'.

colorgrid: Use the same background color as pie/bar charts

This should result in the colors working better for different color
themes.
(cherry picked from commit 331a81803a83eeb983f4abdd8f9de687e783b125)

M  +1    -1    faces/facepackages/colorgrid/contents/ui/SensorRect.qml

https://invent.kde.org/plasma/libksysguard/-/commit/8b9ace2a5c9e8c26af13ee67476a049f7f26d159