[music2] [1] Re-layout the whole ui

RESOLVED INVALID

Status

defect
RESOLVED INVALID
6 years ago
5 years ago

People

(Reporter: dkuo, Assigned: eshapiro)

Tracking

unspecified
1.3 Sprint 5 - 11/22
ARM
Gonk (Firefox OS)
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

This is the first action item for music2 to adapt the phone/tablet ui, basically this work is to have the ui structure(modify the index.html and css) so that we can implement the features base on it.

This item should cover:
1-1. Handle tablet and phone sizes.
1-2. Handle switching between landscape and portrait.
Blocks: 936397
Blocks: 936405
Blocks: 936419
Posted file pull request
Attachment #830180 - Flags: review?(dkuo)
Evan, thanks for working on this so quickly, I got the review request and reviewing it, thanks!
Comment on attachment 830180 [details]
pull request

Evan,

The patch looks good to me, however, some ui changes is not sync with the latest spec. Also, the new js files that you newly added are not jslinted, there are several lint errors. And since we are using strict mode in gaia, you should add "use strict" at the start of every js file, so I am cancelling the review request.

Here is how to install and use the gjslint:
https://developers.google.com/closure/utilities/docs/linter_howto

To speed up the progress, I am addressing those issue and will land it for you because after this first patch is landed, Jim and I can start to implement [2], [3] and [4] in parallel and won't be blocked by this bug.
Attachment #830180 - Flags: review?(dkuo)
Evan,

This is the modified patch that I cherry-picked from your branch with issues addressed, I think the patch covers 1-2 but 1-1, but it's enough for the basic structure so I will land it first, please work on 1-1 and I will leave this bug open so that we can still trace it, thanks.
Attachment #831334 - Flags: review+
1-2 landed on master: 7bb12c33a7df34e4f210879ceebb0bc3e792a01d
Assignee: nobody → eshapiro
this is subset of Music2 re-layout for tablet, tag it 1.3+ with target milestones.
blocking-b2g: --- → 1.3+
Target Milestone: --- → 1.3 Sprint 5 - 11/22
Whiteboard: [Flatfish only]
Whiteboard: [Flatfish only] → [Flatfish only][developer+]
blocking-b2g: 1.3+ → ---
Whiteboard: [Flatfish only][developer+] → [flatfish][TCP]
Whiteboard: [flatfish][TCP]
Closing all music2-specific bugs. The current plan is to gradually incorporate bits of music2 into the current music app, since it will take too long to achieve feature parity in music2.
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.