942704 - [Status bar] Followup of bug 936201, considering activity.

Status: VERIFIED FIXED

(Firefox OS Graveyard :: Gaia::System, defect, P1)

Description

[(inactive after 6/18) Alive Kuo [:alive]]

STR:
1. In homescreen
2. Longpress
3. Choose wallpaper activity
4. Wait it is opened.

Expected:
Statusbar is not semitransparent.

Actual:
Statusbar is semitransparent.

Note:
One line patch: Consider activityopening event in statusbar as well as appopening.

+++ This bug was initially created as a clone of Bug #936201 +++

Spoke to Peter and decided to go even darker for the status bar headers.
#20282e

Comment 1

[[e-mail]]

steal it

Comment 2

[[e-mail]]

Attachment: pull request redirect to github

Comment 3

[(inactive after 6/18) Alive Kuo [:alive]]

Comment on attachment 8339704 [details] [review]
pull request redirect to github

Github

Comment 4

[[e-mail]]

merged to gaia-master https://github.com/mozilla-b2g/gaia/commit/ee1bfd4a260e732264f8486cbf62197760de984a

thanks!

Comment 5

[Sam Joch [:samjoch]]

Attachment: Need to REOPENED this bug.The problem is not resolved see attachment.

Comment 6

[Sam Joch [:samjoch]]

I don't think the problem come from the 'activity'.
On the homescreen this 'contextmenu' is managed by contextmenu.js 
which is a fake contextmenu controlled by the Homescreen app not the System app.

Comment 7

[[e-mail]]

Sam, so you think the problem is 'the statusbar is transparent while contextmenu is triggered', right?

Any app's fake context menu can't control the statusbar transparency though...

Comment 8

[Sam Joch [:samjoch]]

Also https://bugzilla.mozilla.org/show_bug.cgi?id=936201 need to be backout as this one involved conflicted i backed out this one too.

revert commit: d41da2a45d55f53b621e03d808d00fec67ef9f3b

Comment 9

[Sam Joch [:samjoch]]

Well if the app handle its own fake contextmenu, yes it won't control the statusbar.

Comment 10

[Jason Smith [:jsmith]]

With the backout in bug 936201, this issue is no longer present in the build, so I'm closing this out. We can take care of this in an updated patch on bug 936201.

Comment 11

[Peter Bylenga [:PBylenga]]

Verified that the status bar is not semitransparent.

Environmental Variables
Device: Buri v1.3 Mozilla RIL
Build ID: 20131205040201
Gecko: http://hg.mozilla.org/mozilla-central/rev/725c36b5de1a
Gaia: 1dd0e5c644b4c677a4e8fa02e50d52136db489d9
Platform Version: 28.0a1
Firmware Version: 20131115