Merge pull request #129 from limxingzhi/fix/119/unnecessary-renders

fix: optimized react for unnecessary renders
formkit · Sep 5, 2023 · c8d2202 · c8d2202 · vercel · Sep 5, 2023
2 parents 5a4cb34 + 388d037
commit c8d2202
Showing 1 changed file with 10 additions and 7 deletions.
diff --git a/src/react/index.ts b/src/react/index.ts
@@ -20,14 +20,17 @@ export function useAutoAnimate<T extends Element>(
     if (node instanceof HTMLElement) {
       setController(autoAnimate(node, options))
     } else {
-      setController(undefined);
+      setController(undefined)
     }
-  }, [])
-  const setEnabled = (enabled: boolean) => {
-    if (controller) {
-      enabled ? controller.enable() : controller.disable()
-    }
-  }
+  }, [options])
+  const setEnabled = useCallback(
+    (enabled: boolean) => {
+      if (controller) {
+        enabled ? controller.enable() : controller.disable()
+      }
+    },
+    [controller]
+  )
 
   return [element, setEnabled]
 }