When launching Settings via an activity, the status bar color scheme does not change

VERIFIED FIXED in Firefox OS v2.2

Status

VERIFIED FIXED
4 years ago
4 years ago

People

(Reporter: khuey, Assigned: apastor)

Tracking

({regression})

unspecified
2.1 S8 (7Nov)
ARM
Gonk (Firefox OS)
regression

Firefox Tracking Flags

(b2g-v2.1 unaffected, b2g-v2.2 verified)

Details

(Whiteboard: [systemsfe])

Attachments

(3 attachments)

STR:

1. Launch SMS
2. Click the ... menu
3. Click Settings

Expected results:

The status bar should use the "dark" color scheme, like it does if you launch the settings app directly

Actual results:

The status bar is still using the "light" color scheme, as it does for the SMS app.
qawanted for branch checks
Keywords: qawanted
The bug repros on Flame 2.2 engineering with shallow flash.
Actual result: After selecting Settings from the Messaging app's ... menu, the status bar text will be light making it hard to read on the white bar.

BuildID: 20141029054810
Gaia: a9a847920b51b79c4ad4ad339f0a005777a6228c
Gecko: c6989e473f97
Platform Version: 36.0a1
Firmware Version: V188
User Agent: Mozilla/5.0 (Mobile; rv:36.0) Gecko/36.0 Firefox/36.0

--------------------------------------------------------------------------------------------------------

The bug does not repro on Flame 2.1 engineering with shallow flash.
Actual result: After selecting Settings from the Messaging app's ... menu, the status bar text will be dark making it easy to read on the white bar.

BuildID: 20141029082611
Gaia: 2099fb0df60548cf7d4afc367f5048622cc29b3e
Gecko: f02f3fbd0bb0
Platform Version: 34.0
Firmware Version: V188
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0
QA Whiteboard: [QAnalyst-Triage?]
status-b2g-v2.1: --- → unaffected
status-b2g-v2.2: --- → affected
Flags: needinfo?(jmitchell)
Keywords: qawanted → regression
blocking-b2g: --- → 2.2?
QA Whiteboard: [QAnalyst-Triage?]
Flags: needinfo?(jmitchell)
Keywords: regressionwindow-wanted
QA Contact: ckreinbring
Regression window
Last working
BuildID: 20141021064010
Gaia: 457a54fc3200b80e4f5e1cd3acaa062309230732
Gecko: 676acfb44e8b
Platform Version: 36.0a1
Firmware Version: V188
User Agent: Mozilla/5.0 (Mobile; rv:36.0) Gecko/36.0 Firefox/36.0

First broken
BuildID: 20141021064810
Gaia: 3ec94f448bb5c1c9c264896685c6ef77ab718c87
Gecko: fe1513fc09f6
Platform Version: 36.0a1
Firmware Version: V188
User Agent: Mozilla/5.0 (Mobile; rv:36.0) Gecko/36.0 Firefox/36.0

Working Gaia / Broken Gecko = No repro
Gaia: 457a54fc3200b80e4f5e1cd3acaa062309230732
Gecko: fe1513fc09f6
Broken Gaia / Working Gecko = Repro
Gaia: 3ec94f448bb5c1c9c264896685c6ef77ab718c87
Gecko: 676acfb44e8b
Gaia pushlog: https://github.com/mozilla-b2g/gaia/compare/457a54fc3200b80e4f5e1cd3acaa062309230732...3ec94f448bb5c1c9c264896685c6ef77ab718c87


B2G Inbound
Last working
BuildID: 20141020205719
Gaia: e09e1734ad523cf63351a28f6f84454319349fbe
Gecko: 4da1f6a151d6
Platform Version: 36.0a1
Firmware Version: V188
User Agent: Mozilla/5.0 (Mobile; rv:36.0) Gecko/36.0 Firefox/36.0

First broken
BuildID: 20141020214218
Gaia: ba10744d64411a8a12ae68f7cf1ec3e3ac897d21
Gecko: bcc5df613d83
Platform Version: 36.0a1
Firmware Version: V188
User Agent: Mozilla/5.0 (Mobile; rv:36.0) Gecko/36.0 Firefox/36.0

Working Gaia / Broken Gecko = No repro
Gaia: e09e1734ad523cf63351a28f6f84454319349fbe
Gecko: bcc5df613d83
Broken Gaia / Working Gecko = Repro
Gaia: ba10744d64411a8a12ae68f7cf1ec3e3ac897d21
Gecko: 4da1f6a151d6
Gaia pushlog: https://github.com/mozilla-b2g/gaia/compare/e09e1734ad523cf63351a28f6f84454319349fbe...ba10744d64411a8a12ae68f7cf1ec3e3ac897d21
QA Whiteboard: [QAnalyst-Triage?]
Flags: needinfo?(jmitchell)
Keywords: regressionwindow-wanted
Broken by Bug 1007600 - can you take a look Arthur
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(jmitchell) → needinfo?(arthur.chen)
QA Contact: ckreinbring
Alive, it seems there are issues when calculating the theme color in app chrome. Any ideas?
Flags: needinfo?(arthur.chen) → needinfo?(alive)
(In reply to Arthur Chen [:arthurcc] from comment #6)
> Alive, it seems there are issues when calculating the theme color in app
> chrome. Any ideas?

You should specify your theme color in activity page just like in app page.
https://github.com/mozilla-b2g/gaia/blob/master/apps/settings/index.html#L6

One line fix!
Flags: needinfo?(alive)
Assignee: nobody → b.mcb
Created attachment 8514912 [details] [review]
First patch

Hey Arthur, as Alive said in the previous comment, we need to change the meta tag of the inline activity. I've noticed that the inline activity doesn't have its own file so, we must update the theme once we receive the activity request.

I don't like the approach I've made, could you give your opinion on that?

Thanks!
Attachment #8514912 - Flags: review?(arthur.chen)
Settings app uses the same page for inline activity. Per the offline discussion with Alive, there are issues related to app chrome. The calculated theme changes during the transition of inline activity. Flagging Michael for inputs.

(In reply to Alive Kuo [:alive][NEEDINFO!] from comment #7)
> (In reply to Arthur Chen [:arthurcc] from comment #6)
> > Alive, it seems there are issues when calculating the theme color in app
> > chrome. Any ideas?
> 
> You should specify your theme color in activity page just like in app page.
> https://github.com/mozilla-b2g/gaia/blob/master/apps/settings/index.html#L6
> 
> One line fix!
Flags: needinfo?(mhenretty)
Comment on attachment 8514912 [details] [review]
First patch

Per comment 9, this is not a proper fix. There should be no changes required in settings app.
Attachment #8514912 - Flags: review?(arthur.chen)
Alberto, can you try and figure out what is happening with theme color changes during inline activity launches?
Flags: needinfo?(mhenretty) → needinfo?(apastor)
(Assignee)

Comment 12

4 years ago
I'm surprised this didn't get caught by https://github.com/mozilla-b2g/gaia/blob/master/apps/system/test/marionette/statusbar_icon_color_test.js#L86

I'll take a look.
Flags: needinfo?(apastor)
(Assignee)

Comment 13

4 years ago
Stealing, as is a System problem
Assignee: b.mcb → apastor
(Assignee)

Comment 14

4 years ago
Created attachment 8515902 [details] [review]
Link to Pull Request: https://github.com/mozilla-b2g/gaia/pull/25760
Attachment #8515902 - Flags: review?(mhenretty)
Comment on attachment 8515902 [details] [review]
Link to Pull Request: https://github.com/mozilla-b2g/gaia/pull/25760

LGTM, thanks! I left a comment on github not really related to your patch, but just a thought.
Attachment #8515902 - Flags: review?(mhenretty) → review+
(Assignee)

Comment 16

4 years ago
master: https://github.com/mozilla-b2g/gaia/commit/6b11299636900190ae47d28a13a418fc6b91cb64
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
blocking-b2g: 2.2? → ---
status-b2g-v2.2: affected → fixed
Whiteboard: [systemsfe]
Target Milestone: --- → 2.1 S8 (7Nov)
This issue is verified fixed on Flame 2.2.

Result: The status bar icons are displayed in gray on "Messaging Settings" screen from Messages app.

Device: Flame 2.2 (319mb, KK, Shallow Flash)
BuildID: 20141119040205
Gaia: e64428c5b2dce5db90b75a5055077a04f4bd4819
Gecko: bc2c36dda0a9
Version: 36.0a1 (2.2 Master)
Firmware: V188-1
User Agent: Mozilla/5.0 (Mobile; rv:36.0) Gecko/36.0 Firefox/36.0
Status: RESOLVED → VERIFIED
QA Whiteboard: [QAnalyst-Triage+] → [QAnalyst-Triage?]
status-b2g-v2.2: fixed → verified
Flags: needinfo?(ktucker)
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(ktucker)
You need to log in before you can comment on or make changes to this bug.