Bug 479324

Summary:	The list of process table columns exits dialog bounds on scroll
Product:	[Applications] plasma-systemmonitor	Reporter:	Ilya Bizyaev <bizyaev>
Component:	general	Assignee:	KSysGuard Developers <ksysguard-bugs>
Status:	RESOLVED FIXED
Severity:	normal	CC:	ahiemstra, nate, notmart, plasma-bugs-null
Priority:	NOR	Keywords:	qt6
Version First Reported In:	5.92.0
Target Milestone:	---
Platform:	Compiled Sources
OS:	Linux
Latest Commit:	https://invent.kde.org/plasma/plasma-systemmonitor/-/commit/40fde9d25978cf14365b8e5cde1a63f5fd76d62c	Version Fixed/Implemented In:	6.0
Sentry Crash Report:

Description Ilya Bizyaev 2024-01-02 23:22:59 UTC

STEPS TO REPRODUCE
1.  System Monitor > Processes
2.  “Configure columns…”
3. Scroll back and forth

OBSERVED RESULT
The top and the bottom items of the visible part of the scroll view extend past the dialog's bounds, which looks like flashing.

EXPECTED RESULT
The dialog restricts the boundaries of its children.

SOFTWARE/OS VERSIONS
Operating System: openSUSE Tumbleweed 20231226
KDE Plasma Version: 5.91.90
KDE Frameworks Version: 5.248.0
Qt Version: 6.6.1
Graphics Platform: Wayland

ADDITIONAL INFORMATION

Comment 1 Nate Graham 2024-01-11 22:32:20 UTC

Can reproduce.

Comment 2 Bug Janitor Service 2024-01-11 22:39:22 UTC

A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-systemmonitor/-/merge_requests/246

Comment 3 Nate Graham 2024-01-12 13:54:06 UTC

Git commit 479b9303606333ebca3167f5b32debd9d764c9d3 by Nate Graham.
Committed on 11/01/2024 at 23:37.
Pushed by ngraham into branch 'master'.

Clip scrollable listviews

Otherwise when scrolling them, their backgrounds will briefly escape
beyond the headers and footers of their parent dialogs.
FIXED-IN: 6.0

M  +1    -0    src/page/LoadPresetDialog.qml
M  +2    -0    src/page/MissingSensorsDialog.qml
M  +1    -0    src/page/PageSortDialog.qml
M  +2    -0    src/table/ColumnConfigurationDialog.qml
M  +1    -0    src/table/KillDialog.qml

https://invent.kde.org/plasma/plasma-systemmonitor/-/commit/479b9303606333ebca3167f5b32debd9d764c9d3

Comment 4 Nate Graham 2024-01-12 13:55:10 UTC

Git commit 40fde9d25978cf14365b8e5cde1a63f5fd76d62c by Nate Graham.
Committed on 12/01/2024 at 14:55.
Pushed by ngraham into branch 'Plasma/6.0'.

Clip scrollable listviews

Otherwise when scrolling them, their backgrounds will briefly escape
beyond the headers and footers of their parent dialogs.
FIXED-IN: 6.0


(cherry picked from commit 479b9303606333ebca3167f5b32debd9d764c9d3)

M  +1    -0    src/page/LoadPresetDialog.qml
M  +2    -0    src/page/MissingSensorsDialog.qml
M  +1    -0    src/page/PageSortDialog.qml
M  +2    -0    src/table/ColumnConfigurationDialog.qml
M  +1    -0    src/table/KillDialog.qml

https://invent.kde.org/plasma/plasma-systemmonitor/-/commit/40fde9d25978cf14365b8e5cde1a63f5fd76d62c