fix: revert use of setPointerCapture (#305)

pmndrs · Oct 12, 2023 · f58eee2 · f58eee2
1 parent 35e55ad
commit f58eee2
Showing 1 changed file with 6 additions and 8 deletions.
diff --git a/src/controls/OrbitControls.ts b/src/controls/OrbitControls.ts
@@ -414,8 +414,8 @@ class OrbitControls extends EventDispatcher {
       scope.domElement?.removeEventListener('pointerdown', onPointerDown)
       scope.domElement?.removeEventListener('pointercancel', onPointerCancel)
       scope.domElement?.removeEventListener('wheel', onMouseWheel)
-      scope.domElement?.removeEventListener('pointermove', onPointerMove)
-      scope.domElement?.removeEventListener('pointerup', onPointerUp)
+      scope.domElement?.ownerDocument.removeEventListener('pointermove', onPointerMove)
+      scope.domElement?.ownerDocument.removeEventListener('pointerup', onPointerUp)
       if (scope._domElementKeyEvents !== null) {
         scope._domElementKeyEvents.removeEventListener('keydown', onKeyDown)
       }
@@ -812,10 +812,8 @@ class OrbitControls extends EventDispatcher {
       if (scope.enabled === false) return
 
       if (pointers.length === 0) {
-        scope.domElement?.setPointerCapture(event.pointerId)
-
-        scope.domElement?.addEventListener('pointermove', onPointerMove)
-        scope.domElement?.addEventListener('pointerup', onPointerUp)
+        scope.domElement?.ownerDocument.addEventListener('pointermove', onPointerMove)
+        scope.domElement?.ownerDocument.addEventListener('pointerup', onPointerUp)
       }
 
       addPointer(event)
@@ -843,8 +841,8 @@ class OrbitControls extends EventDispatcher {
       if (pointers.length === 0) {
         scope.domElement?.releasePointerCapture(event.pointerId)
 
-        scope.domElement?.removeEventListener('pointermove', onPointerMove)
-        scope.domElement?.removeEventListener('pointerup', onPointerUp)
+        scope.domElement?.ownerDocument.removeEventListener('pointermove', onPointerMove)
+        scope.domElement?.ownerDocument.removeEventListener('pointerup', onPointerUp)
       }
 
       scope.dispatchEvent(endEvent)