1020755 - (vertical-homescreen) Status bar should become 100% opaque when user scrolls

Status: RESOLVED FIXED

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

Description

[Peter La]

When the user scrolls the rocketbar and icon grid up under the status bar, the status bar should become 100% opaque according to the spec.

Please see Scenario 2 from the spec:
https://mozilla.box.com/s/2wf017uxi7wmxme489kd

I will also be attaching a video showing this in action.

Comment 1

[Peter La]

Attachment: Vertical Scroll.mp4

This is the expected behaviour of statusbar on vertical scroll.  It should also transition back to being transparent if the user scrolls all the way back to the top of the screen (and the rocketbar is no longer tucked under the statusbar).  The fade duration would be the same in that case, but going from 100% opaque to 15% opaque.

Comment 2

[Cristian Rodriguez (:crdlc)]

I try to prototype it today

Comment 3

[Cristian Rodriguez (:crdlc)]

Attachment: Github pull request

The features is already implemented but before doing something regarding to unit test I would like to know your feedback about the solution implemented

Comment 4

[Kevin Grandon :kgrandon]

Comment on attachment 8434841 [details]
Github pull request

I wish there was another way. Unfortunately that we also need to do this for some dialogs and maybe smart collections I don't really see a way around the IAC message for now.

I would really love to see a marionette tests for this, but seems that might be a bit tricky to do, so a follow-up is fine.

Comment 5

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

Comment on attachment 8434841 [details]
Github pull request

I wonder why we don't use mozbrowserscroll here.

Comment 6

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

After trying, mozbrowserscroll seems not working on homescreen,
Ben do you know the story? Is mozbrowserscroll failing on apz enabled mozbrowser iframe?

Comment 7

[Kevin Grandon :kgrandon]

We can't use mozbrowserscroll because this is inside of an overflow-y div. It's like this because we also need to show the scroll bar. There may be some platform patches we can do to also show the scrollbar, but we'd still need some hook into the statusbar background (when showing dialogs, etc).

Comment 8

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

(In reply to Kevin Grandon :kgrandon from comment #7)
> We can't use mozbrowserscroll because this is inside of an overflow-y div.
> It's like this because we also need to show the scroll bar. There may be
> some platform patches we can do to also show the scrollbar, but we'd still
> need some hook into the statusbar background (when showing dialogs, etc).

Well I am concern why UX designs this...statusbar is really not part of homescreen.
For dialog it would be easy if the dialog is trigger by some mozbrowser events or mozChromeEvent and system app could jump in.

Comment 9

[Kevin Grandon :kgrandon]

I guess we don't really have the events we need today, and are trying to fill this with IAC messages. I agree it is not ideal, but I'm not sure what options we have for 2.0 =/

Comment 10

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

Comment on attachment 8434841 [details]
Github pull request

It seems I could only f+.

Comment 11

[Cristian Rodriguez (:crdlc)]

Comment on attachment 8434841 [details]
Github pull request

Ready to review and added a marionette test

Comment 12

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

Comment on attachment 8434841 [details]
Github pull request

r+ for system app part.

Comment 13

[Cristian Rodriguez (:crdlc)]

Thanks a lot Alive

(In reply to Alive Kuo [:alive][NEEDINFO!] from comment #12)
> Comment on attachment 8434841 [details]
> Github pull request
> 
> r+ for system app part.

Comment 14

[Kevin Grandon :kgrandon]

Comment on attachment 8434841 [details]
Github pull request

I can't quite give R+ on this yet, there's some problems with the transition. When I scroll down, I see that the background appears to start from the bottom of the statusbar, then expand upwards (seems like the direction could be wrong).

I'll give the code a look to see if I can figure out exactly why. I think we are close, we just need to fix that transition.

Comment 15

[Kevin Grandon :kgrandon]

Comment on attachment 8434841 [details]
Github pull request

Ok, I am gonna update one transform origin and we should be good. Thanks!

Comment 16

[Kevin Grandon :kgrandon]

Landed: https://github.com/mozilla-b2g/gaia/commit/637213e32deb42b051acf6e47755518a8baad34c

Comment 17

[Kevin Grandon :kgrandon]

Mass modify - set status-b2g-v2.0 fixed for fixed bugs under vertical homescreen dependency tree.