Closed
Bug 1208669
Opened 9 years ago
Closed 6 years ago
Duplicate contact screen has a status bar color that does not match header
Categories
(Firefox OS Graveyard :: Gaia::Contacts, defect)
Tracking
(b2g-v2.2 unaffected, b2g-master affected)
RESOLVED
WONTFIX
FxOS-S9 (16Oct)
Tracking | Status | |
---|---|---|
b2g-v2.2 | --- | unaffected |
b2g-master | --- | affected |
People
(Reporter: AdamA, Assigned: mancas)
References
Details
(Keywords: regression, Whiteboard: [NG Gaia Contacts][2.5-Daily-Testing][Spark][patch])
Attachments
(3 files)
Description: When the user is on the duplicate contact screen the status bar color is black which does not match the color of the header. Repro Steps: 1) Update a Aries to 20150925134904 2) Create a contact 3) Create another contact with the same number 4) Observe status bar Actual: the status bar color is not consistent with the header Expected: It is expected that the status bar color matches the header or the previous app Environmental Variables: Device: Aries 2.5 [Full Flash] Build ID: 20150925134904 Gaia: 5a0a1bb1a863b07528ebc1a04652ed656d8333b3 Gecko: 543e1b3a25887258a70d1f2dab3cb287d0d500e4 Gonk: 2916e2368074b5383c80bf5a0fba3fc83ba310bd Version: 44.0a1 (2.5) Firmware Version: D5803_23.1.A.1.28_NCB.ftf User Agent: Mozilla/5.0 (Mobile; rv:44.0) Gecko/44.0 Firefox/44.0 Repro frequency: 10/10 See attached: Screenshot, logcat
Reporter | ||
Comment 1•9 years ago
|
||
Reporter | ||
Comment 2•9 years ago
|
||
This issue DOES occur on Flame 2.5. Environmental Variables: Device: Flame 2.5 [Full Flash] BuildID: 20150925103004 Gaia: 5a0a1bb1a863b07528ebc1a04652ed656d8333b3 Gecko: 5abe3c4deab94270440422c850bbeaf512b1f38d Gonk: c4779d6da0f85894b1f78f0351b43f2949e8decd Version: 44.0a1 (2.5) Firmware Version: v18D User Agent: Mozilla/5.0 (Mobile; rv:44.0) Gecko/44.0 Firefox/44.0 Result: The status bar color is black ------------------------------------ This issue DOES NOT occur on Flame 2.2. Environmental Variables: Device: Flame 2.2 [Full Flash] BuildID: 20150923003003 Gaia: 5dd95cfb9f1d6501ce0e34414596ef3dd9c2f583 Gecko: 65ddad73ad6b Gonk: bd9cb3af2a0354577a6903917bc826489050b40d Version: 37.0 (2.2) Firmware Version: v18D User Agent: Mozilla/5.0 (Mobile; rv:37.0) Gecko/37.0 Firefox/37.0 Result: The status bar color matches the previous screen
Flags: needinfo?(ktucker)
Updated•9 years ago
|
Flags: needinfo?(ktucker)
Keywords: regressionwindow-wanted
Updated•9 years ago
|
Flags: needinfo?(jmercado)
QA Contact: jmercado
Updated•9 years ago
|
Flags: needinfo?(jmercado)
Comment 3•9 years ago
|
||
B2G Inbound Regression Window: Last Working Environmental Variables: Device: Flame 2.5 BuildID: 20150728071744 Gaia: ba33139c251108308cf882c152d9a7fff001bb83 Gecko: 7a4c3baef26d Gonk: 040bb1e9ac8a5b6dd756fdd696aa37a8868b5c67 Version: 42.0a1 (2.5) Firmware Version: v18D First Broken Environmental Variables: Device: Flame 2.5 BuildID: 20150728120244 Gaia: 862f0895f3f5a97200601542d99a152a46385a0b Gecko: 833403badc41 Gonk: 040bb1e9ac8a5b6dd756fdd696aa37a8868b5c67 Version: 42.0a1 (2.5) Firmware Version: v18D Last Working gaia / First Broken gecko - Issue does not occur Gaia: ba33139c251108308cf882c152d9a7fff001bb83 Gecko: 833403badc41 First Broken gaia / Last Working gecko - Issue does occur Gaia: 862f0895f3f5a97200601542d99a152a46385a0b Gecko: 7a4c3baef26d Gaia Pushlog: https://github.com/mozilla-b2g/gaia/compare/ba33139c251108308cf882c152d9a7fff001bb83...862f0895f3f5a97200601542d99a152a46385a0b This issue is caused by changes made in Bug 1182509
Blocks: 1182509
QA Whiteboard: [QAnalyst-Triage?]
Flags: needinfo?(jmercado)
Keywords: regressionwindow-wanted
Comment 4•9 years ago
|
||
Jorge this issues seems to have been caused by the changes for bug 1182509. Can you please take a look?
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(jmercado) → needinfo?(jpruden92)
Comment 5•9 years ago
|
||
Manu, can you have a look at this bug, please?
Flags: needinfo?(jpruden92) → needinfo?(b.mcb)
Whiteboard: [2.5-Daily-Testing][Spark] → [NG Gaia Contacts][2.5-Daily-Testing][Spark]
Target Milestone: --- → FxOS-S8 (02Oct)
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → b.mcb
Flags: needinfo?(b.mcb)
Updated•9 years ago
|
Status: NEW → ASSIGNED
Comment 6•9 years ago
|
||
Assignee | ||
Comment 7•9 years ago
|
||
Comment on attachment 8668420 [details] [review] [gaia] mancas:bug1208669 > mozilla-b2g:nga The main issue here is caused by the popup window created by the system when the app uses |window.open|. It is because of a css rule that set the background color of the status bar. I'm not sure if this patch will broke anything else so feel free to ping anyone who have more idea about the status bar theme.
Attachment #8668420 -
Flags: review?(francisco)
Updated•9 years ago
|
Whiteboard: [NG Gaia Contacts][2.5-Daily-Testing][Spark] → [NG Gaia Contacts][2.5-Daily-Testing][Spark][patch]
Updated•9 years ago
|
Target Milestone: FxOS-S8 (02Oct) → FxOS-S9 (16Oct)
Comment 8•9 years ago
|
||
Comment on attachment 8668420 [details] [review] [gaia] mancas:bug1208669 > mozilla-b2g:nga This must be reviewed by a system peer. Alberto are you ok with this review? Or can you redirect to the correct peer? Thanks!
Attachment #8668420 -
Flags: review?(francisco) → review?(apastor)
Comment 9•9 years ago
|
||
I'm not sure what the spec was for window.open statusbar color. Etienne, any idea?
Flags: needinfo?(etienne)
Comment 10•9 years ago
|
||
(In reply to Alberto Pastor [:albertopq] from comment #9) > I'm not sure what the spec was for window.open statusbar color. Etienne, any > idea? Don't think we have one. My best guest would be to honor the meta if there is one, or default to black. That said I don't understand the patch since, is contacts using the GlobalOverlayWindow?
Flags: needinfo?(etienne)
Comment 11•9 years ago
|
||
It just works because it makes the selector more specific. Without that, is using this selector: https://github.com/mozilla-b2g/gaia/blob/master/apps/system/style/window.css#L10 I'm not sure what the .render class is meant to do. If the fact that .render is added or not to the window is what makes the difference for using black or transparent, I think the patch is acceptable.
Flags: needinfo?(etienne)
Comment 12•9 years ago
|
||
(In reply to Alberto Pastor [:albertopq] from comment #11) > It just works because it makes the selector more specific. Without that, is > using this selector: > > https://github.com/mozilla-b2g/gaia/blob/master/apps/system/style/window. > css#L10 > > I'm not sure what the .render class is meant to do. If the fact that .render > is added or not to the window is what makes the difference for using black > or transparent, I think the patch is acceptable. Well adding specificity to a rule that is already using |!important| is probably a sign that several things are not acceptable :) (none of which are yours or Manuel's fault). So we should at least file a follow up and add a comment referencing it.
Flags: needinfo?(etienne)
Comment 13•9 years ago
|
||
Well, is the background-color the one that affects here, but I agree, we should fix it properly. I'm fine with landing this (taking into account that the selector without the '.render' class is as good/bad as this one), but only if we create a new bug for fixing it and add a comment pointing to it.
Flags: needinfo?(b.mcb)
Comment 14•9 years ago
|
||
Comment on attachment 8668420 [details] [review] [gaia] mancas:bug1208669 > mozilla-b2g:nga Clearing
Attachment #8668420 -
Flags: review?(apastor)
Assignee | ||
Updated•9 years ago
|
Flags: needinfo?(b.mcb)
Comment 15•6 years ago
|
||
Firefox OS is not being worked on
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•