[Ambient indicator] should follow the animation on edge gestures

RESOLVED FIXED in 2.2 S1 (5dec)

Status

Firefox OS
Gaia::System
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: albertopq, Assigned: albertopq)

Tracking

unspecified
2.2 S1 (5dec)

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [systemsfe])

Attachments

(1 attachment)

(Assignee)

Description

3 years ago
STR

1.- Create a notification
2.- Open an app
3.- Perform an edge gesture

Expected:

The ambient indicator moves with the app screenshot

Actual

The ambient indicator remains static
(Assignee)

Updated

3 years ago
Assignee: nobody → apastor
(Assignee)

Comment 1

3 years ago
Created attachment 8525180 [details] [review]
Link to Pull Request: https://github.com/mozilla-b2g/gaia/pull/26282
Attachment #8525180 - Flags: review?(mhenretty)
Comment on attachment 8525180 [details] [review]
Link to Pull Request: https://github.com/mozilla-b2g/gaia/pull/26282

Left some comments on github. Normally I'd just r+ this, but I think we need to fix the tests first, which could require a different approach.

In the meantime, let's get a ui-review from Eric.
Attachment #8525180 - Flags: review?(mhenretty) → ui-review?(epang)
Comment on attachment 8525180 [details] [review]
Link to Pull Request: https://github.com/mozilla-b2g/gaia/pull/26282

Good catch, looks good to me r+.  Thanks for working on this Alberto!
Attachment #8525180 - Flags: ui-review?(epang) → ui-review+
Target Milestone: 2.1 S9 (21Nov) → 2.2 S1 (5dec)
(Assignee)

Comment 4

3 years ago
Comment on attachment 8525180 [details] [review]
Link to Pull Request: https://github.com/mozilla-b2g/gaia/pull/26282

Hi Michael, I fixed the performance tests. The problem wasn't about using top or translateY, but that I forgot that the ambient indicator is now relative to the utility tray, so using top on relative positioned elements was causing the reflows. I'm hiding the utility tray (which is absolute) now, so should be fine.

Could you please review again?
Attachment #8525180 - Flags: review?(mhenretty)
Comment on attachment 8525180 [details] [review]
Link to Pull Request: https://github.com/mozilla-b2g/gaia/pull/26282

Looks good to me, and works like a charm!
Attachment #8525180 - Flags: review?(mhenretty) → review+
(Assignee)

Comment 6

3 years ago
master: https://github.com/mozilla-b2g/gaia/commit/7e43509eeb36baa1786d15fd829ec302786ae18a
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.