Closed Bug 1795224 Opened 2 years ago Closed 2 years ago

The Profile Manager is wider after the latest update to Nightly 107

Categories

(Toolkit :: Startup and Profile System, defect)

Firefox 107
Desktop
All
defect

Tracking

()

VERIFIED FIXED
108 Branch
Tracking Status
firefox-esr102 --- unaffected
firefox105 --- unaffected
firefox106 --- unaffected
firefox107 --- verified
firefox108 --- verified

People

(Reporter: sbadau, Assigned: emilio)

References

(Regression)

Details

(Keywords: regression, Whiteboard: regression)

Attachments

(6 files)

Attached image Screenshot with the issue (deleted) —

Found in

  • Nightly 107.0a1

Affected versions

  • Nightly 107.0a1

Tested platforms

  • Affected platforms: Ubuntu 20.04, macOS 11.6, and Windows 10

Steps to reproduce

  1. Install Firefox
  2. Open the Profile Manager and observe it.

Expected result

  • The profile manager is properly displayed.

Actual result

  • The window of the profile manager is wider. Please see the screenshot.

Regression range

Has STR: --- → yes

:emilio, since you are the author of the regressor, bug 1794969, could you take a look?

For more information, please visit auto_nag documentation.

Flags: needinfo?(emilio)
Attached image Profile Manager on Windows 10.png (deleted) —

Bottom is cut off on Windows 10.
Windows from Create/Rename/Delete Profile are not affected.

It's the same across platforms. The box#managebuttons selector never
matches (the element is a vbox), so remove that while at it.

Assignee: nobody → emilio
Status: NEW → ASSIGNED

No behavior change, but while at it let's do this.

Depends on D159389

Like it used to be (though resizing it didn't work before and does now).

Depends on D159390

Flags: needinfo?(emilio)
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/0c76d755cd34 Unify profile selection css. r=Gijs https://hg.mozilla.org/integration/autoland/rev/5bf2d21ead59 Use flexbox on the profile window. r=Gijs https://hg.mozilla.org/integration/autoland/rev/c437eb557374 Size profile selection dialog to min-size. r=Gijs
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 108 Branch

The patch landed in nightly and beta is affected.
:emilio, is this bug important enough to require an uplift?

  • If yes, please nominate the patch for beta approval.
  • If no, please set status-firefox107 to wontfix.

For more information, please visit auto_nag documentation.

Flags: needinfo?(emilio)
Attached patch Beta patch. (deleted) — Splinter Review

Approval Request Comment
[Feature/Bug causing the regression]: bug 1794969
[User impact if declined]: comment 0
[Is this code covered by automated tests?]: no
[Has the fix been verified in Nightly?]: N/A
[Needs manual test from QE? If yes, steps to reproduce]: comment 0
[List of other uplifts needed for the feature/fix]: none
[Is the change risky?]: no
[Why is the change risky/not risky?]: Partially backs out bug 1794969.
[String changes made/needed]: none

Flags: needinfo?(emilio)
Attachment #9299225 - Flags: approval-mozilla-beta?

Comment on attachment 9299225 [details] [diff] [review]
Beta patch.

Approved for 107.0b3.

Attachment #9299225 - Flags: approval-mozilla-beta? → approval-mozilla-beta+

Verified as fixed on the latest Nightly 108.0a1 and Firefox 107.0b3 - tested on Windows 7, Windows 10, Ubuntu 20.04, and macOS 11.7.

Status: RESOLVED → VERIFIED
Regressions: 1798836
Regressions: 1799394
No longer regressions: 1807838
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: