Bug 482664 - System Monitor Sensor widget has wrong colors in colour grid mode
Summary: System Monitor Sensor widget has wrong colors in colour grid mode
Status: RESOLVED FIXED
Alias: None
Product: frameworks-kquickcharts
Classification: Frameworks and Libraries
Component: general (show other bugs)
Version: 6.0.0
Platform: Arch Linux Linux
: NOR normal
Target Milestone: ---
Assignee: Arjen Hiemstra
URL:
Keywords: qt6
: 482877 (view as bug list)
Depends on:
Blocks:
 
Reported: 2024-03-07 08:53 UTC by iansens
Modified: 2024-03-23 03:27 UTC (History)
7 users (show)

See Also:
Latest Commit:
Version Fixed In: 6.0.3


Attachments
Video demo showing bug with both light and dark themes (3.47 MB, video/mp4)
2024-03-07 08:53 UTC, iansens
Details

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