Merge lp:~vthompson/music-app/remix-header-sections into lp:music-app/remix

Proposed by Victor Thompson
Status: Rejected
Rejected by: Victor Thompson
Proposed branch: lp:~vthompson/music-app/remix-header-sections
Merge into: lp:music-app/remix
Diff against target: 36 lines (+5/-10)
2 files modified
MusicNowPlaying.qml (+4/-9)
music-app.qml (+1/-1)
To merge this branch: bzr merge lp:~vthompson/music-app/remix-header-sections
Reviewer Review Type Date Requested Status
Victor Thompson Disapprove
Kill Animals (community) Needs Fixing
Ubuntu Phone Apps Jenkins Bot continuous-integration Needs Fixing
Review via email: mp+238036@code.launchpad.net

Commit message

* Separate Now Playing and Queue views via header sections

Description of the change

This is a design proposal to clarify some of the confusion around the transitions to and from the Now Playing and Queue views.

To post a comment you must log in.
Revision history for this message
Victor Thompson (vthompson) wrote :

#blocked on design

Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Victor Thompson (vthompson) wrote :

The tests that switch views will need to be updated, but this is what the change looks like:

Now playing: http://i.imgur.com/ZGwYDoK.png
Queue: http://i.imgur.com/8BCAsJr.png

Revision history for this message
Kill Animals (kill-animals) wrote :

I think this looks great; Much better imho. Will test and see if it works with my latest mp

Revision history for this message
Kill Animals (kill-animals) wrote :

Rectangle will need to be resized.
http://i.imgur.com/7MMJneN.png

It would be nice to see an animation in this, but this can be done after this presumably gets merged.

Tested it with my branch; no conflicts, and worked nicely :)

review: Needs Fixing
Revision history for this message
Victor Thompson (vthompson) wrote :

This is being rejected. Design has come up with another way of fixing the current issue.

review: Disapprove

Unmerged revisions

663. By Victor Thompson

Design proposal for Now Playing/Queue confusion

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'MusicNowPlaying.qml'
2--- MusicNowPlaying.qml 2014-10-10 17:56:32 +0000
3+++ MusicNowPlaying.qml 2014-10-10 22:41:20 +0000
4@@ -49,15 +49,10 @@
5 }
6
7 head {
8- actions: [
9- Action {
10- objectName: "toggleView"
11- iconName: isListView ? "clear" : "media-playlist"
12- onTriggered: {
13- isListView = !isListView
14- }
15- }
16- ]
17+ sections {
18+ model: [i18n.tr("Now playing"), i18n.tr("Queue")]
19+ onSelectedIndexChanged: isListView = (head.sections.selectedIndex === 1)
20+ }
21 }
22
23 Connections {
24
25=== modified file 'music-app.qml'
26--- music-app.qml 2014-10-09 02:17:21 +0000
27+++ music-app.qml 2014-10-10 22:41:20 +0000
28@@ -1103,7 +1103,7 @@
29 mainPageStack.push(nowPlaying);
30 }
31
32- nowPlaying.isListView = false; // ensure full view
33+ nowPlaying.head.sections.selectedIndex = 0; // ensure Now Playing view (first index)
34 }
35
36 Component.onCompleted: musicToolbar.currentTab = selectedTab

Subscribers

People subscribed via source and target branches