fix(TagInput): onDragSort capture context error caused by useRef (#3003)

* fix(TagInput): onDragSort capture context error caused by useRef * fix(usedragsorter): usage useEventCallback --------- Co-authored-by: HaixingOoO <[email protected]>
Tencent · Jul 31, 2024 · 7e41d7a · 7e41d7a
1 parent 897299d
commit 7e41d7a
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/src/hooks/useDragSorter.tsx b/src/hooks/useDragSorter.tsx
@@ -1,4 +1,5 @@
-import { useCallback, useRef, useState } from 'react';
+import { useCallback, useState } from 'react';
+import useEventCallback from './useEventCallback';
 
 interface DragSortProps<T> {
   sortOnDraggable: boolean;
@@ -36,8 +37,8 @@ function useDragSorter<T>(props: DragSortProps<T>): DragSortInnerProps {
   const [dragStartData, setDragStartData] = useState(null);
   const [isDropped, setIsDropped] = useState(null);
   const [startInfo, setStartInfo] = useState({ nodeX: 0, nodeWidth: 0, mouseX: 0 });
+  const onDragSortEvent = useEventCallback(onDragSort);
 
-  const onDragSortRef = useRef(onDragSort);
   const onDragOver = useCallback(
     (e, index, record: T) => {
       e.preventDefault();
@@ -64,7 +65,7 @@ function useDragSorter<T>(props: DragSortProps<T>): DragSortInnerProps {
         if (!overlap) return;
       }
 
-      onDragSortRef.current?.({
+      onDragSortEvent({
         currentIndex: draggingIndex,
         current: dragStartData,
         target: record,
@@ -80,6 +81,7 @@ function useDragSorter<T>(props: DragSortProps<T>): DragSortInnerProps {
       startInfo.nodeWidth,
       startInfo.mouseX,
       startInfo.nodeX,
+      onDragSortEvent,
     ],
   );