From 16e2bdc023200c978dd75626a2553642dbd957e2 Mon Sep 17 00:00:00 2001 From: sa-0001 <4160233+sa-0001@users.noreply.github.com> Date: Wed, 8 Mar 2023 12:02:06 +0100 Subject: [PATCH] fix-useResizeEvents-declare-order --- src/hooks/useResizeEvents.jsx | 50 +++++++++++++++++------------------ 1 file changed, 25 insertions(+), 25 deletions(-) diff --git a/src/hooks/useResizeEvents.jsx b/src/hooks/useResizeEvents.jsx index 53c2163..66a9707 100644 --- a/src/hooks/useResizeEvents.jsx +++ b/src/hooks/useResizeEvents.jsx @@ -7,6 +7,31 @@ const useResizeEvents = ( onResize, onResizeEnd ) => { + const onMouseMove = useCallback( + (event) => { + onResize({ event, target: resizeHandleRef.current, column }); + }, + [column, onResize, resizeHandleRef] + ); + + const onMouseUp = useCallback( + (event) => { + onResizeEnd({ event, target: resizeHandleRef.current, column }); + window.removeEventListener("mousemove", onMouseMove); + window.removeEventListener("mouseup", onMouseUp); + }, + [column, onMouseMove, onResizeEnd, resizeHandleRef] + ); + + const onMouseDown = useCallback( + (event) => { + onResizeStart({ event, target: resizeHandleRef.current, column }); + window.addEventListener("mousemove", onMouseMove); + window.addEventListener("mouseup", onMouseUp); + }, + [column, onMouseMove, onMouseUp, onResizeStart, resizeHandleRef] + ); + useEffect(() => { const resizeEl = resizeHandleRef.current; if (resizeEl) resizeEl.addEventListener("mousedown", onMouseDown); @@ -27,31 +52,6 @@ const useResizeEvents = ( onMouseMove, onMouseUp, ]); - - const onMouseDown = useCallback( - (event) => { - onResizeStart({ event, target: resizeHandleRef.current, column }); - window.addEventListener("mousemove", onMouseMove); - window.addEventListener("mouseup", onMouseUp); - }, - [column, onMouseMove, onMouseUp, onResizeStart, resizeHandleRef] - ); - - const onMouseMove = useCallback( - (event) => { - onResize({ event, target: resizeHandleRef.current, column }); - }, - [column, onResize, resizeHandleRef] - ); - - const onMouseUp = useCallback( - (event) => { - onResizeEnd({ event, target: resizeHandleRef.current, column }); - window.removeEventListener("mousemove", onMouseMove); - window.removeEventListener("mouseup", onMouseUp); - }, - [column, onMouseMove, onResizeEnd, resizeHandleRef] - ); }; export default useResizeEvents;