From 5925b1821a5706d59d1504ab3b426ce5df6aff52 Mon Sep 17 00:00:00 2001 From: Richard Antalik Date: Tue, 8 Nov 2022 19:10:26 +0100 Subject: Fix T102328: Incorrect strip state after copying Effect strip start position was translated twice. This is caused by recent refactoring, see 7afcfe111aacc8bc. Don't change `seq->start` of attached effect strips during translation and only rely on `seq_time_update_effects_strip_range` function. --- source/blender/sequencer/intern/strip_transform.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/source/blender/sequencer/intern/strip_transform.c b/source/blender/sequencer/intern/strip_transform.c index 2c7bb69da66..4296a8ea35c 100644 --- a/source/blender/sequencer/intern/strip_transform.c +++ b/source/blender/sequencer/intern/strip_transform.c @@ -145,7 +145,7 @@ void SEQ_transform_translate_sequence(Scene *evil_scene, Sequence *seq, int delt /* Move meta start/end points. */ seq_time_translate_handles(evil_scene, seq, delta); } - else { /* All other strip types. */ + else if (seq->seq1 == NULL && seq->seq2 == NULL) { /* All other strip types. */ seq->start += delta; /* Only to make files usable in older versions. */ seq->startdisp = SEQ_time_left_handle_frame_get(evil_scene, seq); -- cgit v1.2.3