fix: useLongPress onClick can't get the latest state (alibaba#1426)

lily7129 · Jan 4, 2022 · 2ffa9b1 · 2ffa9b1
1 parent 1a67662
commit 2ffa9b1
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/packages/hooks/src/useLongPress/index.ts b/packages/hooks/src/useLongPress/index.ts
@@ -22,6 +22,7 @@ function useLongPress(
   { delay = 300, onClick }: Options = {},
 ) {
   const onLongPressRef = useLatest(onLongPress);
+  const onClickRef = useLatest(onClick);
 
   const timerRef = useRef<ReturnType<typeof setTimeout>>();
   const isTriggeredRef = useRef(false);
@@ -44,8 +45,8 @@ function useLongPress(
         if (timerRef.current) {
           clearTimeout(timerRef.current);
         }
-        if (shouldTriggerClick && !isTriggeredRef.current && onClick) {
-          onClick(event);
+        if (shouldTriggerClick && !isTriggeredRef.current && onClickRef.current) {
+          onClickRef.current(event);
         }
         isTriggeredRef.current = false;
       };