delete selected text in backspace key binding (fixes #373)

Akuli · Feb 22, 2021 · 7e36750 · 7e36750
1 parent 168eadf
commit 7e36750
Showing 1 changed file with 12 additions and 11 deletions.
diff --git a/porcupine/default_keybindings.tcl b/porcupine/default_keybindings.tcl
@@ -97,15 +97,17 @@ bind Text <$contmand-Delete> {
 }
 
 bind Text <BackSpace> {
-    set beforecursor [%W get {insert linestart} insert]
-    if {
-        [string length [bind %W <<Dedent>>]] != 0 &&
-        [string length $beforecursor] != 0 &&
-        [string is space $beforecursor]
-    } {
-        event generate %W <<Dedent>>
-    } else {
-        %W delete {insert - 1 char}
+    try {%W delete sel.first sel.last} on error {} {
+        set beforecursor [%W get {insert linestart} insert]
+        if {
+            [string length [bind %W <<Dedent>>]] != 0 &&
+            [string length $beforecursor] != 0 &&
+            [string is space $beforecursor]
+        } {
+            event generate %W <<Dedent>>
+        } else {
+            %W delete {insert - 1 char}
+        }
     }
 }
 
@@ -137,8 +139,7 @@ bind Text <Shift-$contmand-BackSpace> {
     }
 }
 
-# When pasting, delete what was selected
-# Here + adds to end of existing binding
+# When pasting, delete what was selected. Here + adds to end of existing binding.
 bind Text <<Paste>> {+
     catch {%W delete sel.first sel.last}
 }