chore: revert

Author: sadmann7

docs/registry/default/ui/kanban.tsx

@@ -28,11 +28,9 @@ import {
   useSensors,
 } from "@dnd-kit/core";
 import {
-  type AnimateLayoutChanges,
   SortableContext,
   type SortableContextProps,
   arrayMove,
-  defaultAnimateLayoutChanges,
   useSortable,
   verticalListSortingStrategy,
 } from "@dnd-kit/sortable";
@@ -177,8 +175,6 @@ interface KanbanContextValue<T> {
   setActiveId: (id: UniqueIdentifier | null) => void;
   getItemValue: (item: T) => UniqueIdentifier;
   flatCursor: boolean;
-  clonedItems: Record<UniqueIdentifier, T[]> | null;
-  setClonedItems: (items: Record<UniqueIdentifier, T[]> | null) => void;
 }
 
 const KanbanContext = React.createContext<KanbanContextValue<unknown> | null>(
@@ -228,25 +224,11 @@ function Kanban<T>(props: KanbanProps<T>) {
   } = props;
 
   const [activeId, setActiveId] = React.useState<UniqueIdentifier | null>(null);
-  const [clonedItems, setClonedItems] = React.useState<Record<
-    UniqueIdentifier,
-    T[]
-  > | null>(null);
   const lastOverIdRef = React.useRef<UniqueIdentifier | null>(null);
-  const isMovedToNewContainerRef = React.useRef(false);
-
+  const isShiftingRef = React.useRef(false);
   const sensors = useSensors(
-    useSensor(MouseSensor, {
-      activationConstraint: {
-        distance: 8,
-      },
-    }),
-    useSensor(TouchSensor, {
-      activationConstraint: {
-        delay: 100,
-        tolerance: 8,
-      },
-    }),
+    useSensor(MouseSensor),
+    useSensor(TouchSensor),
     useSensor(KeyboardSensor, {
       coordinateGetter: coordinateGetter,
     }),
@@ -281,7 +263,7 @@ function Kanban<T>(props: KanbanProps<T>) {
     [value, getItemValue],
   );
 
-  const collisionDetectionStrategy: CollisionDetection = React.useCallback(
+  const collisionDetection: CollisionDetection = React.useCallback(
     (args) => {
       if (activeId && activeId in value) {
         return closestCenter({
@@ -300,7 +282,7 @@ function Kanban<T>(props: KanbanProps<T>) {
       let overId = getFirstCollision(intersections, "id");
 
       if (!overId) {
-        if (isMovedToNewContainerRef.current) {
+        if (isShiftingRef.current) {
           lastOverIdRef.current = activeId;
         }
         return lastOverIdRef.current ? [{ id: lastOverIdRef.current }] : [];
@@ -332,24 +314,6 @@ function Kanban<T>(props: KanbanProps<T>) {
     [activeId, value, getItemValue],
   );
 
-  const onDragStart = React.useCallback(
-    ({ active }: { active: { id: UniqueIdentifier } }) => {
-      setActiveId(active.id);
-      setClonedItems(value);
-      isMovedToNewContainerRef.current = false;
-    },
-    [value],
-  );
-
-  const onDragCancel = React.useCallback(() => {
-    if (clonedItems) {
-      onValueChange?.(clonedItems);
-    }
-    setActiveId(null);
-    setClonedItems(null);
-    isMovedToNewContainerRef.current = false;
-  }, [clonedItems, onValueChange]);
-
   const onDragOver = React.useCallback(
     (event: DragOverEvent) => {
       const { active, over } = event;
@@ -379,8 +343,6 @@ function Kanban<T>(props: KanbanProps<T>) {
       const activeItem = activeItems[activeIndex];
       if (!activeItem) return;
 
-      isMovedToNewContainerRef.current = true;
-
       const updatedItems = {
         ...value,
         [activeContainer]: activeItems.filter(
@@ -400,7 +362,6 @@ function Kanban<T>(props: KanbanProps<T>) {
 
       if (!over) {
         setActiveId(null);
-        isMovedToNewContainerRef.current = false;
         return;
       }
 
@@ -409,15 +370,13 @@ function Kanban<T>(props: KanbanProps<T>) {
 
       if (!activeContainer || !overContainer) {
         setActiveId(null);
-        isMovedToNewContainerRef.current = false;
         return;
       }
 
       if (activeContainer === overContainer) {
         const items = value[activeContainer];
         if (!items) {
           setActiveId(null);
-          isMovedToNewContainerRef.current = false;
           return;
         }
 
@@ -440,7 +399,6 @@ function Kanban<T>(props: KanbanProps<T>) {
       }
 
       setActiveId(null);
-      isMovedToNewContainerRef.current = false;
       onMove?.(event);
     },
     [value, getContainer, getItemValue, onValueChange, onMove],
@@ -457,8 +415,6 @@ function Kanban<T>(props: KanbanProps<T>) {
       setActiveId,
       getItemValue,
       flatCursor,
-      clonedItems,
-      setClonedItems,
     }),
     [
       id,
@@ -469,7 +425,6 @@ function Kanban<T>(props: KanbanProps<T>) {
       orientation,
       getItemValue,
       flatCursor,
-      clonedItems,
     ],
   );
 
@@ -479,19 +434,23 @@ function Kanban<T>(props: KanbanProps<T>) {
         id={id}
         modifiers={modifiers}
         sensors={sensorsProp ?? sensors}
-        collisionDetection={collisionDetectionStrategy}
+        collisionDetection={collisionDetection}
         measuring={{
           droppable: {
             strategy: MeasuringStrategy.Always,
           },
         }}
-        onDragStart={composeEventHandlers(kanbanProps.onDragStart, onDragStart)}
+        onDragStart={composeEventHandlers(
+          kanbanProps.onDragStart,
+          ({ active }) => {
+            setActiveId(active.id);
+          },
+        )}
         onDragOver={composeEventHandlers(kanbanProps.onDragOver, onDragOver)}
         onDragEnd={composeEventHandlers(kanbanProps.onDragEnd, onDragEnd)}
-        onDragCancel={composeEventHandlers(
-          kanbanProps.onDragCancel,
-          onDragCancel,
-        )}
+        onDragCancel={composeEventHandlers(kanbanProps.onDragCancel, () => {
+          setActiveId(null);
+        })}
         {...kanbanProps}
       />
     </KanbanContext.Provider>