Bug 465980

Summary: folder view in panel: grabbing scrollbar moves files instead
Product: [Plasma] plasmashell Reporter: S. Christian Collins <s_chriscollins>
Component: Desktop icons & Folder View widgetAssignee: Plasma Bugs List <plasma-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: hein, nate
Priority: NOR    
Version First Reported In: 5.27.0   
Target Milestone: 1.0   
Platform: Neon   
OS: Linux   
Latest Commit: Version Fixed/Implemented In: 5.27.2
Sentry Crash Report:
Attachments: GIF file showing the bug

Description S. Christian Collins 2023-02-18 05:19:49 UTC
Created attachment 156411 [details]
GIF file showing the bug

SUMMARY
***
If the folder view widget is added to the panel and the list of files in the popup is long enough to show a scrollbar, the scrollbar overlaps the file highlight rectangle following the cursor. If you attempt to drag the scrollbar while a file is highlighted, it will instead drag the underlying file. Please see the attached GIF file for an example.

I didn't notice this bug until after the upgrade to Plasma 5.27.0, so I'm guessing it's a pretty recent regression.
***

STEPS TO REPRODUCE
1. Add Folder View widget to the panel pointing to a folder with lots of files.
2. Click the Folder View icon to show the files in a popup.
3. Move the cursor across the file area over to the scrollbar (this will cause files to be highlighted along the way) and drag the scrollbar down/up.

OBSERVED RESULT
Instead of dragging the scrollbar, the underlying file is dragged.

EXPECTED RESULT
I should be able to scroll the view without accidentally grabbing a file.

SOFTWARE/OS VERSIONS
Linux: KDE neon 5.27
KDE Plasma Version: 5.27.0
KDE Frameworks Version: 5.103.0
Qt Version: 5.15.8
Comment 1 Nate Graham 2023-02-21 23:40:24 UTC
The issue appears to be that the scrollbar overlaps the file view. Will investigate.
Comment 2 Bug Janitor Service 2023-02-24 18:56:06 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/1401
Comment 3 Nate Graham 2023-02-24 19:02:20 UTC
Git commit 24b8dc1bc5e08df3a998630f148f25b271b15ba8 by Nate Graham.
Committed on 24/02/2023 at 18:49.
Pushed by ngraham into branch 'master'.

Folder View: fix scrollbar overlapping list view items

This view has a DIY scrollview implementation with manually-attached
scrollbars, which means it becomes responsible for making the content
dodge the scrollbars. Before, the list representation didn't; now, it
does.
FIXED-IN: 5.27.2

M  +4    -2    containments/desktop/package/contents/ui/FolderView.qml

https://invent.kde.org/plasma/plasma-desktop/commit/24b8dc1bc5e08df3a998630f148f25b271b15ba8
Comment 4 Nate Graham 2023-02-24 19:03:52 UTC
Git commit 4e6eff892000d159b282afcec53df7135e0602b7 by Nate Graham.
Committed on 24/02/2023 at 19:03.
Pushed by ngraham into branch 'Plasma/5.27'.

Folder View: fix scrollbar overlapping list view items

This view has a DIY scrollview implementation with manually-attached
scrollbars, which means it becomes responsible for making the content
dodge the scrollbars. Before, the list representation didn't; now, it
does.
FIXED-IN: 5.27.2


(cherry picked from commit 24b8dc1bc5e08df3a998630f148f25b271b15ba8)

M  +4    -2    containments/desktop/package/contents/ui/FolderView.qml

https://invent.kde.org/plasma/plasma-desktop/commit/4e6eff892000d159b282afcec53df7135e0602b7