fix : getPointFromEvent remove rect argument (#49)

Co-authored-by: myeonggyu.lee <[email protected]>
AirLabsTeam · Oct 3, 2024 · f8fcc1e · f8fcc1e
1 parent 7ae9ada
commit f8fcc1e
Showing 1 changed file with 3 additions and 5 deletions.
diff --git a/src/hooks/useSelectionLogic.ts b/src/hooks/useSelectionLogic.ts
@@ -95,10 +95,8 @@ export function useSelectionLogic<T extends HTMLElement>({
    * method to calculate point from event in context of the whole screen
    */
   const getPointFromEvent = useCallback(
-    (event: MouseEvent, rect?: DOMRect): Point => {
-      if (!rect) {
-        rect = containerRef.current?.getParentBoundingClientRect();
-      }
+    (event: MouseEvent): Point => {
+      const rect = containerRef.current?.getParentBoundingClientRect();
 
       return {
         x: event.clientX - (typeof rect?.left === 'number' ? rect.left : 0),
@@ -157,7 +155,7 @@ export function useSelectionLogic<T extends HTMLElement>({
       }
 
       const rect = containerRef.current?.getParentBoundingClientRect();
-      endPoint.current = getPointFromEvent(event, rect);
+      endPoint.current = getPointFromEvent(event);
       handleMouseMove(event, rect);
     },
     [handleMouseMove, getPointFromEvent, containerRef],