Bug 448461

Summary:	Can't scroll in playlist using touch screen as it drags around the tracks instead
Product:	[Applications] Elisa	Reporter:	Firlaev-Hans <firlaevhans.fiete>
Component:	general	Assignee:	Matthieu Gallien <matthieu_gallien>
Status:	RESOLVED FIXED
Severity:	normal	CC:	nate
Priority:	NOR	Keywords:	usability
Version:	21.12.0
Target Milestone:	---
Platform:	Fedora RPMs
OS:	Linux
Latest Commit:	https://invent.kde.org/multimedia/elisa/commit/5d8c2630d8b113c3f3a7779e299cd43c2234614a	Version Fixed In:	22.04
Sentry Crash Report:

Description Firlaev-Hans 2022-01-14 14:04:01 UTC

SUMMARY
On my 2in1 laptop when I try to scroll the playlist in Elisa I end up changing the order of the tracks instead.

STEPS TO REPRODUCE
1. Have a touch screen
2. Have enough tracks in Elisa's playlist so that it can be scrolled
3. Try to scroll the playlist using the touch screen (in the normal playlist view, not in party mode)

OBSERVED RESULT
Unless you start scrolling from the album banner, you end up dragging the tracks around and changing their order in the playlist instead of scrolling.

EXPECTED RESULT
Dragging on a touch screen in the playlist area should make it scroll. Dragging the tracks around should require touching and holding them for a short moment.

SOFTWARE/OS VERSIONS
Operating System: Fedora Linux 35
KDE Plasma Version: 5.23.4
KDE Frameworks Version: 5.89.0
Qt Version: 5.15.2
Kernel Version: 5.15.13-200.fc35.x86_64 (64-bit)
Graphics Platform: Wayland (not tested on X11)

Comment 1 Nate Graham 2022-01-14 21:26:47 UTC

Yeah, for the mobile UI there's a separate playlist component to work around this issue. We should ideally fix it and then unify them.

Comment 2 Nate Graham 2022-02-21 18:43:02 UTC

Git commit 5d8c2630d8b113c3f3a7779e299cd43c2234614a by Nate Graham, on behalf of Tranter Madi.
Committed on 21/02/2022 at 18:42.
Pushed by ngraham into branch 'master'.

Make more use of Kirigami stuff

- Use Kirigami stuff instead of handmade ones.
- `alternativeBackground` should only be used when section is not used.
Related: bug 447984, bug 434437
FIXED-IN: 22.04

M  +0    -1    src/CMakeLists.txt
M  +4    -8    src/qml/BasicPlayListAlbumHeader.qml
M  +0    -2    src/qml/ContentView.qml
D  +0    -254  src/qml/DraggableItem.qml
M  +0    -2    src/qml/ElisaMainWindow.qml
M  +1    -3    src/qml/HeaderBar.qml
M  +100  -161  src/qml/MediaPlayListView.qml
M  +209  -249  src/qml/PlayListEntry.qml
M  +32   -50   src/qml/SimplePlayListView.qml
M  +130  -168  src/qml/mobile/MobilePlayListDelegate.qml
M  +29   -3    src/qml/shared/BasePlayListDelegate.qml
M  +0    -1    src/resources.qrc

https://invent.kde.org/multimedia/elisa/commit/5d8c2630d8b113c3f3a7779e299cd43c2234614a