942339 - Modify SMS App Scrollable Area to stay in 1 active layer

Status: RESOLVED FIXED

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

Description

[Mason Chang [Inactive] [:mchang]]

Modify SMS app so that the scrollable area has the will-animate property.

Comment 1

[Julien Wajsberg [:julienw]]

How do we do that?

Comment 2

[Benoit Girard (:BenWa)]

Once bug 940842 is ready we just need to add the new CSS property.

Comment 3

[Mason Chang [Inactive] [:mchang]]

Attachment: smsWillScroll.patch

Patch attached to enable will-animate. Tested on reference-workload-light with the SMS app and clicking into a single message. Nice that both scrollable areas are in the same div.

Comment 4

[Julien Wajsberg [:julienw]]

Ok, great idea :) I never liked the idea of abusing 3D translate transformations to do this.

Will this be prefixed, or hidden behind a pref, or something?

Comment 5

[Benoit Girard (:BenWa)]

I imagine for now it will be restricted to b2g app until it's accepted.

Comment 6

[Mason Chang [Inactive] [:mchang]]

Dropbox video with patch applied:

https://www.dropbox.com/s/221ih9d0pbcwlq3/Video%20Dec%2002%2C%2010%2017%2038%20AM.mov

Comment 7

[Benoit Girard (:BenWa)]

Comment on attachment 8337916 [details] [diff] [review]
smsWillScroll.patch

Review of attachment 8337916 [details] [diff] [review]:
-----------------------------------------------------------------

r=me because this is a platform hint. The only behavioral change is forcing a stacking context.

Comment 8

[Mason Chang [Inactive] [:mchang]]

https://bugzilla.mozilla.org/show_bug.cgi?id=942339

Comment 9

[Mason Chang [Inactive] [:mchang]]

https://github.com/mozilla-b2g/gaia/pull/14274

Comment 10

[Julien Wajsberg [:julienw]]

master: 9898d5c527e0f74855b7431d50d0148c6e0a6faa