| Summary: | Do not disable next/previous track buttons when current track is being paused | ||
|---|---|---|---|
| Product: | [Applications] Elisa | Reporter: | Nagy Tibor <xnagytibor> |
| Component: | general | Assignee: | Matthieu Gallien <matthieu_gallien> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | nate |
| Priority: | NOR | Keywords: | usability |
| Version First Reported In: | 21.08.1 | ||
| Target Milestone: | --- | ||
| Platform: | Neon | ||
| OS: | Linux | ||
| Latest Commit: | https://invent.kde.org/multimedia/elisa/commit/62d3fd14db7bce19cc738ef9393008ea3b753457 | Version Fixed/Implemented In: | 21.08.2 |
| Sentry Crash Report: | |||
| Attachments: | Repro | ||
Yeah, that's weird. Why *do* we do that, anyway? Notably, the in-app and global Next and Previous shortcuts still work, so this seems like just a bug. I'll fix it. ElisaApplication.playerControl.skipForwardControlEnabled and ElisaApplication.playerControl.skipBackwardControlEnabled seem to be getting set to false inappropriately, but they aren't even blocking the shortcuts anyway... :/ A possibly relevant merge request was started @ https://invent.kde.org/multimedia/elisa/-/merge_requests/289 Git commit 62d3fd14db7bce19cc738ef9393008ea3b753457 by Nate Graham. Committed on 16/09/2021 at 20:02. Pushed by ngraham into branch 'release/21.08'. Don't unnecessarily disable back and forward buttons when paused You might still want to move backwards or forward when the player is paused. FIXED-IN: 21.08.2 M +7 -7 autotests/managemediaplayercontroltest.cpp M +2 -2 src/managemediaplayercontrol.cpp https://invent.kde.org/multimedia/elisa/commit/62d3fd14db7bce19cc738ef9393008ea3b753457 |
Created attachment 141367 [details] Repro SUMMARY Currently when a track is being paused, Elisa always disables the next and previous track buttons. This behavior is kinda counterproductive, I haven't seen any media players other than Elisa doing this. Just because the current track is paused it doesn't mean that jumping backwards/forwards on the current playlist becomes an invalid action to do. Elisa should only disable these buttons when it's at the start/end of the current playlist. SOFTWARE/OS VERSIONS Operating System: KDE neon 5.22 KDE Plasma Version: 5.22.5 KDE Frameworks Version: 5.85.0 Qt Version: 5.15.3