From 0f7c4d8375e5066500ad9a44402adbb197d1c1a2 Mon Sep 17 00:00:00 2001 From: ImpishOaths Date: Sun, 7 Jul 2024 23:40:34 -0700 Subject: [PATCH] Don't lose in/out handles when bezier keyframe inserted on top of existing keyframe Apply suggestions from code review Co-authored-by: Silc Lizard (Tokage) Renew --- editor/animation_track_editor.cpp | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/editor/animation_track_editor.cpp b/editor/animation_track_editor.cpp index 6810b802a1c..e84ef53f2b6 100644 --- a/editor/animation_track_editor.cpp +++ b/editor/animation_track_editor.cpp @@ -4464,7 +4464,14 @@ AnimationTrackEditor::TrackIndices AnimationTrackEditor::_confirm_insert(InsertD } break; case Animation::TYPE_BEZIER: { - value = animation->make_default_bezier_key(p_id.value); + int existing = animation->track_find_key(p_id.track_idx, time, Animation::FIND_MODE_APPROX); + if (existing != -1) { + Array arr = animation->track_get_key_value(p_id.track_idx, existing); + arr[0] = p_id.value; + value = arr; + } else { + value = animation->make_default_bezier_key(p_id.value); + } bezier_edit_icon->set_disabled(false); } break; default: {