Make the backspace key remove text directly instead of sending a DEL …

…keycode (#1162)
dessalines · Nov 29, 2024 · 9c08def · 9c08def
1 parent ac514da
commit 9c08def
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 8 deletions.
diff --git a/app/src/main/java/com/dessalines/thumbkey/keyboards/CommonKeys.kt b/app/src/main/java/com/dessalines/thumbkey/keyboards/CommonKeys.kt
@@ -144,14 +144,7 @@ val BACKSPACE_KEY_ITEM =
         center =
             KeyC(
                 display = KeyDisplay.IconDisplay(Icons.AutoMirrored.Outlined.KeyboardBackspace),
-                action =
-                    SendEvent(
-                        KeyEvent(
-                            KeyEvent.ACTION_DOWN,
-                            KeyEvent
-                                .KEYCODE_DEL,
-                        ),
-                    ),
+                action = DeleteKeyAction,
                 size = LARGE,
                 color = SECONDARY,
             ),

diff --git a/app/src/main/java/com/dessalines/thumbkey/utils/Types.kt b/app/src/main/java/com/dessalines/thumbkey/utils/Types.kt
@@ -164,6 +164,8 @@ sealed class KeyAction {
         data object CycleRight : KeyAction()
     }
 
+    data object DeleteKeyAction : KeyAction()
+
     data object DeleteWordBeforeCursor : KeyAction()
 
     data object DeleteWordAfterCursor : KeyAction()

diff --git a/app/src/main/java/com/dessalines/thumbkey/utils/Utils.kt b/app/src/main/java/com/dessalines/thumbkey/utils/Utils.kt
@@ -361,6 +361,14 @@ fun performKeyAction(
             ime.currentInputConnection.sendKeyEvent(ev)
         }
 
+        is KeyAction.DeleteKeyAction -> {
+            if (ime.currentInputConnection.getSelectedText(0)?.isEmpty() != false) {
+                ime.currentInputConnection.deleteSurroundingText(1, 0)
+            } else {
+                ime.currentInputConnection.commitText("", 0)
+            }
+        }
+
         is KeyAction.DeleteWordBeforeCursor -> {
             Log.d(TAG, "deleting last word")
             deleteWordBeforeCursor(ime)