Back button doesn't appear when queue is empty
Bug #1268061 reported by
Dennis O'Flaherty
This bug affects 2 people
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Ubuntu Music App |
Fix Released
|
Medium
|
Victor Thompson | ||
Bug Description
1. Start application
2. Switch to the artists tab
3. Select an artist
4. Swipe up from the bottom
Expected Result:
5. Toolbar should show a button to return to artist list.
Actual Result:
5. "No songs queued" label without a back button
The back button only appears when the playlist has something in it.
Related branches
lp:~vthompson/music-app/use-sheet-pattern
- Andrew Hayzen: Approve
- Ubuntu Phone Apps Jenkins Bot: Approve (continuous-integration)
-
Diff: 3373 lines (+1132/-1931)12 files modifiedLibraryListModel.qml (+10/-0)
MusicAlbums.qml (+16/-320)
MusicArtists.qml (+102/-426)
MusicPlaylists.qml (+349/-1091)
MusicStart.qml (+33/-56)
MusicToolbar.qml (+1/-27)
common/AlbumsSheet.qml (+154/-0)
common/SongsSheet.qml (+403/-0)
meta-database.js (+27/-1)
music-app.qml (+15/-9)
tests/autopilot/music_app/emulators.py (+1/-1)
tests/autopilot/music_app/tests/test_music.py (+21/-0)
Revision history for this message
| Victor Thompson (vthompson) wrote | #1 |
| Changed in music-app: | |
| status: | New → Triaged |
| assignee: | nobody → Victor Thompson (vthompson) |
| status: | Triaged → In Progress |
| importance: | Undecided → Low |
| importance: | Low → Medium |
Revision history for this message
| Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote | #2 |
| Changed in music-app: | |
| status: | In Progress → Fix Committed |
| Changed in music-app: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Dennis, thanks for taking the time to fill out a bug report. I agree that this is an issue, it was one that was made more prevalent when the app switched to starting with an empty queue. This will be fixed once code that changes this behavior from a PageStack to using a DefaultSheet [1] is merged. I expect it will be merged shortly.
[1] https:/