Bug 475905 - Pressing play on a song that was deleted does not show appropriate UI for this case
Summary: Pressing play on a song that was deleted does not show appropriate UI for thi...
Status: RESOLVED FIXED
Alias: None
Product: Elisa
Classification: Applications
Component: general (show other bugs)
Version: 23.08.2
Platform: Other Linux
: NOR minor
Target Milestone: ---
Assignee: Nate Graham
URL:
Keywords: usability
Depends on:
Blocks:
 
Reported: 2023-10-21 04:59 UTC by Justin Zobel
Modified: 2023-10-24 20:28 UTC (History)
1 user (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Justin Zobel 2023-10-21 04:59:15 UTC
SUMMARY
Pressing play on a song that was deleted does nothing. It should show a message saying the file cannot be found.

STEPS TO REPRODUCE
1. Play a song
2. Exit elisa
3. Delete file
4. Start Elisa and press play

OBSERVED RESULT
Nothing

EXPECTED RESULT
Some sort of message letting the user know why playback failed.

elisa 23.08.1 on postmarketOS Edge (Frameworks 5.110).
Comment 1 Nate Graham 2023-10-23 21:29:05 UTC
Can mostly confirm with git master. The song becomes grayed out, but yeah, some kind of message would be good to show to the user here.
Comment 3 Bug Janitor Service 2023-10-24 20:16:08 UTC
A possibly relevant merge request was started @ https://invent.kde.org/frameworks/kirigami/-/merge_requests/1350
Comment 4 Nate Graham 2023-10-24 20:28:07 UTC
Git commit 10161cdd7d08fd9fb9e0a3b49967ee73586cef6d by Nate Graham.
Committed on 24/10/2023 at 22:23.
Pushed by ngraham into branch 'master'.

Subtly notify user on invalidated songs

When a song becomes invalidated, it greys out and becomes unplayable.
But we don't display any text to the user regarding this, so they have
to guess at what might have happened.
FIXED-IN: 24.02

M  +9    -0    src/qml/shared/BasePlayListDelegate.qml

https://invent.kde.org/multimedia/elisa/-/commit/10161cdd7d08fd9fb9e0a3b49967ee73586cef6d