@os-design/drag-sort 1.0.46 → 1.0.48

package/dist/Droppable.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"Droppable.d.ts","sourceRoot":"","sources":["../src/Droppable.tsx"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,EAEZ,KAAK,SAAS,EAKf,MAAM,OAAO,CAAC;AACf,OAAiB,EAGf,KAAK,iBAAiB,EACvB,MAAM,qBAAqB,CAAC;AAK7B,MAAM,WAAW,sBAAsB;IACrC;;;OAGG;IACH,GAAG,EAAE,SAAS,CAAC,cAAc,GAAG,IAAI,CAAC,CAAC;IACtC;;;OAGG;IACH,QAAQ,EAAE,SAAS,CAAC,cAAc,GAAG,IAAI,CAAC,CAAC;CAC5C;AAED,MAAM,WAAW,cAAc;IAC7B;;OAEG;IACH,iBAAiB,EAAE,iBAAiB,CAAC;IACrC;;;;OAIG;IACH,EAAE,CAAC,EAAE,MAAM,CAAC;IACZ;;;OAGG;IACH,UAAU,CAAC,EAAE,OAAO,CAAC;IACrB;;OAEG;IACH,QAAQ,EAAE,CAAC,KAAK,EAAE,sBAAsB,KAAK,KAAK,CAAC,SAAS,CAAC;CAC9D;AAED,QAAA,MAAM,SAAS,EAAE,KAAK,CAAC,EAAE,CAAC,cAAc,CA8EvC,CAAC;AAEF,eAAe,SAAS,CAAC"}
+{"version":3,"file":"Droppable.d.ts","sourceRoot":"","sources":["../src/Droppable.tsx"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,EAEZ,KAAK,SAAS,EAMf,MAAM,OAAO,CAAC;AACf,OAAiB,EAGf,KAAK,iBAAiB,EACvB,MAAM,qBAAqB,CAAC;AAK7B,MAAM,WAAW,sBAAsB;IACrC;;;OAGG;IACH,GAAG,EAAE,SAAS,CAAC,cAAc,GAAG,IAAI,CAAC,CAAC;IACtC;;;OAGG;IACH,QAAQ,EAAE,SAAS,CAAC,cAAc,GAAG,IAAI,CAAC,CAAC;CAC5C;AAED,MAAM,WAAW,cAAc;IAC7B;;OAEG;IACH,iBAAiB,EAAE,iBAAiB,CAAC;IACrC;;;;OAIG;IACH,EAAE,CAAC,EAAE,MAAM,CAAC;IACZ;;;OAGG;IACH,UAAU,CAAC,EAAE,OAAO,CAAC;IACrB;;OAEG;IACH,QAAQ,EAAE,CAAC,KAAK,EAAE,sBAAsB,KAAK,KAAK,CAAC,SAAS,CAAC;CAC9D;AAED,QAAA,MAAM,SAAS,EAAE,KAAK,CAAC,EAAE,CAAC,cAAc,CAwFvC,CAAC;AAEF,eAAe,SAAS,CAAC"}

package/dist/Droppable.js

@@ -1,5 +1,5 @@
 import useMemoObject from '@os-design/use-memo-object';
-import React, { useEffect, useMemo, useRef } from 'react';
+import React, { useCallback, useEffect, useMemo, useRef } from 'react';
 import NodeList from './utils/NodeList.js';
 import useDragAndDrop from './utils/useDragAndDrop.js';
 import { DroppableContext } from './utils/useDroppable.js';
@@ -55,18 +55,20 @@ const Droppable = ({
     const listId = listRef.current.id;
     return () => deregisterList(listId);
   }, [deregisterList, registerList]);
-  const registerNode = props => listRef.current.add(props);
-  const deregisterNode = node => {
+
+  /* eslint-disable react-hooks/preserve-manual-memoization */
+  const registerNode = useCallback(props => listRef.current.add(props), []);
+  const deregisterNode = useCallback(node => {
     listRef.current.remove(node);
-  };
+  }, []);
 
   // Handlers that determine whether a user clicks on the node
-  const mouseDownHandler = (e, node) => {
+  const mouseDownHandler = useCallback((e, node) => {
     onMouseDown(e, listRef.current, node);
-  };
-  const touchStartHandler = (e, node) => {
+  }, [onMouseDown]);
+  const touchStartHandler = useCallback((e, node) => {
     onTouchStart(e, listRef.current, node);
-  };
+  }, [onTouchStart]);
   const droppableContext = useMemoObject({
     registerNode,
     deregisterNode,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@os-design/drag-sort",
-  "version": "1.0.46",
+  "version": "1.0.48",
   "license": "UNLICENSED",
   "repository": "git@gitlab.com:os-team/libs/os-design.git",
   "type": "module",
@@ -19,7 +19,7 @@
   ],
   "sideEffects": false,
   "scripts": {
-    "clean": "rimraf dist",
+    "clean": "rm -r dist",
     "build:esm": "cross-env BABEL_ENV=esm babel src --root-mode upward --extensions .ts,.tsx --out-dir dist",
     "build:types": "tsc -p tsconfig.build.json --emitDeclarationOnly --declaration --declarationDir dist",
     "build": "yarn clean && npm-run-all 'build:*'",
@@ -29,22 +29,22 @@
     "access": "public"
   },
   "dependencies": {
-    "@os-design/portal": "^1.0.28",
-    "@os-design/use-auto-scroll": "^1.0.32",
-    "@os-design/use-drag": "^1.0.31",
-    "@os-design/use-event": "^1.0.33",
-    "@os-design/use-memo-object": "^1.0.29",
-    "@os-design/use-prevent-default-event": "^1.0.30",
-    "@os-design/use-throttle": "^1.0.32"
+    "@os-design/portal": "^1.0.29",
+    "@os-design/use-auto-scroll": "^1.0.33",
+    "@os-design/use-drag": "^1.0.32",
+    "@os-design/use-event": "^1.0.34",
+    "@os-design/use-memo-object": "^1.0.30",
+    "@os-design/use-prevent-default-event": "^1.0.31",
+    "@os-design/use-throttle": "^1.0.33"
   },
   "devDependencies": {
-    "@os-design/omit-emotion-props": "^1.0.31",
-    "@os-design/use-size": "^1.0.34",
+    "@os-design/omit-emotion-props": "^1.0.32",
+    "@os-design/use-size": "^1.0.35",
     "react-window": "^1.8.11"
   },
   "peerDependencies": {
     "react": "19",
     "react-dom": "19"
   },
-  "gitHead": "f333f60e39273574d2227cf0ee70b0c8897b2dde"
+  "gitHead": "4a7b019591e2ff76adb0452f2a850a35ee785d38"
 }

package/src/Droppable.tsx

@@ -3,6 +3,7 @@ import React, {
   type MouseEvent,
   type RefObject,
   type TouchEvent,
+  useCallback,
   useEffect,
   useMemo,
   useRef,
@@ -101,18 +102,28 @@ const Droppable: React.FC<DroppableProps> = ({
     return () => deregisterList(listId);
   }, [deregisterList, registerList]);
 
-  const registerNode = (props: NodeProps) => listRef.current.add(props);
-  const deregisterNode = (node: ExistingNode) => {
+  /* eslint-disable react-hooks/preserve-manual-memoization */
+  const registerNode = useCallback(
+    (props: NodeProps) => listRef.current.add(props),
+    []
+  );
+  const deregisterNode = useCallback((node: ExistingNode) => {
     listRef.current.remove(node);
-  };
+  }, []);
 
   // Handlers that determine whether a user clicks on the node
-  const mouseDownHandler = (e: MouseEvent, node: ExistingNode) => {
-    onMouseDown(e, listRef.current, node);
-  };
-  const touchStartHandler = (e: TouchEvent, node: ExistingNode) => {
-    onTouchStart(e, listRef.current, node);
-  };
+  const mouseDownHandler = useCallback(
+    (e: MouseEvent, node: ExistingNode) => {
+      onMouseDown(e, listRef.current, node);
+    },
+    [onMouseDown]
+  );
+  const touchStartHandler = useCallback(
+    (e: TouchEvent, node: ExistingNode) => {
+      onTouchStart(e, listRef.current, node);
+    },
+    [onTouchStart]
+  );
 
   const droppableContext = useMemoObject({
     registerNode,