Skip to content

Commit

Permalink
Trigger pointer leave when active controller switched (#36662)
Browse files Browse the repository at this point in the history
Summary:
Pull Request resolved: #36662

For VR, JSPointerDispatcher skips calculating the delta of hit targets (which happens in onMove) on the frame in which the active controller is switched because the motion event is a DOWN event, not a MOVE event in that specific frame.

This diff fixes the issue by just calling onMove in addition to onDown for DOWN motion events. Unfortunately we do not have separate pointer IDs for each controller.

This won't change the behavior for non-hoverable pointers. For hoverable pointers, it will dispatch an extra pointer_enter if the hit path has changed between the last move event and the down event.

Changelog:
[Internal][Fixed] - Trigger pointer leave when active controller switched

Reviewed By: lunaleaps, javache

Differential Revision: D44377324

fbshipit-source-id: 001ba36cce7b545643b700e0ad89075e05288619
  • Loading branch information
genkikondo authored and facebook-github-bot committed Mar 27, 2023
1 parent 406f9fc commit d6cc198
Showing 1 changed file with 4 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -158,6 +158,10 @@ private void onDown(
motionEvent,
enterViewTargets,
eventDispatcher);
} else {
// There are cases when the pointer may have moved in the same frame as the down event.
// Dispatch the move event before the down event.
onMove(activeTargetTag, eventState, motionEvent, eventDispatcher);
}

boolean listeningForDown =
Expand Down

0 comments on commit d6cc198

Please sign in to comment.