Rounding error with some text zoom values cause mispositioned select dropdowns
Categories
(Core :: Widget, defect)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr91 | --- | unaffected |
firefox-esr102 | --- | unaffected |
firefox103 | --- | wontfix |
firefox104 | --- | verified |
firefox105 | --- | verified |
People
(Reporter: emilio, Assigned: emilio)
References
(Regression, )
Details
(Keywords: regression)
Attachments
(3 files)
8.15 KB,
image/png
|
Details | |
48 bytes,
text/x-phabricator-request
|
diannaS
:
approval-mozilla-beta+
|
Details | Review |
3.12 KB,
image/png
|
Details |
This is more visible with 1.5 in the windows / GNOME text zoom settings, since it maximizes the rounding error.
Assignee | ||
Comment 1•2 years ago
|
||
Make sure to use the same scale the browser chrome would actually use by
rounding to full app units.
This is specially visible with 1.5 text scale factor.
Updated•2 years ago
|
Comment 2•2 years ago
|
||
Set release status flags based on info from the regressing bug 1773342
Comment 3•2 years ago
|
||
(In reply to Emilio Cobos Álvarez (:emilio) from comment #0)
This is more visible with 1.5 in the windows / GNOME text zoom settings, since it maximizes the rounding error.
FWIW, I can't repro anywhere as much offset as your screenshot shows, with current Nightly on Ubuntu, after setting 1.5 as the scaling-factor in the gnome-tweaks
tool's "fonts" pane. (In case it matters: I have 100% pixel-scaling, i.e. no HiDPI pixel-scaling in my screen resolution settings.)
Here's the worst that I was able to trigger (after trying various window-sizes; the popup-offset seems to vary subtly based on window size). There's a very tiny offset between the popup and the button, but not very much.
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/501e11ab6aea Avoid rounding error accumulation in nsDOMWindowUtils::ToScreenRectInCSSUnits. r=dholbert
Comment 5•2 years ago
|
||
bugherder |
Assignee | ||
Comment 6•2 years ago
|
||
Comment on attachment 9287529 [details]
Bug 1782082 - Avoid rounding error accumulation in nsDOMWindowUtils::ToScreenRectInCSSUnits. r=#layout,handyman
Beta/Release Uplift Approval Request
- User impact if declined: comment 0
- Is this code covered by automated tests?: Yes
- Has the fix been verified in Nightly?: No
- Needs manual test from QE?: Yes
- If yes, steps to reproduce: comment 0 is an example, on a HiDPI screen with a large text scale factor.
- List of other uplifts needed: none
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): Simple behavior-preserving patch that just improves the precision of the returned values.
- String changes made/needed: none
- Is Android affected?: No
Assignee | ||
Updated•2 years ago
|
Comment 7•2 years ago
|
||
Comment on attachment 9287529 [details]
Bug 1782082 - Avoid rounding error accumulation in nsDOMWindowUtils::ToScreenRectInCSSUnits. r=#layout,handyman
Approved for 104.0b5
Updated•2 years ago
|
Comment 8•2 years ago
|
||
bugherder uplift |
Comment 9•2 years ago
|
||
I've reproduced this bug using an affected Beta build, 104.0b4, with the STR from comment 0.
The issue is verified as fixed on latest Nightly 105.0a1, under Win 10 x64.
Comment 10•2 years ago
|
||
This is also verified as fixed on Beta 104.0b5 with Win 10 x64.
Updated•2 years ago
|
Description
•