Closed Bug 1821548 Opened 2 years ago Closed 2 years ago

Stop/reload icon wiggles when you it switches between animated & static modes

Categories

(Core :: XUL, defect)

defect

Tracking

()

VERIFIED FIXED
112 Branch
Tracking Status
firefox-esr102 --- unaffected
firefox110 --- unaffected
firefox111 --- unaffected
firefox112 --- disabled
firefox113 --- verified

People

(Reporter: dholbert, Assigned: emilio)

References

(Regression)

Details

(Keywords: regression)

Attachments

(2 files, 1 obsolete file)

Attached video screencast (deleted) —

STR:

  1. Load (or reload) https://yahoo.com , or probably any non-trivially-sized site.
  2. Watch your stop/reload icon in your toolbar.

ACTUAL RESULTS:
The icon shifts rightwards a bit while it's animating, and then shifts left again. See attached screencast.

EXPECTED RESULTS:
No such shift.

Regression range:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=53491bc68f81d4c082d3d6cfdda617fe4deca8b0&tochange=c25af897c9bcb33045fe7664d93684de65cbf2a8

Attachment #9322260 - Attachment description: screemcast → screecast
Attachment #9322260 - Attachment description: screecast → screencast
Flags: needinfo?(emilio)

This behavior change I didn't see coming, I admit. This rule:

https://searchfox.org/mozilla-central/rev/2af0d81d29bcd264083e04a2473d82a91f0f7eb6/browser/themes/shared/toolbarbuttons.css#73

Used to be -moz-box-pack: center instead.

Which didn't have any effect on abspos behavior. Now it uses
justify-content: center, which (per spec) applies to abspos boxes as
well, and gives us the centering in the inline axis.

So that means we don't really need to apply the centering in the inline
axis manually, and that margin was only shifting our box to the right.

Assignee: nobody → emilio
Status: NEW → ASSIGNED

Set release status flags based on info from the regressing bug 1820534

Flags: needinfo?(emilio)

I don't see any visible issue on the reload / stop buttons without this.

Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/105612cbf8b0 Remove no longer needed margin / manual centering in toolbarbutton-animatable-box. r=Gijs,dao

Comment on attachment 9322346 [details]
Bug 1821548 - Remove BrowserUIUtils.setToolbarButtonHeightProperty and related code. r=Gijs

Revision D172238 was moved to bug 1821657. Setting attachment 9322346 [details] to obsolete.

Attachment #9322346 - Attachment is obsolete: true
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 112 Branch

This was backed out of beta v112 via bug 1820534

Flags: qe-verify+

Reproduced using an old Nightly build before the fix, verified that using Firefox 113.0b8 across platforms (macOS 13, Windows 10, Ubuntu 22.04) the stop/reload icon does not wiggles anymore.

Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: