Closed Bug 1041910 Opened 7 years ago Closed 7 years ago

[Music] Customize Status Bar Color

Categories

(Firefox OS Graveyard :: Gaia::Music, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(b2g-v2.1 verified)

VERIFIED FIXED
2.1 S3 (29aug)
Tracking Status
b2g-v2.1 --- verified

People

(Reporter: mikehenrty, Assigned: mikehenrty)

References

Details

(Whiteboard: [systemsfe])

Attachments

(2 files)

For 2.1, each app will have a customized status bar color. Work to support this was completed in bug 1013913 and bug 1033364. Now we must update each of the apps with the appropriate meta tags as specified in the visual spec:

https://bug1041625.bugzilla.mozilla.org/attachment.cgi?id=8460033
Target Milestone: 2.1 S1 (1aug) → 2.1 S2 (15aug)
Hi Michael, before r+ing this simple patch, just want to clarify that, do we have to also apply the theme when some app is launched as web activity? if so, then we should add the theme meta to the other html(open.html) so that the theme will be applied when music is in activity mode, thanks!
Flags: needinfo?(mhenretty)
That's a good question. Eric, do we need to apply statusbar changes for web activities?
Flags: needinfo?(mhenretty) → needinfo?(epang)
(In reply to Michael Henretty [:mhenretty] from comment #3)
> That's a good question. Eric, do we need to apply statusbar changes for web
> activities?

Hi, how are web activities launched in media apps?  Do they go into the browser?  If that's the case the status bar should match the browser specs. But if they stay in the media app then they should follow the media spec.  Let me know if I'm not making sesne :)
Flags: needinfo?(epang)
(In reply to Eric Pang [:epang] from comment #4)
> (In reply to Michael Henretty [:mhenretty] from comment #3)
> > That's a good question. Eric, do we need to apply statusbar changes for web
> > activities?
> 
> Hi, how are web activities launched in media apps?  Do they go into the
> browser?  If that's the case the status bar should match the browser specs.
> But if they stay in the media app then they should follow the media spec. 
> Let me know if I'm not making sesne :)

For media apps like gallery, video and music, besides the regular mode they also have activity mode to support OPEN or PICK some files, so the cases are like we add some attachments in email/sms, it will launch the media pickers, or to view some attachments with the media viewers, those are the web activities for the media apps.
(In reply to Dominic Kuo [:dkuo] from comment #5)
> (In reply to Eric Pang [:epang] from comment #4)
> > (In reply to Michael Henretty [:mhenretty] from comment #3)
> > > That's a good question. Eric, do we need to apply statusbar changes for web
> > > activities?
> > 
> > Hi, how are web activities launched in media apps?  Do they go into the
> > browser?  If that's the case the status bar should match the browser specs.
> > But if they stay in the media app then they should follow the media spec. 
> > Let me know if I'm not making sesne :)
> 
> For media apps like gallery, video and music, besides the regular mode they
> also have activity mode to support OPEN or PICK some files, so the cases are
> like we add some attachments in email/sms, it will launch the media pickers,
> or to view some attachments with the media viewers, those are the web
> activities for the media apps.

Yup, we will need to update the activity entry points as well. Good catch Dominic!
Comment on attachment 8471251 [details] [review]
[Gaia PR] add theme color for music

I'll remove review flag until we get this fixed.
Attachment #8471251 - Flags: review?(dkuo)
Comment on attachment 8471251 [details] [review]
[Gaia PR] add theme color for music

Good catch again Dominic. I updated the PR with both entry points and made it adhere to the latest spec, which is here:

https://bug1041625.bugzilla.mozilla.org/attachment.cgi?id=8476849
Attachment #8471251 - Flags: review?(dkuo)
Target Milestone: 2.1 S2 (15aug) → 2.1 S3 (29aug)
Comment on attachment 8471251 [details] [review]
[Gaia PR] add theme color for music

The patch looks good to me and thanks for working on this, Michael!
Attachment #8471251 - Flags: review?(dkuo) → review+
master: https://github.com/mozilla-b2g/gaia/commit/803d9142ee42dc4640e47ad0644996e4e2493505
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Status: RESOLVED → VERIFIED
Attached image Music screenshot.png
This issue has been successfully verified on Flame 2.1:
Gaia-Rev        5372b675e018b6aac97d95ff5db8d4bd16addb9b
Gecko-Rev       https://hg.mozilla.org/releases/mozilla-b2g34_v2_1/rev/f34377ae402b
Build-ID        20141127001201
Version         34.0
Device-Name     flame
FW-Release      4.4.2
You need to log in before you can comment on or make changes to this bug.