1033926 - [Vertical Homescreen] Statusbar does not get opaque after exiting an app using the keyboard and re-entering it

Status: VERIFIED FIXED

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

Description

[Julian Schweinsberg]

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:30.0) Gecko/20100101 Firefox/30.0 (Beta/Release)
Build ID: 20140611075517

Steps to reproduce:

1. Put an app, which uses the keyboard, into the first row of the vertical homescreen.
2. Open the app, use an input so that the keyboard is shown.
3. Push the home button.
4. Open the app again.


Actual results:

The statusbar is transparent.


Expected results:

The statusbar should be opaque.

Comment 1

[Johan Lorenzo [:jlorenzo]]

Thank you for filing this bug. It appears that the build you use is a bit old (from the 11th of June). Bug 1032122 should have fixed this issue. Can you try again with a build from this month?

Thanks!

Comment 2

[Julian Schweinsberg]

Ooops, I think I have filed the bug with my desktop machine. :)

My current build is 20140703185320.  Built this afternoon.  The issue persists.

Comment 3

[Johan Lorenzo [:jlorenzo]]

Attachment: Non opaque statusbar - Video

You're right. It's also in the latest build. Good catch!

Comment 4

[Johan Lorenzo [:jlorenzo]]

We tried to avoid the non-opaque status bar for 2.0. Would this case be a blocker from a UX perspective? See video in comment 3 for details.

Comment 5

[Johan Lorenzo [:jlorenzo]]

Renaming the title to be more explicit about the edge case.

Comment 6

[Jason Smith [:jsmith]]

The video here is missing a part of the phone that needed to be seen to identify the bug, since the current video does not accurately identify the bug as written.

Comment 7

[Johan Lorenzo [:jlorenzo]]

Comment on attachment 8450509 [details]
Non opaque statusbar - Video

I don't how this video has been cropped. But here is the same hosted on Google Drive: http://mzl.la/1mp9WR7

Comment 8

[Jason Smith [:jsmith]]

This is a blocker since UX has indicated already that any instance of a transparent status bar appearing when it shouldn't be is a blocker for 2.0 visual refresh. This isn't a vertical homescreen bug though, so I'm removing the flags for that.

Comment 9

[Kevin Grandon :kgrandon]

Just CCing myself here. I will try to investigate this when I get some time - it could actually be a homescreen bug still, so I'm not *fully* ruling that out.

What could be happening is that some async message is not happening quick enough, and we end up with a delayed state.

Comment 10

[Jason Smith [:jsmith]]

(In reply to Kevin Grandon :kgrandon from comment #9)
> Just CCing myself here. I will try to investigate this when I get some time
> - it could actually be a homescreen bug still, so I'm not *fully* ruling
> that out.
> 
> What could be happening is that some async message is not happening quick
> enough, and we end up with a delayed state.

Okay, I'll tag as a VH bug for now.

Comment 11

[Cristian Rodriguez (:crdlc)]

If you follow these steps you can see how the 'homescreenclosing' [1] is not dispatched so this rule [2] is not applied

[1] https://github.com/mozilla-b2g/gaia/blob/master/apps/system/js/homescreen_launcher.js#L225
[2] https://github.com/mozilla-b2g/gaia/blob/master/apps/system/style/statusbar/statusbar.css#L36

Alive, do you know where could be the issue? Thanks a lot

Comment 12

[Cristian Rodriguez (:crdlc)]

Commented on IRC so clearing needinfo

Comment 13

[Cristian Rodriguez (:crdlc)]

Attachment: Github pull request

Thanks for your time

Comment 14

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

Comment on attachment 8450853 [details]
Github pull request

r=me thank u

Comment 15

[Cristian Rodriguez (:crdlc)]

Merged in master:

https://github.com/crdlc/gaia/commit/8dab4044b82dc3a165a2551cc521f6b47aed1958

Comment 16

[Ryan VanderMeulen [:RyanVM]]

v2.0: https://github.com/mozilla-b2g/gaia/commit/92c4cd249f2643c459bddba4c2e7a16109cf44f4

Comment 17

[Sue]

Attachment: verify_video.MP4

This issue has been verified successfully on Flame v2.1 & v2.0
See attachment: verify_video.MP4
Reproducing rate: 0/5
Flame 2.1 versions:
Gaia-Rev        5655269098c7e82254e56933f1af05b4abe2a2f3
Gecko-Rev       https://hg.mozilla.org/releases/mozilla-b2g34_v2_1/rev/86608c9389b5
Build-ID        20141204001201
Version         34.0
Device-Name     flame
FW-Release      4.4.2
FW-Incremental  eng.cltbld.20141204.034958
FW-Date         Thu Dec  4 03:50:09 EST 2014
Bootloader      L1TC00011880

Flame 2.0 versions:
Gaia-Rev        8d1e868864c8a8f1e037685f0656d1da70d08c06
Gecko-Rev       https://hg.mozilla.org/releases/mozilla-b2g32_v2_0/rev/ff1100ba2ab8
Build-ID        20141204000228
Version         32.0
Device-Name     flame
FW-Release      4.4.2
FW-Incremental  eng.cltbld.20141204.040425
FW-Date         Thu Dec  4 04:04:36 EST 2014
Bootloader      L1TC00011880