From efe1cebb4c1d8f07c0f1afb8bf08732d9fe20125 Mon Sep 17 00:00:00 2001 From: Fabian Mastenbroek Date: Sun, 26 Mar 2023 21:49:26 +0100 Subject: bug(web): Do not offset hover layer after dragging This change fixes #136 which reported that the grid and cursor will fall out of sync when dragging or moving the grid when placing rooms or objects. Fixes #136 --- opendc-web/opendc-web-ui/src/util/effect-ref.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'opendc-web/opendc-web-ui/src/util/effect-ref.js') diff --git a/opendc-web/opendc-web-ui/src/util/effect-ref.js b/opendc-web/opendc-web-ui/src/util/effect-ref.js index cda0324b..78528585 100644 --- a/opendc-web/opendc-web-ui/src/util/effect-ref.js +++ b/opendc-web/opendc-web-ui/src/util/effect-ref.js @@ -26,9 +26,9 @@ const noop = () => {} /** * A hook that will invoke the specified callback when the reference returned by this function is initialized. - * The callback can return an optional clean up function. + * The callback can return an optional clean-up function. */ -export function useEffectRef(callback) { +export function useEffectRef(callback, deps = []) { const disposeRef = useRef(noop) return useCallback((element) => { disposeRef.current() @@ -37,5 +37,5 @@ export function useEffectRef(callback) { if (element) { disposeRef.current = callback(element) || noop } - }, []) // eslint-disable-line react-hooks/exhaustive-deps + }, deps) // eslint-disable-line react-hooks/exhaustive-deps } -- cgit v1.2.3