fix: useDebounce remove deps from function arguments (#623)

streamich · Sep 26, 2019 · 23d6a5a · 23d6a5a
1 parent 8c0d178
commit 23d6a5a
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/src/useDebounce.ts b/src/useDebounce.ts
@@ -1,14 +1,14 @@
+import { DependencyList } from 'react';
 import useUpdateEffect from './useUpdateEffect';
 
-const useDebounce = (fn: () => any, ms: number = 0, args: any[] = []) => {
+const useDebounce = (fn: () => any, ms: number = 0, deps: DependencyList = []) => {
   useUpdateEffect(() => {
-    const handle = setTimeout(fn.bind(null, args), ms);
+    const timeout = setTimeout(fn, ms);
 
     return () => {
-      // if args change then clear timeout
-      clearTimeout(handle);
+      clearTimeout(timeout);
     };
-  }, args);
+  }, deps);
 };
 
 export default useDebounce;