release/2025-04-25: Patch Updates to packages

client/package-lock.json

@@ -70,7 +70,7 @@
         "react-resizable": "^3.0.5",
         "react-router-dom": "^6.30.0",
         "react-sticky": "^6.0.3",
-        "react-virtuoso": "4.12.5",
+        "react-virtuoso": "^4.12.6",
         "recharts": "^2.15.2",
         "redux": "^5.0.1",
         "redux-actions": "^3.0.3",
@@ -14484,9 +14484,9 @@
       }
     },
     "node_modules/react-virtuoso": {
-      "version": "4.12.5",
-      "resolved": "https://registry.npmjs.org/react-virtuoso/-/react-virtuoso-4.12.5.tgz",
-      "integrity": "sha512-YeCbRRsC9CLf0buD0Rct7WsDbzf+yBU1wGbo05/XjbcN2nJuhgh040m3y3+6HVogTZxEqVm45ac9Fpae4/MxRQ==",
+      "version": "4.12.6",
+      "resolved": "https://registry.npmjs.org/react-virtuoso/-/react-virtuoso-4.12.6.tgz",
+      "integrity": "sha512-bfvS6aCL1ehXmq39KRiz/vxznGUbtA27I5I24TYCe1DhMf84O3aVNCIwrSjYQjkJGJGzY46ihdN8WkYlemuhMQ==",
       "license": "MIT",
       "peerDependencies": {
         "react": ">=16 || >=17 || >= 18 || >= 19",

client/package.json

@@ -69,7 +69,7 @@
     "react-resizable": "^3.0.5",
     "react-router-dom": "^6.30.0",
     "react-sticky": "^6.0.3",
-    "react-virtuoso": "4.12.5",
+    "react-virtuoso": "^4.12.6",
     "recharts": "^2.15.2",
     "redux": "^5.0.1",
     "redux-actions": "^3.0.3",

client/src/components/production-board-kanban/trello-board/controllers/BoardContainer.jsx

@@ -100,26 +100,28 @@ const BoardContainer = ({
   const onLaneDrag = useCallback(
     async ({ draggableId, type, source, reason, mode, destination, combine }) => {
       setIsDragging(false);
-      setDragTime(source.droppableId);
-      if (!type || type !== "lane" || !source || !destination || isEqual(source, destination)) return;
 
-      setIsProcessing(true);
+      // Only update drag time if it's a valid drop with a different destination
+      if (type === "lane" && source && destination && !isEqual(source, destination)) {
+        setDragTime(source.droppableId);
+        setIsProcessing(true);
 
-      dispatch(
-        actions.moveCardAcrossLanes({
-          fromLaneId: source.droppableId,
-          toLaneId: destination.droppableId,
-          cardId: draggableId,
-          index: destination.index
-        })
-      );
+        dispatch(
+          actions.moveCardAcrossLanes({
+            fromLaneId: source.droppableId,
+            toLaneId: destination.droppableId,
+            cardId: draggableId,
+            index: destination.index
+          })
+        );
 
-      try {
-        await onDragEnd({ draggableId, type, source, reason, mode, destination, combine });
-      } catch (err) {
-        console.error("Error in onLaneDrag", err);
-      } finally {
-        setIsProcessing(false);
+        try {
+          await onDragEnd({ draggableId, type, source, reason, mode, destination, combine });
+        } catch (err) {
+          console.error("Error in onLaneDrag", err);
+        } finally {
+          setIsProcessing(false);
+        }
       }
     },
     [dispatch, onDragEnd, setDragTime]

client/src/components/production-board-kanban/trello-board/controllers/Lane.jsx

@@ -120,15 +120,14 @@ const Lane = ({
       const Component = orientation === "vertical" ? VirtuosoGrid : Virtuoso;
       const FinalComponent = collapsed ? "div" : Component;
       const commonProps = {
-        useWindowScroll: true,
-        data: renderedCards
+        data: renderedCards,
+        customScrollParent: laneRef.current
       };
 
       const verticalProps = {
         ...commonProps,
         listClassName: "grid-container",
         itemClassName: "grid-item",
-        customScrollParent: laneRef.current,
         components: {
           List: ListComponent,
           Item: ItemComponent
@@ -142,7 +141,6 @@ const Lane = ({
         components: { Item: HeightPreservingItem },
         overscan: { main: 3, reverse: 3 },
         itemContent: (index, item) => renderDraggable(index, item),
-        scrollerRef: provided.innerRef,
         style: {
           minWidth: maxCardWidth,
           minHeight: maxLaneHeight
@@ -180,13 +178,14 @@ const Lane = ({
           override={orientation !== "horizontal" && (collapsed || !renderedCards.length)}
         >
           <div
-            {...provided.droppableProps}
-            ref={provided.innerRef}
+            ref={laneRef} // Ensure laneRef is set here
+            style={{ height: "100%", width: "100%" }} // Make it scrollable
             className={`react-trello-lane ${collapsed ? "lane-collapsed" : ""}`}
-            style={{ ...provided.droppableProps.style }}
           >
-            <FinalComponent {...finalComponentProps} />
-            {shouldRenderPlaceholder && provided.placeholder}
+            <div {...provided.droppableProps} ref={provided.innerRef} style={{ ...provided.droppableProps.style }}>
+              <FinalComponent {...finalComponentProps} />
+              {shouldRenderPlaceholder && provided.placeholder}
+            </div>
           </div>
         </HeightMemoryWrapper>
       );