Closed Bug 1977669 Opened 3 months ago Closed 3 months ago

Firefox Profiler UI timeline labels no longer centered vertically

Categories

(Core :: Layout, defect)

defect

Tracking

()

VERIFIED FIXED
142 Branch
Tracking Status
firefox-esr128 --- unaffected
firefox-esr140 --- unaffected
firefox140 --- unaffected
firefox141 --- unaffected
firefox142 --- verified
firefox143 --- verified

People

(Reporter: mstange, Assigned: emilio)

References

(Regression)

Details

(Keywords: regression)

Attachments

(3 files)

Attached image screenshot

Example: https://share.firefox.dev/4kGW2c0

The labels in the <button> at the left of the timeline were vertically centered in the past, but aren't anymore. Removing visibility:hidden makes the labels centered again.
In Chrome they are centered even with visibility:hidden.

Flags: needinfo?(emilio)

Removing visibility:hidden makes the labels centered again.

wat.

Assignee: nobody → emilio

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

Oh, overflow: hidden, not visibility: hidden :)

Avoid the scroll container special-case for inline-blocks for them, and
allow a ::-moz-scrolled-content button box to still center-align its
contents.

Flags: needinfo?(emilio)

(In reply to Emilio Cobos Álvarez (:emilio) from comment #3)

Oh, overflow: hidden, not visibility: hidden :)

Hah, oops! I read it so many times and never noticed I had written the wrong property.

Blocks: 1977839

FWIW I confirmed locally that this patch fixes a version of the same bug on Amazon.com when shopping for Whole Foods Market grocery-pickup (see screenshot). They have some dropdown-menu-widgets which they represent (when closed at least) as a <button> with overflow:hidden, baseline-aligned with an i element with a downarrow-graphic as its background-image.

In Firefox Nightly and in a build from current git tip: the button label text is misaligned as shown in this screenshot. But I confirmed that it's fixed in a build with this patch attached.

Attachment #9501389 - Attachment description: image.png → screenshot of similar bug on Amazon/Whole Foods UI
Pushed by ealvarez@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/7e5b2a9fd233 https://hg.mozilla.org/integration/autoland/rev/5e36dc444954 Scrolled button frames should still be centered and baseline aligned to the content box. r=dholbert
Pushed by sstanca@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/c09786b7a55b https://hg.mozilla.org/integration/autoland/rev/fdcb70fb304a Revert "Bug 1977669 - Scrolled button frames should still be centered and baseline aligned to the content box. r=dholbert" for causing wpt failures in anonymous-button-content-box.html.

Reverted this because it was causing wpt failures in anonymous-button-content-box.html.

Flags: needinfo?(emilio)
Flags: needinfo?(emilio)
Pushed by ealvarez@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/24ea549f78df https://hg.mozilla.org/integration/autoland/rev/119fc6ab76b6 Scrolled button frames should still be centered and baseline aligned to the content box. r=dholbert
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/53830 for changes under testing/web-platform/tests
Status: NEW → RESOLVED
Closed: 3 months ago
Resolution: --- → FIXED
Target Milestone: --- → 142 Branch
Upstream PR merged by moz-wptsync-bot
Duplicate of this bug: 1978707
QA Whiteboard: [qa-triage-done-c143/b142][qa-ver-opt-c143/b142]
QA Contact: oardelean

Issue is reproducible on a 2025-07-16 Firefox Nightly build on Windows 10, with the example and steps from Comment 0.
Verified as fixed on Firefox Nightly 143.0a1 and Firefox 142.0b5 on Windows 10, Ubuntu 22, macOS 13.

Status: RESOLVED → VERIFIED
QA Whiteboard: [qa-triage-done-c143/b142][qa-ver-opt-c143/b142] → [qa-triage-done-c143/b142][qa-ver-done-c143/b142]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: