- Status Closed
- Percent Complete
- Task Type Patches
- Category User Interface
- Assigned To No-one
- Operating System All players
- Severity Low
- Priority Very Low
- Reported Version
- Due in Version Undecided
-
Due Date
Undecided
- Votes
- Private
FS#7078 - Display a message if there is no playlist to view (fix for FS#7027)
This is a fix for the bug reported in FS#7027
Loading...
Available keyboard shortcuts
- Alt + ⇧ Shift + l Login Dialog / Logout
- Alt + ⇧ Shift + a Add new task
- Alt + ⇧ Shift + m My searches
- Alt + ⇧ Shift + t focus taskid search
Tasklist
- o open selected task
- j move cursor down
- k move cursor up
Task Details
- n Next task
- p Previous task
- Alt + ⇧ Shift + e ↵ Enter Edit this task
- Alt + ⇧ Shift + w watch task
- Alt + ⇧ Shift + y Close Task
Task Editing
- Alt + ⇧ Shift + s save task
Ah, petur, I’ve just noticed your comment in the bug report. And you’re probably right. So I’m not sure anymore that this patch fixes the problem!
I still think that a message should be shown. The problem is that the check whether a list is present is wrong. It has false negatives, i.e. says that no list is available when there is one. This e.g. the case when the playback is paused. But if the list was played to the end then there is really no list to view. Resume playback will tell you “Nothing to resume.” So the check must be somehow synchronized with the check in ‘resume playback.’
I had a quick try getting the playlist viewer to load the playlist from the resume info, but for some reason it likes crashing… so its more than a quick fix :(
This seems to work as defined by petur: if resume playback is possible, you’ll be able to view the playlist.
that works? hmm… its bassically exactly what tried but it kept crashing the sim..
anyway, I dont think we need a new lang string for this, “No Playlists” is fine here imo.
This worked for me at least. Other testers are welcome. And I think we need a specific message. “No Playlists” doesn’t fit the context IMO.
well, the reason I say we shouldnt add another string for this is because its not worth the added space imo.. “No Playlists” is close enough to near enough for this context to be fine. thats the only reason I havnt commited it yet.