@guiexpert/table 1.1.110 → 1.1.113

package/index.js

@@ -1250,20 +1250,26 @@ class Hn {
     (this.mouseDown || this.pendingDraggingEnd) && t && t.areaIdent === "body" && t.rowIndex >= 0 && (t.rowIndex = Math.max(0, e.mapIndexOrNext(t.rowIndex)));
   }
   onMouseMove(e) {
-    var t, s, o, i;
+    var t, s, o, i, r;
     if (!this.mouseDown && ((s = (t = this.tableScope).isEventInDetail) != null && s.call(t, e))) {
       (i = (o = this.tableScope).hideHover) == null || i.call(o);
       return;
     }
-    if (!(!this.mouseDown && this.isBlocked(e)))
+    if (!(!this.mouseDown && this.isBlocked(e))) {
+      if (this.mouseDown && !this.dragging) {
+        const l = (r = this.startMouseEvent) == null ? void 0 : r.originalEvent;
+        if (l && l.clientX === e.clientX && l.clientY === e.clientY)
+          return;
+      }
       if (this.mouseEvent = e, this.mouseDown) {
         this.dragging || (this.dragging = !0, this.tableScope.setDragging(!0));
-        const r = this.dragGeneration;
+        const l = this.dragGeneration;
         requestAnimationFrame(() => {
-          r === this.dragGeneration && this.mouseDraggingOnFrame();
+          l === this.dragGeneration && this.mouseDraggingOnFrame();
         });
       } else
         requestAnimationFrame(this.mouseMoveOnFrame.bind(this));
+    }
   }
   onMouseUp(e) {
     var s, o;
@@ -20206,7 +20212,7 @@ class th {
 const Ne = class Ne extends na {
   constructor(e, t, s, o, i, r = new Ae(), l = new Ia()) {
     var u;
-    if (super(e, t, new Qs(s), o), this.eventListener = i, this.copyService = r, this.excelService = l, this.licenseManager = at.getInstance(), this.listenerAbortController = new AbortController(), this.destroyed = !1, this.selectionService = new ge(this), this.api = new Yl(this), this.initialized = !1, this.mouseStartAction = "", this.mouseStartWidth = -1, this.mouseStartColumnIndex = -1, this.dragFrom = -1, this.dragTo = -1, this.lastDragFrom = -1, this.lastDragTo = -1, this.firstDraggingRendering = !0, this.editingOptions = _o.create(lt(this.tableOptions, "editing")), this.subscribers = /* @__PURE__ */ new Set(), this.clipboardOptions = ws.create(lt(this.tableOptions, "clipboard")), this.undoOptions = Go.create(lt(this.tableOptions, "undo")), this.lastTabKeyTime = 0, this.lastContextmenu = Date.now(), this.allExpanded = !0, this.selectionEventsHeld = 0, t.setTableScope(this), this.liveAnnouncer = new ao(e.ownerDocument, this.ariaIds.liveRegion(), s), this.eventListener = qc(i ?? new ls(), this.subscribers), this.stateService = new no(this), this.mouseHandler = new Hn(this), this.inputHandler = new pa(this), this.resizeHandler = new Ma(this, o.resizeEventDebounceDelay), this.shortcutService = new va(this), this.rowDragService = new ka(this), this.batchEditService = new Na(this), this.fillHandleService = new ja(this), this.editingOptions) {
+    if (super(e, t, new Qs(s), o), this.eventListener = i, this.copyService = r, this.excelService = l, this.licenseManager = at.getInstance(), this.listenerAbortController = new AbortController(), this.destroyed = !1, this.selectionService = new ge(this), this.api = new Yl(this), this.initialized = !1, this.mouseStartAction = "", this.mouseStartWidth = -1, this.mouseStartColumnIndex = -1, this.dragFrom = -1, this.lastDragFrom = -1, this.lastDragTo = -1, this.firstDraggingRendering = !0, this.editingOptions = _o.create(lt(this.tableOptions, "editing")), this.subscribers = /* @__PURE__ */ new Set(), this.clipboardOptions = ws.create(lt(this.tableOptions, "clipboard")), this.undoOptions = Go.create(lt(this.tableOptions, "undo")), this.lastTabKeyTime = 0, this.lastContextmenu = Date.now(), this.allExpanded = !0, this.selectionEventsHeld = 0, t.setTableScope(this), this.liveAnnouncer = new ao(e.ownerDocument, this.ariaIds.liveRegion(), s), this.eventListener = qc(i ?? new ls(), this.subscribers), this.stateService = new no(this), this.mouseHandler = new Hn(this), this.inputHandler = new pa(this), this.resizeHandler = new Ma(this, o.resizeEventDebounceDelay), this.shortcutService = new va(this), this.rowDragService = new ka(this), this.batchEditService = new Na(this), this.fillHandleService = new ja(this), this.editingOptions) {
       this.cellEditorService = new dc(this, this.editingOptions);
       const f = t.getBodyModel();
       this.editingOptions.defaultEditable && "defaultEditable" in f && (f.defaultEditable = !0), this.setHostAriaAttribute("aria-description", this.getSheetTexts().gridEditingHint());
@@ -21662,12 +21668,16 @@ const Ne = class Ne extends na {
    * horizontal scroll bar must stay usable), so a mouse event on a footer cell arrives at the viewport: then the footer
    * element below the mouse pointer is returned (not on the scroll bar itself).
    *
+   * A target that a render replaced meanwhile (an event handled in a later animation frame – a drag frame, the end of a
+   * drag – while live data renders) is resolved again at the pointer position: its `data-row-index` would be the index
+   * of an earlier render (since 1.1.112).
+   *
    * @param {MouseEvent} evt - a mouse event on the host element
    * @returns {EventTarget | null} the target element
    */
   getMouseEventTarget(e) {
-    const t = e.target;
-    if ((t === this.scrollViewport || t === this.contentWrapperDiv || t === this.contentDiv) && this.tableModel.getPadding().bottom > 0) {
+    let t = e.target;
+    if (t instanceof Element && !t.isConnected && Number.isFinite(e.clientX) && Number.isFinite(e.clientY) && (t = this.getCellElementAtPoint(e.clientX, e.clientY) ?? t), (t === this.scrollViewport || t === this.contentWrapperDiv || t === this.contentDiv) && this.tableModel.getPadding().bottom > 0) {
       const s = this.scrollViewport.getBoundingClientRect(), o = e.clientX - s.left, i = e.clientY - s.top >= this.scrollViewport.clientHeight || o < this.scrollViewport.clientLeft || o >= this.scrollViewport.clientLeft + this.scrollViewport.clientWidth, r = this.hostElement.getRootNode();
       if (!i && r.elementsFromPoint) {
         const l = r.elementsFromPoint(e.clientX, e.clientY).find((a) => this.hostElement.contains(a) && a.closest(".ge-table-footer"));
@@ -21677,6 +21687,12 @@ const Ne = class Ne extends na {
     }
     return t;
   }
+  /** The cell element (or an element in it) of this table at a client position, `undefined` if there is none. */
+  getCellElementAtPoint(e, t) {
+    var o;
+    const s = this.hostElement.getRootNode();
+    return (o = s.elementsFromPoint) == null ? void 0 : o.call(s, e, t).find((i) => this.hostElement.contains(i) && !!i.closest("[data-col-index]"));
+  }
   /**
    * Handles the mouse down event.
    *
@@ -21705,7 +21721,7 @@ const Ne = class Ne extends na {
   /** Forgets the column action of the last press (no events). */
   resetColumnDrag() {
     var e, t;
-    (t = (e = this.columnDrag) == null ? void 0 : e.keyAbort) == null || t.abort(), this.columnDrag = void 0, this.mouseStartAction = "", this.mouseStartWidth = -1, this.mouseStartColumnIndex = -1, this.dragFrom = -1, this.dragTo = -1, this.firstDraggingRendering = !0;
+    (t = (e = this.columnDrag) == null ? void 0 : e.keyAbort) == null || t.abort(), this.columnDrag = void 0, this.mouseStartAction = "", this.mouseStartWidth = -1, this.mouseStartColumnIndex = -1, this.dragFrom = -1, this.firstDraggingRendering = !0;
   }
   /**
    * Starts the column drag of this press once the mouse moved past {@link MouseHandler.clickMoveTolerance}: reports
@@ -21823,7 +21839,7 @@ const Ne = class Ne extends na {
       const r = Array.from({ length: s }, (a, c) => this.isColumnMovable(c)), l = la(r, this.dragFrom, o);
       l != null && l.length && (l.forEach(([a, c]) => t.changeColumnOrder(a, c)), (i = this.columnDrag) == null || i.moves.push(...l), this.lastDragFrom = this.dragFrom, this.lastDragTo = o, this.dragFrom = o, this.columnDrag && (this.columnDrag.currentIndex = o), this.resetSizeOfWrapperDiv(), this.adjustContainersAndRows());
     }
-    this.draggingTargetColumnIndex = this.dragFrom, this.dragTo = this.dragFrom;
+    this.draggingTargetColumnIndex = this.dragFrom;
   }
   /**
    * Handles the end of mouse dragging event on a frame.

package/lib/table/table-scope.d.ts

@@ -128,7 +128,6 @@ export declare class TableScope extends RenderScope implements OnActionTriggered
     /** The column resize or move of the current press (prompt 022) – `undefined` without an allowed column action. */
     private columnDrag?;
     private dragFrom;
-    private dragTo;
     private lastDragFrom;
     private lastDragTo;
     private firstDraggingRendering;
@@ -777,10 +776,16 @@ export declare class TableScope extends RenderScope implements OnActionTriggered
      * horizontal scroll bar must stay usable), so a mouse event on a footer cell arrives at the viewport: then the footer
      * element below the mouse pointer is returned (not on the scroll bar itself).
      *
+     * A target that a render replaced meanwhile (an event handled in a later animation frame – a drag frame, the end of a
+     * drag – while live data renders) is resolved again at the pointer position: its `data-row-index` would be the index
+     * of an earlier render (since 1.1.112).
+     *
      * @param {MouseEvent} evt - a mouse event on the host element
      * @returns {EventTarget | null} the target element
      */
     getMouseEventTarget(evt: MouseEvent): EventTarget | null;
+    /** The cell element (or an element in it) of this table at a client position, `undefined` if there is none. */
+    private getCellElementAtPoint;
     /**
      * Handles the mouse down event.
      *

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@guiexpert/table",
-  "version": "1.1.110",
+  "version": "1.1.113",
   "license": "MIT",
   "author": "Dr. Marc Kronberg <info@gui.expert> (https://gui.expert)",
   "repository": {