Closed Bug 1377182 Opened 3 years ago Closed 3 years ago

Session Restore button in tabstrip has janky animation

Categories

(Firefox :: Session Restore, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 56
Tracking Status
firefox56 --- fixed

People

(Reporter: ewright, Assigned: ewright)

References

Details

Attachments

(1 file)

After turning on the pref in my own Nightly, I noticed that the animation is very jittery. As well, many times when clicked, the button animates small but does not fully disappear.
current patch changes: 
Button animates at same speed as new tabs open to appear smoother (100ms).
Padding removed from inner label so it smoothly goes to width of 0.
Setting the max-width of the sessionRestoreButton when it is first created, and removing the 'requestAnimationFrame', otherwise sometimes the listener will not catch the animation in time, resulting in not fully removing the button.
Comment on attachment 8882271 [details]
Bug 1377182 - Fix animation for tabbar session restore button.

https://reviewboard.mozilla.org/r/153348/#review158770
Attachment #8882271 - Flags: review?(gijskruitbosch+bugs) → review+
Keywords: checkin-needed
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/993e3817d2fe
Fix animation for tabbar session restore button. r=Gijs
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/993e3817d2fe
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 56
I have reproduced this bug with Nightly 56.0a1 (2017-06-29) on Windows 8.1 , 64 Bit ! 

This bug's fix is Verified with latest Beta 56.0b12 !

Build   ID    20170914024831
User Agent    Mozilla/5.0 (Windows NT 6.3; WOW64; rv:56.0) Gecko/20100101 Firefox/56.0

[testday-20170915]
You need to log in before you can comment on or make changes to this bug.