If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Modify SMS App Scrollable Area to stay in 1 active layer

RESOLVED FIXED in 1.2 C6(Dec6)

Status

Firefox OS
Gaia::SMS
P1
normal
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: mchang, Assigned: mchang)

Tracking

({perf})

unspecified
1.2 C6(Dec6)
ARM
Gonk (Firefox OS)
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [c=handeye p=3 s= u=])

Attachments

(1 attachment)

(Assignee)

Description

4 years ago
Modify SMS app so that the scrollable area has the will-animate property.
(Assignee)

Updated

4 years ago
Blocks: 942333
How do we do that?
Once bug 940842 is ready we just need to add the new CSS property.
(Assignee)

Comment 3

4 years ago
Created attachment 8337916 [details] [diff] [review]
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.
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?
I imagine for now it will be restricted to b2g app until it's accepted.
(Assignee)

Comment 6

4 years ago
Dropbox video with patch applied:

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

Updated

4 years ago
Attachment #8337916 - Flags: review?(bgirard)
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.
Attachment #8337916 - Flags: review?(bgirard) → review+
(Assignee)

Comment 8

4 years ago
https://bugzilla.mozilla.org/show_bug.cgi?id=942339
Keywords: checkin-needed
(Assignee)

Comment 9

4 years ago
https://github.com/mozilla-b2g/gaia/pull/14274
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
(Assignee)

Updated

4 years ago
Keywords: checkin-needed
master: 9898d5c527e0f74855b7431d50d0148c6e0a6faa

Updated

4 years ago
Depends on: 961871
You need to log in before you can comment on or make changes to this bug.