Bug 448461 - Can't scroll in playlist using touch screen as it drags around the tracks instead
Summary: Can't scroll in playlist using touch screen as it drags around the tracks ins...
Status: RESOLVED FIXED
Alias: None
Product: Elisa
Classification: Applications
Component: general (show other bugs)
Version: 21.12.0
Platform: Fedora RPMs Linux
: NOR normal
Target Milestone: ---
Assignee: Matthieu Gallien
URL:
Keywords: usability
Depends on:
Blocks:
 
Reported: 2022-01-14 14:04 UTC by Firlaev-Hans
Modified: 2022-02-21 18:43 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In: 22.04
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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