From 96cba557f60e52d9a5b3d858953d3e40dcb21f86 Mon Sep 17 00:00:00 2001
From: Christopher Allen <cpcallen+git@google.com>
Date: Fri, 2 May 2025 15:10:56 +0100
Subject: [PATCH 1/2] feat(mover): Report keyboard moves to WorkspaceSvg

Call Workspace.prototype.setKeyboardMoveInProgress at the beginning
and end of each keyboard-initiated move.
---
 src/actions/mover.ts | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/src/actions/mover.ts b/src/actions/mover.ts
index 9093e065..9673a26a 100644
--- a/src/actions/mover.ts
+++ b/src/actions/mover.ts
@@ -127,6 +127,7 @@ export class Mover {
     if (!DraggerClass) throw new Error('no Dragger registered');
     const dragger = new DraggerClass(block, workspace);
     // Record that a move is in progress and start dragging.
+    workspace.setKeyboardMoveInProgress(true);
     const info = new MoveInfo(block, dragger);
     this.moves.set(workspace, info);
     // Begin drag.
@@ -157,6 +158,7 @@ export class Mover {
     this.unpatchWorkspace(workspace);
     this.unpatchDragStrategy(info.block);
     this.moves.delete(workspace);
+    workspace.setKeyboardMoveInProgress(false);
     // Delay scroll until after block has finished moving.
     setTimeout(() => this.scrollCurrentBlockIntoView(workspace), 0);
     return true;
@@ -203,6 +205,7 @@ export class Mover {
     this.unpatchWorkspace(workspace);
     this.unpatchDragStrategy(info.block);
     this.moves.delete(workspace);
+    workspace.setKeyboardMoveInProgress(false);
     // Delay scroll until after block has finished moving.
     setTimeout(() => this.scrollCurrentBlockIntoView(workspace), 0);
     return true;

From b71eaad91698fb74063041fb02d7c618e2c282cf Mon Sep 17 00:00:00 2001
From: Christopher Allen <cpcallen+git@google.com>
Date: Fri, 2 May 2025 19:31:41 +0100
Subject: [PATCH 2/2] chore(Mover): Remove obsolete WorkspaceSvg monkey
 patching

---
 src/actions/mover.ts | 42 ------------------------------------------
 1 file changed, 42 deletions(-)

diff --git a/src/actions/mover.ts b/src/actions/mover.ts
index 9673a26a..28795d0b 100644
--- a/src/actions/mover.ts
+++ b/src/actions/mover.ts
@@ -116,7 +116,6 @@ export class Mover {
     block: BlockSvg,
     insertStartPoint: RenderedConnection | null,
   ) {
-    this.patchWorkspace(workspace);
     this.patchDragStrategy(block, insertStartPoint);
     // Begin dragging block.
     const DraggerClass = registry.getClassFromOptions(
@@ -155,7 +154,6 @@ export class Mover {
       new utils.Coordinate(0, 0),
     );
 
-    this.unpatchWorkspace(workspace);
     this.unpatchDragStrategy(info.block);
     this.moves.delete(workspace);
     workspace.setKeyboardMoveInProgress(false);
@@ -202,7 +200,6 @@ export class Mover {
       if (newNode) workspace.getCursor()?.setCurNode(newNode);
     }
 
-    this.unpatchWorkspace(workspace);
     this.unpatchDragStrategy(info.block);
     this.moves.delete(workspace);
     workspace.setKeyboardMoveInProgress(false);
@@ -256,45 +253,6 @@ export class Mover {
     return true;
   }
 
-  /**
-   * Monkeypatch over workspace functions to consider keyboard drags as
-   * well as mouse/pointer drags.
-   *
-   * @param workspace The workspace to patch.
-   */
-  private patchWorkspace(workspace: WorkspaceSvg) {
-    // Keyboard drags are real drags.
-    this.oldIsDragging = workspace.isDragging;
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    (workspace as any).isDragging = () => this.isMoving(workspace);
-
-    // Ignore mouse/pointer events during keyboard drags.
-    this.oldGetGesture = workspace.getGesture;
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    (workspace as any).getGesture = (e: PointerEvent) => {
-      // Normally these would be called from Gesture.doStart.
-      e.preventDefault();
-      e.stopPropagation();
-      return null;
-    };
-  }
-
-  /**
-   * Remove monkeypatches on the workspace.
-   *
-   * @param workspace The workspace to unpatch.
-   */
-  private unpatchWorkspace(workspace: WorkspaceSvg) {
-    if (this.oldIsDragging) {
-      // eslint-disable-next-line @typescript-eslint/no-explicit-any
-      (workspace as any).isDragging = this.oldIsDragging;
-    }
-    if (this.oldGetGesture) {
-      // eslint-disable-next-line @typescript-eslint/no-explicit-any
-      (workspace as any).getGesture = this.oldGetGesture;
-    }
-  }
-
   /**
    * Monkeypatch: replace the block's drag strategy and cache the old value.
    *