Use Content Model to handle Delete/Backspace key in more cases (#2162)

microsoft · Oct 24, 2023 · bc809e7 · bc809e7
1 parent 1ffe2e3
commit bc809e7
Showing 1 changed file with 2 additions and 6 deletions.
diff --git a/...ages-content-model/roosterjs-content-model-editor/lib/publicApi/editing/keyboardDelete.ts b/...ages-content-model/roosterjs-content-model-editor/lib/publicApi/editing/keyboardDelete.ts
@@ -89,16 +89,12 @@ function shouldDeleteWithContentModel(range: Range | null, rawEvent: KeyboardEve
 }
 
 function canDeleteBefore(rawEvent: KeyboardEvent, range: Range) {
-    return (
-        rawEvent.key == 'Backspace' &&
-        (range.startOffset > 1 || range.startContainer.previousSibling)
-    );
+    return rawEvent.key == 'Backspace' && range.startOffset > 1;
 }
 
 function canDeleteAfter(rawEvent: KeyboardEvent, range: Range) {
     return (
         rawEvent.key == 'Delete' &&
-        (range.startOffset < (range.startContainer.nodeValue?.length ?? 0) - 1 ||
-            range.startContainer.nextSibling)
+        range.startOffset < (range.startContainer.nodeValue?.length ?? 0) - 1
     );
 }