Closed Bug 1704113 Opened 4 years ago Closed 4 years ago

Advanced font in-content dialog is cut-off at the bottom

Categories

(Toolkit :: Preferences, defect, P2)

Firefox 89
Desktop
Windows 10
defect

Tracking

()

VERIFIED FIXED
90 Branch
Tracking Status
firefox-esr78 --- unaffected
firefox87 --- unaffected
firefox88 --- unaffected
firefox89 + fixed
firefox90 --- verified

People

(Reporter: alice0775, Assigned: Gijs)

References

(Blocks 1 open bug)

Details

(Keywords: nightly-community, regression, Whiteboard: [proton] [priority:2c] [proton-uplift])

Attachments

(2 files)

Attached image screenshot (deleted) —

Steps to reproduce:

  1. about:preferences > General
  2. Open Advanced... in Fonts and Colors section

Actual Results:
Immediately after the dialog is displayed, the lower part of the dialog is cut-off.
See attached screenshot, OK Cancel Help button are cut-off

Expected Results:
Should not

I can reproduce the bug in Windows 10 (with a smaller cut-off) but not on Linux and macOS where the buttons look normal.

Blocks: proton
Severity: -- → S4
Whiteboard: [proton]
Priority: -- → P2
Whiteboard: [proton] → [proton] [priority:p2c]
Whiteboard: [proton] [priority:p2c] → [proton] [priority:2c]

This cut-off is happening because the font lists load dynamically. When the
font builder code is running, it removes the menupopup and there is no content.
Then we measure the height of the dialog and its many menulists and persist that.
But the heights of these menulists is smaller (depending on Windows font size
and scaling settings) when they have no content than when they have text in them,
and so when the menulists fill up the height changes, leading to the cut-off.

To fix this, set a minimum height when the menulists are empty, that either matches
or exceeds the min-height they get when they have content.

Assignee: nobody → gijskruitbosch+bugs
Status: NEW → ASSIGNED
Pushed by jwein@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/30873124f269 fix fonts subdialog cut-off in the preferences, r=jaws,preferences-reviewers

Backed out 3 changesets (Bug 1704113, Bug 1705330, Bug 1708148) for causing bc failures in browser_username_select_dialog.js and mochitest failures in test_modal_select.html.
https://hg.mozilla.org/integration/autoland/rev/3d2225b699c12f33140428b4be4bf8362c370df7

Pushed by gijskruitbosch@gmail.com: https://hg.mozilla.org/integration/autoland/rev/1bd69e1e04ce fix fonts subdialog cut-off in the preferences, r=jaws,preferences-reviewers
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 90 Branch

Alice, could you confirm if this is fixed for you in today's nightlies?

Flags: needinfo?(alice0775)

(In reply to :Gijs (he/him) from comment #8)

Alice, could you confirm if this is fixed for you in today's nightlies?

The problem is no longer reproduced on Nightly90.0a1(20210430092829) Windows10.

Flags: needinfo?(alice0775)

(In reply to Alice0775 White from comment #9)

(In reply to :Gijs (he/him) from comment #8)

Alice, could you confirm if this is fixed for you in today's nightlies?

The problem is no longer reproduced on Nightly90.0a1(20210430092829) Windows10.

Perfect, thank you!

Status: RESOLVED → VERIFIED

Comment on attachment 9219071 [details]
Bug 1704113 - fix fonts subdialog cut-off in the preferences, r?jaws

Beta/Release Uplift Approval Request

  • User impact if declined: The font dialog is hard or impossible to use (depending on text size + DPI)
  • Is this code covered by automated tests?: Yes
  • Has the fix been verified in Nightly?: Yes
  • Needs manual test from QE?: No
  • If yes, steps to reproduce:
  • List of other uplifts needed: n/a
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): We're using some CSS to ensure height calculations work more reliably.
  • String changes made/needed: Nope
Attachment #9219071 - Flags: approval-mozilla-beta?
Whiteboard: [proton] [priority:2c] → [proton] [priority:2c] [proton-uplift]

Comment on attachment 9219071 [details]
Bug 1704113 - fix fonts subdialog cut-off in the preferences, r?jaws

Approved for 89 beta 7, thanks.

Attachment #9219071 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: