Closed Bug 1726854 Opened 4 years ago Closed 4 years ago

apz_test_native_event_utils: coordinatesRelativeToScreen does not follow device pixel density

Categories

(Core :: Panning and Zooming, defect)

Product:
Core
Component:
Panning and Zooming
Type:
defect

Tracking

()

Status:
RESOLVED DUPLICATE of bug 1726638

People

(Reporter: saschanaz, Unassigned)

Details

This is problematic when running local test on devices with high density display.

For example, when the window is maximized and the arguments are 60, 60 respectively for aX and aY, it always becomes 60, 145 when with 100% DPI and 200% DPI on Windows.

Hmm, the comment says it returns CSS coordinates, which probably means it's working as expected?

Maybe the problem is in coordinatesRelativeToScreen in apz_test_native_event_utils.js.

Component: DOM: Core & HTML → Graphics
Summary: nsDOMWindowUtils::ToScreenRect does not follow device pixel density → apz_test_native_event_utils: coordinatesRelativeToScreen does not follow device pixel density

Apologies that this hit you. It sounds like you are hitting bug 1726638, which is a recent regression. Hopefully the patches in that bug resolve the problem for you (they worked for me).

Cool, thanks! I hope the patch works when the device pixel ratio is high and the document is zoomed.

It should take into account all types of zoom: full zoom (ctrl +/-), high dpi screens, and pinch zoom.

Component: Graphics → Panning and Zooming

Looks like bug 1726638 has been fixed. Kagami, when you have a chance, please let us know if the issue is resolved for you in a recent nightly.

Flags: needinfo?(krosylight)

Confirming it now works as expected 👍

Status: NEW → RESOLVED
Closed: 4 years ago
Flags: needinfo?(krosylight)
Resolution: --- → DUPLICATE

I am sorry for that. That was my fault!

You need to log in before you can comment on or make changes to this bug.