@rpg-engine/long-bow 0.7.50 → 0.7.51

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rpg-engine/long-bow",
-  "version": "0.7.50",
+  "version": "0.7.51",
   "license": "MIT",
   "main": "dist/index.js",
   "typings": "dist/index.d.ts",

package/src/components/Item/Inventory/ItemSlot.tsx

@@ -142,7 +142,6 @@ export const ItemSlot = React.memo(
       const [contextActions, setContextActions] = useState<IContextMenuItem[]>(
         []
       );
-      const [dragStartTime, setDragStartTime] = useState<number | null>(null);
 
       useEffect(() => {
         setDragState(prev => ({
@@ -216,8 +215,12 @@ export const ItemSlot = React.memo(
         // to prevent the item from being dragged again
         target.classList.remove('react-draggable-dragging');
 
-        const dragEndTime = Date.now();
-        const dragDuration = dragStartTime ? dragEndTime - dragStartTime : 0;
+        const isTouch = e.type.startsWith('touch');
+
+        // Threshold for considering a tap/click as a drag
+        const dragThreshold = 5; // pixels
+        const isDrag =
+          Math.abs(data.x) > dragThreshold || Math.abs(data.y) > dragThreshold;
 
         if (dragState.wasDragged && item && !isSelectingShortcut) {
           //@ts-ignore
@@ -270,23 +273,16 @@ export const ItemSlot = React.memo(
               }));
             }
           }, 50);
-        } else if (item && dragDuration < 200) {
-          // Consider it a click if drag duration is less than 200ms
-          let isTouch = false;
-          if (
+        } else if (item) {
+          if (isTouch && !isDrag) {
+            setTooltipState(prev => ({
+              ...prev,
+              mobileVisible: true,
+            }));
+          } else if (
+            !isTouch &&
             !isContextMenuDisabled &&
-            e.type === 'touchend' &&
             !isSelectingShortcut
-          ) {
-            isTouch = true;
-            setTooltipState(prev => ({ ...prev, mobileVisible: true }));
-          }
-
-          if (
-            !isContextMenuDisabled &&
-            !isSelectingShortcut &&
-            !isTouch &&
-            !dragState.wasDragged
           ) {
             setContextMenuState(prev => ({
               ...prev,
@@ -306,8 +302,14 @@ export const ItemSlot = React.memo(
             }
           }
 
-          onPointerDown(item.type, containerType ?? null, item);
+          if (!isDrag || !isTouch) {
+            console.log('Calling onPointerDown');
+            onPointerDown(item.type, containerType ?? null, item);
+          }
         }
+
+        setDragState(prev => ({ ...prev, wasDragged: false }));
+        console.log('Final dragState:', dragState);
       };
 
       const onDraggableStart: DraggableEventHandler = () => {
@@ -315,8 +317,6 @@ export const ItemSlot = React.memo(
           return;
         }
 
-        setDragStartTime(Date.now());
-
         if (onDragStart && containerType) {
           onDragStart(item, slotIndex, containerType);
         }