Fix backspace deletion of selected items with change tracking

Move cursor at beginning of selection after deletion. Else backspace
operation gets stuck in a loop (tries to remove the selection again
and again).

This also fixes backspace deletion of insets with confirmDeletion() == true.

Fixes #11630

diff --git a/src/Text3.cpp b/src/Text3.cpp
index 664ffb459b8fbd0c9426908330ae36565795e3b8..8643411da1cf0d695fc5c04fd478ccc84afb6cfc 100644 (file)
--- a/src/Text3.cpp
+++ b/src/Text3.cpp
@@ -1250,7 +1250,13 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
                                }
                        }
                } else {
+                       DocIterator const dit = cur.selectionBegin();
                        cutSelection(cur, false);
+                       if (cur.buffer()->params().track_changes)
+                               // since we're doing backwards deletion,
+                               // and the selection is not really cut,
+                               // move cursor before selection (#11630)
+                               cur.setCursor(dit);
                        singleParUpdate = false;
                }
                break;