fixed keyboard delete of ink control points to never delete the stroke.

1 files changed, 9 insertions, 2 deletions

diff --git a/src/client/views/InkControlPtHandles.tsx b/src/client/views/InkControlPtHandles.tsx
index 94c238b2b..0644488b3 100644
--- a/src/client/views/InkControlPtHandles.tsx
+++ b/src/client/views/InkControlPtHandles.tsx
@@ -28,6 +28,13 @@ export class InkControlPtHandles extends React.Component<InkControlProps> {
     @observable private _overControl = -1;
 
     @observable controlUndo: UndoManager.Batch | undefined;
+
+    componentDidMount() {
+        document.addEventListener("keydown", this.onDelete, true);
+    }
+    componentWillUnmount() {
+        document.removeEventListener("keydown", this.onDelete, true);
+    }
     /**
      * Handles the movement of a selected control point when the user clicks and drags.
      * @param controlIndex The index of the currently selected control point.
@@ -91,7 +98,8 @@ export class InkControlPtHandles extends React.Component<InkControlProps> {
     @action
     onDelete = (e: KeyboardEvent) => {
         if (["-", "Backspace", "Delete"].includes(e.key)) {
-            if (InkStrokeProperties.Instance?.deletePoints()) e.stopPropagation();
+            InkStrokeProperties.Instance?.deletePoints();
+            e.stopPropagation();
         }
     }
 
@@ -102,7 +110,6 @@ export class InkControlPtHandles extends React.Component<InkControlProps> {
     changeCurrPoint = (i: number) => {
         if (InkStrokeProperties.Instance) {
             InkStrokeProperties.Instance._currentPoint = i;
-            document.addEventListener("keydown", this.onDelete, true);
         }
     }