919492 - Implement Left/Right gestures in the system application to switch between apps

Status: RESOLVED FIXED

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

Description

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

Basic left/right gesture support to swipe from app to app. In order to implement this it would be good to wait for the new window manager to not do this work twice.

Comment 1

[Etienne Segonzac (:etienne)]

Attachment: UX Spec

Comment 2

[Etienne Segonzac (:etienne)]

Attachment: WIP

Comment 3

[Etienne Segonzac (:etienne)]

Attachment: Pointer to gaia PR

Let the fun begin :)

Comment 4

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

Comment on attachment 827734 [details] [review]
Pointer to gaia PR

Almost is good as an initial draft of swipe gesture implementation but forgive my picky. I'd like (App)StackManager to have it's own list of appWindow instances to avoid coupling with (App)WindowManager too deeply.

Comment 5

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

Also I prefer to move SheetTransition into AppWindow,
that means we could just call App.moveTo(x)/App.FinishTransition()/..... in EdgeGestureDetector or AppStackManager/AppSheetManager ...whatever.

But it's fine that we do this later after bug 907013 is landed.

Comment 6

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

One more thing about screenElement overloading:
Currently it's overloading by lots of classes and some of them may be not necessary.
For example if I move the modal dialog into the appWindow we don't need screen.modal-dialog anymore.
The dom element structure is what I want to speak here: Just let parent(the one control it) or itself do the style controlling.

For example,

A > B > C > D > E {
}

If we want to control E's style, do it in D or E itself by appending classes in D instead of going through up to A.

With this we don't need to write dirty rules like A.aaaa.bbbbb:not(.ccccc):not(.ddddddd).eeee.ffff.gggg { .... }

In your use case I think we could just control the style of edge line directly but in the case about transitioning it's O.K. to do that after bug 907013. So this comment is just added to track this issue addressed in github comment already.

Comment 7

[Etienne Segonzac (:etienne)]

Comment on attachment 827734 [details] [review]
Pointer to gaia PR

Updated!
All the comments should be addressed :)

I'm still listening to launchapp/launchwrapper because |appopen| also gets triggered when you swipe from one app to the other.

But we're now storing app objects in the StackManager which is indeed way cleaner!

Comment 8

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

Comment on attachment 827734 [details] [review]
Pointer to gaia PR

r+ except for the stayBackground.
BTW travis is unhappy https://travis-ci.org/mozilla-b2g/gaia/jobs/13605336

Comment 9

[Etienne Segonzac (:etienne)]

https://github.com/mozilla-b2g/gaia/commit/55cf586b78f1f2d88f376260b3e2a167b3da920b

Thank you *so* much Alive and Vivien, this was a wild ride!

Comment 10

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

I could figure out lots of followup now :)