From 751b5f6c44f11f1ed102413586656a8a992dc72e Mon Sep 17 00:00:00 2001 From: itsmattkc <34096995+itsmattkc@users.noreply.github.com> Date: Tue, 21 Sep 2021 19:03:51 -0700 Subject: [PATCH] timeline: used footage params as basis for manual selection --- app/widget/timelinewidget/tool/import.cpp | 19 ++++++++++--------- 1 file changed, 10 insertions(+), 9 deletions(-) diff --git a/app/widget/timelinewidget/tool/import.cpp b/app/widget/timelinewidget/tool/import.cpp index d51060ed0..457dae3e2 100644 --- a/app/widget/timelinewidget/tool/import.cpp +++ b/app/widget/timelinewidget/tool/import.cpp @@ -326,19 +326,20 @@ void ImportTool::DropGhosts(bool insert) bool sequence_is_valid = true; - if (behavior == kDWSAuto) { + // Even if the user selected manual, set from footage anyway so the user has a useful + // starting point + QVector footage_only; - QVector footage_only; - - for (auto it=dragged_footage_.cbegin(); it!=dragged_footage_.cend(); it++) { - if (!footage_only.contains(it->first)) { - footage_only.append(it->first); - } + for (auto it=dragged_footage_.cbegin(); it!=dragged_footage_.cend(); it++) { + if (!footage_only.contains(it->first)) { + footage_only.append(it->first); } + } - new_sequence->set_parameters_from_footage(footage_only); + new_sequence->set_parameters_from_footage(footage_only); - } else { + // If the user selected manual, show them a dialog with parameters + if (behavior == kDWSManual) { SequenceDialog sd(new_sequence, SequenceDialog::kNew, parent()); sd.SetUndoable(false);