fix(hook): re-attached scroll event behavior in useScrollPosition (#…

…3247) * fix: initialize throttleTimeout in cleanup function * chore(changeset): add changeset --------- Co-authored-by: WK Wong <wingkwong.code@gmail.com>
nextui-org · Sep 12, 2024 · b0728e3 · b0728e3
1 parent 142f394
commit b0728e3
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/.changeset/fast-students-promise.md b/.changeset/fast-students-promise.md
@@ -0,0 +1,5 @@
+---
+"@nextui-org/use-scroll-position": patch
+---
+
+re-attach scroll event behavior in `useScrollPosition`
diff --git a/packages/hooks/use-scroll-position/src/index.ts b/packages/hooks/use-scroll-position/src/index.ts
@@ -77,6 +77,7 @@ export const useScrollPosition = (props: UseScrollPositionOptions): ScrollValue
       target.removeEventListener("scroll", handleScroll);
       if (throttleTimeout.current) {
         clearTimeout(throttleTimeout.current);
+        throttleTimeout.current = null;
       }
     };
   }, [elementRef?.current, delay, handler, isEnabled]);