[refactor] Importing a song can remove the header from the Now Playing view
Bug #1418366 reported by
Victor Thompson
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Ubuntu Music App |
Fix Released
|
Low
|
Andrew Hayzen | ||
Bug Description
In the refactor branch I've verified that the following happens in vivid image #90 if the app is left open and a song is imported: the header on the Now Playing page is not visible. This does not happen in trunk.
To reproduce:
1. Download the "Grooveshark Manager" app (another import source should work)
2. In the Grooveshark app, download a song and open with the Music app
3. When the song plays verify that the header in the Now Playing page allows the user to hit the back button
4. Switch to the Grooveshark app and download another song and open with the Music app
Actual results: Step #4 causes the song to play, but the header is blank (see attached)
Expected results: Step #4 still populates the header in the Now Playing page.
Related branches
lp:~ahayzen/music-app/refactor-fix-1418366-no-empty-page-loader-if-content-hub-importing
- Victor Thompson: Approve
- Ubuntu Phone Apps Jenkins Bot: Approve (continuous-integration)
-
Diff: 14 lines (+3/-1)1 file modifiedapp/music-app.qml (+3/-1)
Revision history for this message
| Victor Thompson (vthompson) wrote | #1 |
Revision history for this message
| Victor Thompson (vthompson) wrote | #2 |
| Changed in music-app: | |
| status: | New → Triaged |
| importance: | Undecided → Low |
| description: | updated |
| Changed in music-app: | |
| assignee: | nobody → Andrew Hayzen (andrew-hayzen) |
| milestone: | none → 3.0 |
| Changed in music-app: | |
| status: | Triaged → In Progress |
Revision history for this message
| Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote | #3 |
| 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.
Marking as "low" since this is a "refactor" series issue only.