Closed Bug 1579003 Opened Last month Closed Last month

Implement megabar growing and shrinking animation

Categories

(Firefox :: Address Bar, enhancement, P2)

enhancement
Points:
3

Tracking

()

RESOLVED FIXED
Firefox 71
Iteration:
71.1 - Sept 2 - 15
Tracking Status
firefox71 --- fixed

People

(Reporter: dao, Assigned: dao)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

verdi says we should use the photon animation curve and 250ms duration.

Assignee: nobody → dao+bmo
Status: NEW → ASSIGNED
Iteration: --- → 71.1 - Sept 2 - 15
Depends on: 1577472
Pushed by dgottwald@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/48e712ae6367
Implement megabar growing and shrinking animation. r=harry

FWIW, left, width and padding transitions are not be able to run on the compositor, which means it consumes the main-thread to update styles on every tick. Also it causes reflow.

(In reply to Hiroyuki Ikezoe (:hiro) from comment #3)

FWIW, left, width and padding transitions are not be able to run on the compositor, which means it consumes the main-thread to update styles on every tick. Also it causes reflow.

It's a positioned element so the reflow should be limited to that? Happy to look into other ways to implement this animation though, if you have suggestions.

Status: ASSIGNED → RESOLVED
Closed: Last month
Resolution: --- → FIXED
Target Milestone: --- → Firefox 71

(In reply to Dão Gottwald [::dao] from comment #4)

(In reply to Hiroyuki Ikezoe (:hiro) from comment #3)

FWIW, left, width and padding transitions are not be able to run on the compositor, which means it consumes the main-thread to update styles on every tick. Also it causes reflow.

It's a positioned element so the reflow should be limited to that? Happy to look into other ways to implement this animation though, if you have suggestions.

If we don't expect siblings of the transitioning element is not moved by the transition, left and padding transition can be replaced by transform. I have no idea about the width transition. Unfortunately transform transition with geometric change property transition, e.g. width, left, etc. can't be run on the compositor.

Regressions: 1586204
You need to log in before you can comment on or make changes to this bug.