Bug 437155 - Scroll views scroll themselves up when completely scrolled down with touchpad
Summary: Scroll views scroll themselves up when completely scrolled down with touchpad
Status: RESOLVED FIXED
Alias: None
Product: libplasma
Classification: Frameworks and Libraries
Component: components (show other bugs)
Version: 5.84.0
Platform: Neon Linux
: NOR normal
Target Milestone: ---
Assignee: Marco Martin
URL:
Keywords: wayland
: 440873 (view as bug list)
Depends on:
Blocks:
 
Reported: 2021-05-15 13:58 UTC by Patrick Silva
Modified: 2023-11-18 04:43 UTC (History)
6 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Patrick Silva 2021-05-15 13:58:20 UTC
STEPS TO REPRODUCE
1. open widgets explorer on Wayland
2. using a touchpad, hover over any widget and scroll the widgets list down completely
3. 

OBSERVED RESULT
when we reach the end of widgets list, widget list scrolls itself up.
I can't reproduce this behavior using a mouse.

EXPECTED RESULT
observed result should not happen

SOFTWARE/OS VERSIONS
Operating System: KDE neon Unstable Edition
KDE Plasma Version: 5.22.80
KDE Frameworks Version: 5.83.0
Qt Version: 5.15.2
Graphics Platform: Wayland

ADDITIONAL INFORMATION
same behavior occurs when I scroll histories of Clipboard and Notifications
applets in system tray.
Comment 1 Nate Graham 2021-05-20 18:38:06 UTC
Whoa! Can reproduce!
Comment 2 Dennis Schridde 2021-06-26 11:22:09 UTC
I noticed something similar when dragging an email in KMail from the mail list into the folder list.  As soon as my mouse cursor is over the folder list, the folders will start scrolling up, until it reaches the top.  Sometimes this stops happening after several attempts, but most of the time there it just keeps doing that on every try.  Is that related or a different bug?
Comment 3 David Redondo 2021-07-22 13:53:26 UTC
I can see the same when you have a notification with scrollbar
Comment 4 Méven Car 2021-08-11 16:47:51 UTC
I couldn't reproduce this with latest master (future plasma 5.23).

I suspect https://invent.kde.org/frameworks/kirigami/-/commit/537e213292cfadb6892a10987cdd9b21df158770 fixed the issue.

I would welcome confirmation that this has been.
Comment 5 Patrick Silva 2021-08-11 16:59:13 UTC
it's still reproducible on neon unstable.
Comment 6 Dennis Schridde 2021-08-11 17:10:21 UTC
(In reply to Dennis Schridde from comment #2)
> I noticed something similar when dragging an email in KMail from the mail
> list into the folder list.  As soon as my mouse cursor is over the folder
> list, the folders will start scrolling up, until it reaches the top. 
> Sometimes this stops happening after several attempts, but most of the time
> there it just keeps doing that on every try.  Is that related or a different
> bug?

I created bug #440874 for this.
Comment 7 Nate Graham 2021-08-11 17:12:20 UTC
Still reproducible, that's unrelated. It's a general problem in the Plasma scroll area. Possibly a different manifestation of Bug 387797.
Comment 8 David Redondo 2021-08-12 07:11:28 UTC
*** Bug 440873 has been marked as a duplicate of this bug. ***
Comment 9 Bug Janitor Service 2021-09-08 14:46:33 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/558
Comment 10 Méven Car 2021-10-12 19:47:15 UTC
Git commit cba7b1f70dc38af3f7c8d5aa5974d6714bad62b3 by Méven Car.
Committed on 12/10/2021 at 15:04.
Pushed by ngraham into branch 'master'.

Port WidgetExplorer' PlasmaExtras.ScrollArea to PC3.ScrollView
Related: bug 426077

M  +14   -10   desktoppackage/contents/explorer/WidgetExplorer.qml
M  +1    -1    desktoppackage/contents/views/Desktop.qml

https://invent.kde.org/plasma/plasma-desktop/commit/cba7b1f70dc38af3f7c8d5aa5974d6714bad62b3
Comment 11 Nate Graham 2021-10-12 20:11:11 UTC
Git commit 5169dff83384cf255ecfbd600036b80763f0885e by Nate Graham, on behalf of Carl Schwan.
Committed on 12/10/2021 at 20:11.
Pushed by ngraham into branch 'master'.

Port away from ScrollArea to ScrollView

In most cases the port is trivial and can even simplify the code like in
the case of the system tray applet. In the case of notifications applet,
this is causing a bigger refactor and it's now also using a TextArea
that provides the automatic scrolling when selecting behavior.

M  +3    -1    applets/batterymonitor/package/contents/ui/PopupDialog.qml
M  +5    -2    applets/clipboard/contents/ui/Menu.qml
M  +3    -1    applets/devicenotifier/package/contents/ui/FullRepresentation.qml
M  +5    -2    applets/notifications/package/contents/ui/FullRepresentation.qml
M  +55   -87   applets/notifications/package/contents/ui/SelectableLabel.qml
M  +26   -28   applets/systemtray/package/contents/ui/HiddenItemsView.qml
M  +6    -3    lookandfeel/contents/runcommand/RunCommand.qml

https://invent.kde.org/plasma/plasma-workspace/commit/5169dff83384cf255ecfbd600036b80763f0885e
Comment 12 Nate Graham 2021-10-12 20:13:23 UTC
Repos that still need to have their scroll views ported from ScrollArea to ScrollView to consider this fully fixes:

- plasma-desktop
- plasma-pa
Comment 13 Patrick Silva 2021-11-27 10:50:57 UTC
Folder View widget added to desktop is affected on neon unstable.
Comment 14 Nate Graham 2022-01-08 04:38:13 UTC
ScrollArea instances that still need to be ported away from to call this fully fixed:

plasma-desktop/desktoppackage/contents/explorer/AppletAlternatives.qml:85
plasma-desktop/desktoppackage/contents/activitymanager/ActivityManager.qml:92
plasma-desktop/applets/window-list/contents/ui/main.qml:76
plasma-desktop/applets/kicker/package/contents/ui/ItemGridView.qml:172
plasma-desktop/applets/kicker/package/contents/ui/ItemMultiGridView.qml:14
plasma-desktop/applets/kicker/package/contents/ui/DashboardRepresentation.qml:800
plasma-desktop/applets/kicker/package/contents/ui/ItemListView.qml:119
plasma-desktop/containments/desktop/package/contents/ui/FolderView.qml:630

plasma-framework/src/declarativeimports/plasmaextracomponents/qml/ExpandableListItem.qml:39
plasma-framework/src/declarativeimports/plasmaextracomponents/qml/PageRow.qml:169

plasma-pk-updates/src/plasma/contents/ui/Full.qml:172

plasma-sdk/themeexplorer/package/contents/ui/delegates/scrollbar.qml:28

plasma-workspace/lookandfeel/contents/windowswitcher/WindowSwitcher.qml:30

kate/addons/sessionapplet/applet/contents/ui/Menu.qml:15

kdeplasma-addons/applets/konsoleprofiles/package/contents/ui/konsoleprofiles.qml:9
kdeplasma-addons/applets/diskquota/package/contents/ui/main.qml:72
kdeplasma-addons/applets/comic/package/contents/ui/FullViewWidget.qml:35
kdeplasma-addons/applets/comic/package/contents/ui/ImageWidget.qml:12
kdeplasma-addons/applets/userswitcher/package/contents/ui/main.qml:154

kup/plasmoid/contents/ui/FullRepresentation.qml:63

kscreenlocker/greeter/fallbacktheme/SessionSwitching.qml:3
Comment 15 Nate Graham 2023-11-18 04:43:49 UTC
This is now fully fixed for Plasma 6.