Closed Bug 1696638 Opened 4 years ago Closed 4 years ago

Password Quality meter bar overlaps with the text in the Primary Password setup modal

Categories

(Firefox :: Settings UI, defect)

Desktop
All
defect

Tracking

()

VERIFIED FIXED
88 Branch
Tracking Status
firefox-esr78 --- unaffected
firefox86 --- unaffected
firefox87 --- unaffected
firefox88 --- verified

People

(Reporter: tbabos, Assigned: ntim)

References

(Regression)

Details

(Keywords: regression)

Attachments

(2 files)

Attached image Screenshot of the issue

Affected Versions:
Nightly 88.0a1 (2021-03-05)

Tested on:
Windows 7/10
MacOS 10.15
Ubuntu 20.04

Steps to reproduce:

  1. Got to about:prefereces#privacy
  2. Choose to use a Primary Password and enter the OS auth password
  3. Reach the Primary Password setup modal

Expected Results:
The password quality meter should be aligned correctly without being overlapped or overlapping other modal elements.

Actual Results:
Password Quality meter bar overlaps with text in Primary Password setup modal

Regression-Range:
Regression caused by Bug 1590884.
2021-03-05T14:12:01.458000: DEBUG : Found commit message:
Bug 1590884 - Move XUL accesskey handling to DOM and remove nsXULLabelFrame. r=emilio

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

Hey Tim, please take a look at this when you have the time, thanks!

Flags: needinfo?(ntim.bugs)
Has Regression Range: --- → yes
Has STR: --- → yes
Assignee: nobody → ntim.bugs
Flags: needinfo?(ntim.bugs)

This was originally added in bug 1120967 to space the help button that used to appear in the header. The help button has since been removed.

It's bogus in this case, because it accidentally overrides the display value of some description (which used to force their own display value).

Attachment #9207149 - Attachment description: Bug 1696638 - Remove bogus .header preferences.css styling. → Bug 1696638 - Remove obsolete .header preferences.css & common.css styling.
Pushed by ntim.bugs@gmail.com: https://hg.mozilla.org/integration/autoland/rev/ff5c36b7a05f Remove obsolete .header preferences.css & common.css styling. r=preferences-reviewers,Gijs
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 88 Branch

Verified-fixed on Windows 7/10, MacOS 10.15 and Ubuntu 20.04 on the latest Firefox Nightly 88.0a1 (2021-03-19) (64-bit).

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

Attachment

General

Created:
Updated:
Size: