Closed
Bug 1015249
Opened 10 years ago
Closed 10 years ago
[Purchased Media] Update to use gaia-header
Categories
(Firefox OS Graveyard :: Gaia, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: yor, Assigned: wilsonpage)
References
Details
Attachments
(1 file)
No description provided.
Blocks: gaia-header
Comment 1•10 years ago
|
||
The "purchased media" app is gaia/apps/fl/ There are changes pending to it in 971618, that might affect the headers works. You might want to wait until that bug lands before updating the headers. The app was never carefully designed by UX, so this might be a funky one to fix up.
Assignee | ||
Comment 2•10 years ago
|
||
(In reply to David Flanagan [:djf] from comment #1) > The "purchased media" app is gaia/apps/fl/ > > There are changes pending to it in 971618, that might affect the headers > works. You might want to wait until that bug lands before updating the > headers. > > The app was never carefully designed by UX, so this might be a funky one to > fix up. Thanks. Can you tell me what the 'purchased media' app is, and how I can launch it?
Assignee | ||
Comment 3•10 years ago
|
||
"To launch the FL app, visit djf.net/t.html, click on "A TCL test suite", and then click on any of the links on that page. That should open the app."
Assignee | ||
Comment 5•10 years ago
|
||
The patch is based on top of bug 1046791 which needs to land first.
Depends on: 1046791
Comment 6•10 years ago
|
||
Wilson, Why did you change the back button from shown by default to hidden by default? Is that a required part of this patch or just an unrelated improvement to avoid a flash of a back button that shows up and then disappears?
Flags: needinfo?(wilsonpage)
Assignee | ||
Comment 7•10 years ago
|
||
(In reply to David Flanagan [:djf] from comment #6) > Wilson, > > Why did you change the back button from shown by default to hidden by > default? Is that a required part of this patch or just an unrelated > improvement to avoid a flash of a back button that shows up and then > disappears? Correct. JavaScript is not being run until 'load' so the user sees a flash of a 'back' button before it is hidden. It seemed to work better to hide and reveal when needed.
Flags: needinfo?(wilsonpage)
Comment 8•10 years ago
|
||
Comment on attachment 8465578 [details]
pull-request (master)
This looks good to me. The Wallpaper and Ringtones apps use the X icon instead of the < icon to cancel the pick activity, and it might be worth fixing that in the FL app now so that it matches those other apps.
Attachment #8465578 -
Flags: review?(dflanagan) → review+
Assignee | ||
Comment 9•10 years ago
|
||
Landed on 'master' https://github.com/mozilla-b2g/gaia/commit/d85bbae28dd9ab9679b42d8d37c84810059e097c
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•