refactor: streamline right-click selection handling and event listener management in SelectionControls

This commit is contained in:
Jerald-Golden-B 2025-06-10 11:34:03 +05:30
parent 81664ba765
commit fdc8255c19
1 changed files with 4 additions and 10 deletions

View File

@ -84,7 +84,6 @@ const SelectionControls: React.FC = () => {
const onPointerUp = (event: PointerEvent) => { const onPointerUp = (event: PointerEvent) => {
if (event.button === 2 && !event.ctrlKey && !event.shiftKey) { if (event.button === 2 && !event.ctrlKey && !event.shiftKey) {
isRightClick.current = false; isRightClick.current = false;
rightClickMoved.current = false;
if (!rightClickMoved.current) { if (!rightClickMoved.current) {
clearSelection(); clearSelection();
} }
@ -171,18 +170,14 @@ const SelectionControls: React.FC = () => {
if (!rightClickMoved.current) { if (!rightClickMoved.current) {
clearSelection(); clearSelection();
} }
rightClickMoved.current = false;
}; };
if (!toggleView && activeModule === "builder") { if (!toggleView && activeModule === "builder") {
helper.enabled = true; helper.enabled = true;
if (duplicatedObjects.length === 0 && pastedObjects.length === 0) { canvasElement.addEventListener("pointermove", onPointerMove);
canvasElement.addEventListener("pointerdown", onPointerDown); canvasElement.addEventListener("pointerup", onPointerUp);
canvasElement.addEventListener("pointermove", onPointerMove); canvasElement.addEventListener("pointerdown", onPointerDown);
canvasElement.addEventListener("pointerup", onPointerUp);
} else {
helper.enabled = false;
helper.dispose();
}
canvasElement.addEventListener("contextmenu", onContextMenu); canvasElement.addEventListener("contextmenu", onContextMenu);
canvasElement.addEventListener("keydown", onKeyDown); canvasElement.addEventListener("keydown", onKeyDown);
} else { } else {
@ -208,7 +203,6 @@ const SelectionControls: React.FC = () => {
}, [activeModule]); }, [activeModule]);
useFrame(() => { useFrame(() => {
console.log(rightClickMoved.current);
if (pastedObjects.length === 0 && duplicatedObjects.length === 0 && movedObjects.length === 0 && rotatedObjects.length === 0) { if (pastedObjects.length === 0 && duplicatedObjects.length === 0 && movedObjects.length === 0 && rotatedObjects.length === 0) {
selectionGroup.current.position.set(0, 0, 0); selectionGroup.current.position.set(0, 0, 0);
} }