From 0fbc507eecbf68c2794da957db08dd8a04df8558 Mon Sep 17 00:00:00 2001
From: Vincent Fretin <vincent.fretin@gmail.com>
Date: Mon, 15 Jul 2024 17:29:58 +0200
Subject: [PATCH] Don't select entity on raycaster click if we used
 EditorControls or TransformControls in the last 500ms (fix #645)

---
 src/editor/lib/raycaster.js | 46 +------------------------------------
 src/editor/lib/viewport.js  | 13 +++++++++++
 2 files changed, 14 insertions(+), 45 deletions(-)

diff --git a/src/editor/lib/raycaster.js b/src/editor/lib/raycaster.js
index c06dfd248..c4be1e0e5 100644
--- a/src/editor/lib/raycaster.js
+++ b/src/editor/lib/raycaster.js
@@ -44,8 +44,6 @@ export function initRaycaster(inspector) {
   mouseCursor.addEventListener('click', handleClick);
   mouseCursor.addEventListener('mouseenter', onMouseEnter);
   mouseCursor.addEventListener('mouseleave', onMouseLeave);
-  inspector.container.addEventListener('mousedown', onMouseDown);
-  inspector.container.addEventListener('mouseup', onMouseUp);
   // inspector.container.addEventListener('dblclick', onDoubleClick);
 
   inspector.sceneEl.canvas.addEventListener('mouseleave', () => {
@@ -54,8 +52,6 @@ export function initRaycaster(inspector) {
     });
   });
 
-  const onDownPosition = new THREE.Vector2();
-  const onUpPosition = new THREE.Vector2();
   // const onDoubleClickPosition = new THREE.Vector2();
 
   function onMouseEnter() {
@@ -73,38 +69,7 @@ export function initRaycaster(inspector) {
   }
 
   function handleClick(evt) {
-    // Check to make sure not dragging.
-    const DRAG_THRESHOLD = 0.03;
-    if (onDownPosition.distanceTo(onUpPosition) >= DRAG_THRESHOLD) {
-      return;
-    }
-    inspector.selectEntity(evt.detail.intersectedEl);
-  }
-
-  function onMouseDown(event) {
-    if (event instanceof CustomEvent) {
-      return;
-    }
-    event.preventDefault();
-    const array = getMousePosition(
-      inspector.container,
-      event.clientX,
-      event.clientY
-    );
-    onDownPosition.fromArray(array);
-  }
-
-  function onMouseUp(event) {
-    if (event instanceof CustomEvent) {
-      return;
-    }
-    event.preventDefault();
-    const array = getMousePosition(
-      inspector.container,
-      event.clientX,
-      event.clientY
-    );
-    onUpPosition.fromArray(array);
+    Events.emit('raycasterclick', evt.detail.intersectedEl);
   }
 
   /**
@@ -128,18 +93,9 @@ export function initRaycaster(inspector) {
     el: mouseCursor,
     enable: () => {
       mouseCursor.setAttribute('raycaster', 'enabled', true);
-      inspector.container.addEventListener('mousedown', onMouseDown);
-      inspector.container.addEventListener('mouseup', onMouseUp);
     },
     disable: () => {
       mouseCursor.setAttribute('raycaster', 'enabled', false);
-      inspector.container.removeEventListener('mousedown', onMouseDown);
-      inspector.container.removeEventListener('mouseup', onMouseUp);
     }
   };
 }
-
-function getMousePosition(dom, x, y) {
-  const rect = dom.getBoundingClientRect();
-  return [(x - rect.left) / rect.width, (y - rect.top) / rect.height];
-}
diff --git a/src/editor/lib/viewport.js b/src/editor/lib/viewport.js
index 8948bf567..9453ea75e 100644
--- a/src/editor/lib/viewport.js
+++ b/src/editor/lib/viewport.js
@@ -136,6 +136,15 @@ export function Viewport(inspector) {
     hoverBox.visible = false;
   });
 
+  let lastControlsEventTime = 0;
+  Events.on('raycasterclick', (el) => {
+    if (Date.now() - lastControlsEventTime < 500) {
+      // Ignore click if we used EditorControls or TransformControls in the last 500ms.
+      return;
+    }
+    inspector.selectEntity(el);
+  });
+
   function updateHelpers(object) {
     object.traverse((node) => {
       if (inspector.helpers[node.uuid] && inspector.helpers[node.uuid].update) {
@@ -151,6 +160,7 @@ export function Viewport(inspector) {
   );
   transformControls.size = 0.75;
   transformControls.addEventListener('objectChange', (evt) => {
+    lastControlsEventTime = Date.now();
     const object = transformControls.object;
     if (object === undefined) {
       return;
@@ -210,6 +220,9 @@ export function Viewport(inspector) {
   controls.rotationSpeed = 0.0035;
   controls.zoomSpeed = 0.05;
   controls.setAspectRatio(sceneEl.canvas.width / sceneEl.canvas.height);
+  controls.addEventListener('change', (evt) => {
+    lastControlsEventTime = Date.now();
+  });
 
   Events.on('cameratoggle', (data) => {
     controls.setCamera(data.camera);