Open Bug 1707983 Opened 6 months ago Updated 5 months ago

Pinch-zooming doesn't work in tabs that have been dragged into a different window

Categories

(Core :: Panning and Zooming, defect, P2)

All
Unspecified
defect

Tracking

()

ASSIGNED

People

(Reporter: mstange, Assigned: tnikkel)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Steps to reproduce:

  1. Pinch-zoom a page and verify that it works.
  2. Drag the tab into a different window (could be an existing window, or a new window by detaching the tab).
  3. Pinch-zoom the page again, in the different window.

Expected results:
Pinch-zooming should work normally.

Actual results:
Nothing happens in response to the pinch-zoom gesture.

If the tab is dragged back into the original window, pinch-zooming works again.

The zoom constraints do not allow zoom, which means they probably haven't gotten an update from content at all and are in their initial values. Not a recent regression, 2020-11-01 has this bug.

Blocks: desktop-zoom

Fix is probably just hooking up the zoom constraints client to a notification to refresh zoom constraints when we do a swapDocShells call, but what notification that is, not sure.

I guess we need to do this every time we get a new APZC tree manager (because that is where the zoom constraints map/cache is stored). I'm not sure how to detect that or if there are other cases we haven't covered.

Assignee: nobody → tnikkel
Status: NEW → ASSIGNED

There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:tnikkel, could you have a look please?
For more information, please visit auto_nag documentation.

Flags: needinfo?(tnikkel)
Flags: needinfo?(botond)
Flags: needinfo?(botond)

A test needs to be written.

Flags: needinfo?(tnikkel)
Severity: -- → S3
Priority: -- → P2
You need to log in before you can comment on or make changes to this bug.