Bug 486993

Summary: System Monitor: Contents shrink when title is disabled
Product: [Applications] plasma-systemmonitor Reporter: Mircea Kitsune <sonichedgehog_hyperblast00>
Component: generalAssignee: KSysGuard Developers <ksysguard-bugs>
Status: RESOLVED FIXED    
Severity: normal CC: ahiemstra, nate, pawel.veselov, plasma-bugs-null, sonichedgehog_hyperblast00
Priority: NOR Keywords: regression
Version First Reported In: 6.0.4   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed/Implemented In: 6.1.0
Sentry Crash Report:
Attachments: Title off, defective behavior
Title on, correct behavior

Description Mircea Kitsune 2024-05-14 00:52:19 UTC
SUMMARY
Issue introduced in Plasma 6. If a System Monitor plasmoid has its title hidden, its contents will be shrunken toward the center of the plasmoid. This isn't immediately visible and happens after plasmashell is restarted.

STEPS TO REPRODUCE
1. Add a System Monitor with the Pie Chart display style and some sensors.
2. Under Appearance disable the Show Title checkbox and save.
3. Restart Plasma, you can do this with the "pkill plasmashell;kstart plasmashell" command.

OBSERVED RESULT
The contents of the window behave as if there's a high amount of padding and shrink to a tiny size in the middle.

SOFTWARE/OS VERSIONS
Operating System: Manjaro Linux 
KDE Plasma Version: 6.0.4
KDE Frameworks Version: 6.1.0
Qt Version: 6.7.0
Kernel Version: 6.9.0-1-MANJARO (64-bit)
Graphics Platform: Wayland
Processors: 16 × AMD Ryzen 7 3700X 8-Core Processor
Memory: 31.3 GiB of RAM
Graphics Processor: AMD Radeon RX 570 Series
Comment 1 Mircea Kitsune 2024-05-14 00:52:57 UTC
Created attachment 169465 [details]
Title off, defective behavior
Comment 2 Mircea Kitsune 2024-05-14 00:53:14 UTC
Created attachment 169466 [details]
Title on, correct behavior
Comment 3 Arjen Hiemstra 2024-05-30 09:55:11 UTC
*** Bug 487326 has been marked as a duplicate of this bug. ***
Comment 4 Bug Janitor Service 2024-05-30 09:57:51 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/libksysguard/-/merge_requests/351
Comment 5 Arjen Hiemstra 2024-06-10 07:51:23 UTC
Git commit ea360656cb60b42bacf6274c69f30ef91785c936 by Arjen Hiemstra.
Committed on 10/06/2024 at 07:49.
Pushed by ahiemstra into branch 'master'.

faces/pie: Remove maximumHeight restriction from Pie chart in full rep

It doesn't really serve a useful purpose and causes the pie to be
limited to the title size even if there is plenty of space for it to
expand.

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

https://invent.kde.org/plasma/libksysguard/-/commit/ea360656cb60b42bacf6274c69f30ef91785c936
Comment 6 Arjen Hiemstra 2024-06-10 08:01:43 UTC
Git commit 7fd880b2c2d1991d7f30d8b681312c4de701aa91 by Arjen Hiemstra.
Committed on 10/06/2024 at 07:58.
Pushed by ahiemstra into branch 'Plasma/6.1'.

faces/pie: Remove maximumHeight restriction from Pie chart in full rep

It doesn't really serve a useful purpose and causes the pie to be
limited to the title size even if there is plenty of space for it to
expand.
(cherry picked from commit ea360656cb60b42bacf6274c69f30ef91785c936)

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

https://invent.kde.org/plasma/libksysguard/-/commit/7fd880b2c2d1991d7f30d8b681312c4de701aa91