1819721 - Status bar not aligned correctly to the right

Status: VERIFIED FIXED

(Firefox :: Theme, defect, P1)

Description

[Almtst]

Attachment: Now

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:109.0) Gecko/20100101 Firefox/111.0

Steps to reproduce:

If the cursor is at the bottom left of the window causing the Status bar move to the right while a page is loading, the Status bar is not aligned to the right leaving an empty space.

Mozregression returned:
Bug 1789892 - Simplify statuspanel a bit. r=dao

I'm a bit baffled about bug 1789877. My best theory so far is that we're
inserting the element a bit deeper in the DOM and that causes us to
reflow slightly more stuff when tab-switching, but...

In any case while going through the code the status panel can be
simplified a bit now, so do that.

Differential Revision: https://phabricator.services.mozilla.com/D156876

pushlog_url: https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=b84775bfccf2243f664b1c15ca987a05c54a95e2&tochange=6fbacc7bb20a65608b74e3ea09cb71af0439cb07

Actual results:

Status bar not aligned correctly

Expected results:

Status bar aligned correctly

Comment 1

[Almtst]

Attachment: Previously

Comment 2

[Ardelean Oana, Desktop Test Engineering [:oardelean]]

Hello, thank you for the bug report!
Managed to reproduce this issue on:

• Firefox 111.0;
• Nightly 112.0a1;

Tested and reproduced on:

• macOS 12;
• Windows 11;

Moving the Component to "General". Please change if there's a better fit.
Setting as NEW so the developers can have a look.
Thank you.

Comment 3

[Amy Churchwell [:amy]]

Looks like this is a regression from bug 1789892 would you be able to take a look?

Comment 4

[Emilio Cobos Álvarez (:emilio)]

Attachment: Bug 1819721 - Don't make the status panel display: block. r=dao

Its label has flex=1, but it doesn't flex since the panel is not a
flexbox.

Comment 5

[Emilio Cobos Álvarez (:emilio)]

Attachment: Bug 1819721 - Add some debugging utilities for StatusPanel debugging. r=dao

Not sure if this is worth it, your call. But I wrote this so I figured
someone debugging this code might find it useful.

Depends on D172033

Comment 6

[BugBot [:suhaib / :marco/ :calixte]]

Based on comment #0, this bug contains a bisection range found by mozregression. However, the Regressed by field is still not filled.

:emilio, if possible, could you fill the Regressed by field and investigate this regression?

For more information, please visit auto_nag documentation.

Comment 7

[BugBot [:suhaib / :marco/ :calixte]]

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

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/650612702922
Don't make the status panel display: block. r=dao

Comment 9

[Iulian Moraru]

https://hg.mozilla.org/mozilla-central/rev/650612702922

Comment 10

[Ardelean Oana, Desktop Test Engineering [:oardelean]]

Reproducible on a 2023-03-07 Nightly build on macOS 12.
Verified as fixed on Firefox 112.0b3(build ID: 20230316192655) and Nightly 113.0a1(build ID: 20230317093727) on macOS 12, Windows 10, Ubuntu 22.

Comment 11

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a18db685a6fe
Add some debugging utilities for StatusPanel debugging. r=dao

Comment 12

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/a18db685a6fe