Browse Source

Consider chained operation when selecting text in TextEdit::undo()

Haoyu Qiu 3 years ago
parent
commit
8cefbfcf43
1 changed files with 4 additions and 3 deletions
  1. 4 3
      scene/gui/text_edit.cpp

+ 4 - 3
scene/gui/text_edit.cpp

@@ -3486,9 +3486,6 @@ void TextEdit::undo() {
 
 	TextOperation op = undo_stack_pos->get();
 	_do_text_op(op, true);
-	if (op.type != TextOperation::TYPE_INSERT && (op.from_line != op.to_line || op.to_column != op.from_column + 1)) {
-		select(op.from_line, op.from_column, op.to_line, op.to_column);
-	}
 
 	current_op.version = op.prev_version;
 	if (undo_stack_pos->get().chain_backward) {
@@ -3504,6 +3501,10 @@ void TextEdit::undo() {
 		}
 	}
 
+	if (op.type != TextOperation::TYPE_INSERT && (op.from_line != op.to_line || op.to_column != op.from_column + 1)) {
+		select(op.from_line, op.from_column, op.to_line, op.to_column);
+	}
+
 	_update_scrollbars();
 	if (undo_stack_pos->get().type == TextOperation::TYPE_REMOVE) {
 		set_caret_line(undo_stack_pos->get().to_line, false);