Bug 489145

Summary: Adjacent pie chart widgets overlap on a vertical panel 56px thick or thicker
Product: [Plasma] plasmashell Reporter: Piotr Mierzwinski <piotr.mierzwinski>
Component: PanelAssignee: Plasma Bugs List <plasma-bugs>
Status: RESOLVED FIXED    
Severity: normal CC: nate, niccolo.venerandi, piotr.mierzwinski
Priority: NOR Keywords: regression
Version: master   
Target Milestone: 1.0   
Platform: Neon   
OS: Linux   
Latest Commit: Version Fixed In: 6.2.0
Sentry Crash Report:
Attachments: Total CPU Use, Memory Usage_overlay

Description Piotr Mierzwinski 2024-06-24 22:06:12 UTC
Created attachment 170933 [details]
Total CPU Use, Memory Usage_overlay

SUMMARY
Since couple days (despite everyday updates) I observe bug like in the subject. Please check details in the attachment.
Please consider, that couple days ago these plasmoids were placed correctly.

STEPS TO REPRODUCE
1. Set panel to be vertical (left or right side)
2. Put on it two plasmoids: CPU Usage and Memory Usage (I did it before issue happened, so now I'm not sure what will be result)
3. 

OBSERVED RESULT
CPU Usage overlays Memory usage plasmoid placed on the vertical panel

EXPECTED RESULT 
CPU Usage should not overlays Memory usage plasmoid placed on the vertical panel

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: 
(available in About System)
KDE Plasma Version: 6.1.80
KDE Frameworks Version: 6.4.0
Qt Version: 6.7.0

ADDITIONAL INFORMATION
Comment 1 Nate Graham 2024-06-25 23:38:52 UTC
Can reproduce with a vertical panel 56px thick or thicker. At that size, the spacing between them disappears. Any thicker, and the bars start to overlap.
Comment 2 Bug Janitor Service 2024-06-28 15:45:24 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/libksysguard/-/merge_requests/360
Comment 3 Arjen Hiemstra 2024-07-03 15:53:42 UTC
Git commit 13cdeaaf7a330c9febff0ff6a67b5267e15dbeae by Arjen Hiemstra.
Committed on 03/07/2024 at 15:07.
Pushed by ahiemstra into branch 'master'.

faces/pie: Properly limit compact chart height to face height

Otherwise we overflow out of the face bounds.

M  +1    -1    faces/facepackages/piechart/contents/ui/CompactRepresentation.qml

https://invent.kde.org/plasma/libksysguard/-/commit/13cdeaaf7a330c9febff0ff6a67b5267e15dbeae