Advanced font in-content dialog is cut-off at the bottom
Categories
(Toolkit :: Preferences, defect, P2)
Tracking
()
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)
(deleted),
image/png
|
Details | |
(deleted),
text/x-phabricator-request
|
pascalc
:
approval-mozilla-beta+
|
Details |
Steps to reproduce:
- about:preferences > General
- 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
Reporter | ||
Comment 1•4 years ago
|
||
Regression window:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=8aa1b80a8c6b66cccc2e37080fbf6197eee8d801&tochange=1716229005d8b97f305bd663b8bc8d3fec4a4f3a
Comment 2•4 years ago
|
||
I can reproduce the bug in Windows 10 (with a smaller cut-off) but not on Linux and macOS where the buttons look normal.
Updated•4 years ago
|
Updated•4 years ago
|
Updated•4 years ago
|
Updated•4 years ago
|
Updated•4 years ago
|
Assignee | ||
Comment 3•4 years ago
|
||
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.
Updated•4 years ago
|
Comment 5•4 years ago
|
||
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
Comment 7•4 years ago
|
||
bugherder |
Assignee | ||
Comment 8•4 years ago
|
||
Alice, could you confirm if this is fixed for you in today's nightlies?
Reporter | ||
Comment 9•4 years ago
|
||
(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.
Assignee | ||
Comment 10•4 years ago
|
||
(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!
Assignee | ||
Comment 11•4 years ago
|
||
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
Assignee | ||
Updated•4 years ago
|
Comment 12•4 years ago
|
||
Comment on attachment 9219071 [details]
Bug 1704113 - fix fonts subdialog cut-off in the preferences, r?jaws
Approved for 89 beta 7, thanks.
Comment 13•4 years ago
|
||
bugherder uplift |
Description
•