Bug 489307 - On a thick panel, graphs don't take up enough space
Summary: On a thick panel, graphs don't take up enough space
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: System Monitor (show other bugs)
Version: 6.1.1
Platform: Arch Linux Linux
: NOR normal
Target Milestone: 1.0
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-06-27 12:41 UTC by Notabile
Modified: 2024-07-05 18:37 UTC (History)
3 users (show)

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


Attachments
Vertical panel (377.19 KB, image/png)
2024-06-27 12:41 UTC, Notabile
Details
Horizontal panel (482.42 KB, image/png)
2024-06-27 12:41 UTC, Notabile
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Notabile 2024-06-27 12:41:17 UTC
Created attachment 171081 [details]
Vertical panel

SUMMARY
System monitor widget works fine as a desktop widget, but if I try to put it in a panel (vertical or horizontal), it doesn't display properly: it's shrinked, so it's unreadable.

STEPS TO REPRODUCE
1. create a system monitor widget (I use "grid" as display style)
2. move it to a new panel (vertical or horizontal)

OBSERVED RESULT
widget it's shrinked (small or narrow).

EXPECTED RESULT
The widget should be of adequate size so that the grid and text can be fully readable.

SOFTWARE/OS VERSIONS
Operating System: EndeavourOS 
KDE Plasma Version: 6.1.1
KDE Frameworks Version: 6.3.0
Qt Version: 6.7.2
Graphics Platform: X11 or Wayland


ADDITIONAL INFORMATION
The problem has been present since Plasma version 6.05
Comment 1 Notabile 2024-06-27 12:41:51 UTC
Created attachment 171082 [details]
Horizontal panel
Comment 2 Bug Janitor Service 2024-06-28 16:04:11 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/libksysguard/-/merge_requests/361
Comment 3 Arjen Hiemstra 2024-07-03 15:55:45 UTC
Git commit ede3833dd8486c3f8ba1358c5206f69b1fbd2d59 by Arjen Hiemstra.
Committed on 03/07/2024 at 15:54.
Pushed by ahiemstra into branch 'master'.

faces/grid: Use CompactSensorFace for the compact rep

Going against my own comment, as it turns out that the size hints from
FaceGrid are not as useful as I hoped and are not really fixable either.
So instead just follow the general sizing hints we set up for compact
reps, with a slight tweak to the minimum size to account for the grid.

M  +4    -2    faces/facepackages/facegrid/contents/ui/CompactRepresentation.qml
M  +2    -0    faces/facepackages/facegrid/contents/ui/FaceGrid.qml

https://invent.kde.org/plasma/libksysguard/-/commit/ede3833dd8486c3f8ba1358c5206f69b1fbd2d59