#6378 main.addon.DragDrop: passing the boundaryContainerRect to the app worker, draggable.toolbar.SortZone: onDragMove() => added a first check if a drag OP leaves the boundary rect

Author: tobiu

src/draggable/DragZone.mjs

@@ -56,6 +56,12 @@ class DragZone extends Base {
          * @member {String|String[]|null} boundaryContainerId=null
          */
         boundaryContainerId: null,
+        /**
+         * Stores the DOMRect matching this.boundaryContainerId
+         * @member {DOMRect|null} data=null
+         * @protected
+         */
+        boundaryContainerRect: null,
         /**
          * Store data which you want to pass to drop related events here
          * @member {Object|null} data=null
@@ -318,16 +324,18 @@ class DragZone extends Base {
             {appName, owner, windowId} = me,
             {cls}                      = owner,
             rect                       = me.getDragElementRect(data),
-            offsetX, offsetY;
+            mainData, offsetX, offsetY;
 
         me.setData();
 
-        Neo.main.addon.DragDrop.setConfigs({
+        mainData = await Neo.main.addon.DragDrop.setConfigs({
             appName,
             windowId,
             ...me.getMainThreadConfigs()
         });
 
+        me.boundaryContainerRect = mainData.boundaryContainerRect
+
         NeoArray.add(cls, 'neo-is-dragging');
         owner.cls = cls;
 

src/draggable/toolbar/SortZone.mjs

@@ -180,8 +180,12 @@ class SortZone extends DragZone {
 
         else if (index < maxItems && (!reversed && delta > 0 || reversed && delta < 0)) {
             if (Math.abs(delta) > itemRects[index + 1][itemWidth] * moveFactor) {
-                me.currentIndex++;
-                me.switchItems(index, me.currentIndex)
+                if (data.clientX > me.boundaryContainerRect.right) {
+                    console.log(delta, me.currentIndex, data, me.boundaryContainerRect);
+                } else {
+                    me.currentIndex++;
+                    me.switchItems(index, me.currentIndex)
+                }
             }
         }
     }

src/main/addon/DragDrop.mjs

@@ -445,6 +445,7 @@ class DragDrop extends Base {
      * @param {String|null}          data.scrollContainerId
      * @param {Number}               data.scrollFactorLeft
      * @param {Number}               data.scrollFactorTop
+     * @returns {Object} return the boundaryContainerRect
      */
     setConfigs(data) {
         let me                    = this,
@@ -496,6 +497,10 @@ class DragDrop extends Base {
                 }
             })
         }
+
+        return {
+            boundaryContainerRect: me.boundaryContainerRect || null
+        }
     }
 
     /**