Bug 482008

Summary: "Configure Columns" too small and not expandable
Product: [Applications] plasma-systemmonitor Reporter: faveoled
Component: generalAssignee: KSysGuard Developers <ksysguard-bugs>
Status: RESOLVED FIXED    
Severity: normal CC: ahiemstra, bugseforuns, nate, nicolas.fella, plasma-bugs
Priority: NOR    
Version: 6.0.0   
Target Milestone: ---   
Platform: Neon   
OS: Linux   
Latest Commit: https://invent.kde.org/plasma/plasma-systemmonitor/-/commit/55dd85aa3fe81c6909e74ef644e12a731965501c Version Fixed In: 6.1
Sentry Crash Report:
Attachments: Small cfg columns

Description faveoled 2024-02-29 10:19:00 UTC 
Created attachment 166185 [details]
Small cfg columns

SUMMARY
***
NOTE: If you are reporting a crash, please try to attach a backtrace with debug symbols.
See https://community.kde.org/Guidelines_and_HOWTOs/Debugging/How_to_create_useful_crash_reports
***


STEPS TO REPRODUCE
1. Open system monitor
2. Right click on columns, configure button

OBSERVED RESULT
Columns only one line and not expandable

EXPECTED RESULT
Taller columns list and expandable

KDE Neon from 2024 02 28
Comment 1 Patrick Silva 2024-02-29 17:51:01 UTC 
Can reproduce on  Arch Linux.

Operating System: Arch Linux 
KDE Plasma Version: 6.0.0
KDE Frameworks Version: 6.0.0
Qt Version: 6.6.2
Graphics Platform: Wayland
Comment 2 Nate Graham 2024-03-01 06:37:37 UTC 
Can reproduce only for the Applications table on the Overview page.
Comment 3 Bug Janitor Service 2024-04-03 01:29:52 UTC 
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-systemmonitor/-/merge_requests/280
Comment 4 Joshua Goins 2024-04-03 16:24:44 UTC 
Git commit c453edab3bac211e1b4dce45bcee4e7e362d6e41 by Joshua Goins.
Committed on 03/04/2024 at 16:24.
Pushed by redstrate into branch 'master'.

Fix the column configuration dialog being too small on the overview page

This sets the parent to the window if available, instead of the sensor
face which could be tiny.

M  +3    -0    src/faces/applicationstable/contents/ui/FullRepresentation.qml
M  +3    -0    src/faces/processtable/contents/ui/FullRepresentation.qml

https://invent.kde.org/plasma/plasma-systemmonitor/-/commit/c453edab3bac211e1b4dce45bcee4e7e362d6e41
Comment 5 Joshua Goins 2024-04-04 19:33:57 UTC 
Git commit 55dd85aa3fe81c6909e74ef644e12a731965501c by Joshua Goins.
Committed on 04/04/2024 at 19:32.
Pushed by redstrate into branch 'Plasma/6.0'.

Fix the column configuration dialog being too small on the overview page

This sets the parent to the window if available, instead of the sensor
face which could be tiny.


(cherry picked from commit c453edab3bac211e1b4dce45bcee4e7e362d6e41)

M  +3    -0    src/faces/applicationstable/contents/ui/FullRepresentation.qml
M  +3    -0    src/faces/processtable/contents/ui/FullRepresentation.qml

https://invent.kde.org/plasma/plasma-systemmonitor/-/commit/55dd85aa3fe81c6909e74ef644e12a731965501c