Make EventStateManager's swipe-to-history detection transform-aware
Categories
(Core :: Panning and Zooming, task, P3)
Tracking
()
People
(Reporter: canalun, Unassigned)
References
Details
Follow-up to Bug 1611949.
In Bug 1611949, swipe-to-history was incorrectly triggered for pan gestures on a rotated scroller (e.g. an element with overflow-y: scroll rotated 90deg in CSS). The cause was explained in Patch D299935.
The patch fixed this on the APZ side: APZ's transform-aware scrollability decision is now plumbed through APZEventResult into MayStartSwipeForAPZ(), which leaves mCanTriggerSwipe false when APZ has determined that the target can scroll the pan.
While that works for the reported case, the content side (where EventStateManager lives) has strictly more information than APZ, so relying on APZ's decision alone is likely to miss edge cases. So, another principled fix is to make the scroll target computation in EventStateManager transform-aware, so it can find the right scroll target regardless of CSS transforms.
While the patch took the narrower approach, this bug tracks doing the more principled fix.
|
||
Updated•1 month ago
|
Description
•