seedlingattempt/godot
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix for stopping the Undo History being desynchronised from actual Undo queue.

Browse Source
This commit is contained in:
Sofox
2023-11-06 23:55:14 +00:00
parent 3e7f638d7b
commit 662522ae5a
3 changed files with 9 additions and 31 deletions
Download Patch File Download Diff File Expand all files Collapse all files

34
editor/editor_undo_redo_manager.cpp
View File

@ -239,6 +239,7 @@ void EditorUndoRedoManager::commit_action(bool p_execute) {
is_committing = true;
History &history = get_or_create_history(pending_action.history_id);
bool merging = history.undo_redo->is_merging();
history.undo_redo->commit_action(p_execute);
history.redo_stack.clear();
@ -248,39 +249,10 @@ void EditorUndoRedoManager::commit_action(bool p_execute) {
return;
}
if (!history.undo_stack.is_empty()) {
// Discard action if it should be merged (UndoRedo handles merging internally).
switch (pending_action.merge_mode) {
case UndoRedo::MERGE_DISABLE:
break; // Nothing to do here.
case UndoRedo::MERGE_ENDS: {
if (history.undo_stack.size() < 2) {
break;
}
const Action &prev_action = history.undo_stack.back()->get();
const Action &pre_prev_action = history.undo_stack.back()->prev()->get();
if (pending_action.merge_mode == prev_action.merge_mode && pending_action.merge_mode == pre_prev_action.merge_mode &&
pending_action.action_name == prev_action.action_name && pending_action.action_name == pre_prev_action.action_name) {
pending_action = Action();
is_committing = false;
emit_signal(SNAME("history_changed"));
return;
}
} break;
case UndoRedo::MERGE_ALL: {
const Action &prev_action = history.undo_stack.back()->get();
if (pending_action.merge_mode == prev_action.merge_mode && pending_action.action_name == prev_action.action_name) {
pending_action = Action();
is_committing = false;
emit_signal(SNAME("history_changed"));
return;
}
} break;
}
if (!merging) {
history.undo_stack.push_back(pending_action);
}
history.undo_stack.push_back(pending_action);
pending_action = Action();
is_committing = false;
emit_signal(SNAME("history_changed"));