more simplification and more robust code

friction2d · Nov 12, 2024 · 8e99259 · 8e99259
1 parent c29542f
commit 8e99259
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 15 deletions.
diff --git a/src/ui/dialogs/scenesettingsdialog.cpp b/src/ui/dialogs/scenesettingsdialog.cpp
@@ -311,25 +311,18 @@ void SceneSettingsDialog::sNewSceneDialog(Document& document,
     dialog->show();
 }
 
-void SceneSettingsDialog::updateDuration() {
-    const QString typetime = mTypeTime->currentData().toString();
+void SceneSettingsDialog::updateDuration(int index)
+{
     const qreal fps = mFPSSpinBox->value();
-    int index = mTypeTime->currentIndex();
-
-    if (mMinFrameSpin->value() >= mMaxFrameSpin->value()) {
-        return;
-    }
-
     switch(index) {
         case 0: // Convert seconds to frames
-            mMinFrameSpin->setValue(mMinFrameSpin->value() * fps);
-            mMaxFrameSpin->setValue(mMaxFrameSpin->value() * fps);
+            mMinFrameSpin->setValue(qRound(mMinFrameSpin->value() * fps));
+            mMaxFrameSpin->setValue(qRound(mMaxFrameSpin->value() * fps));
             break;
         case 1: // Convert frames to seconds
-            mMinFrameSpin->setValue(mMinFrameSpin->value() / fps);
-            mMaxFrameSpin->setValue(mMaxFrameSpin->value() / fps);
+            mMinFrameSpin->setValue(qRound(mMinFrameSpin->value() / fps));
+            mMaxFrameSpin->setValue(qRound(mMaxFrameSpin->value() / fps));
             break;
-        default:
-            return;
+        default:;
     }
 }
diff --git a/src/ui/dialogs/scenesettingsdialog.h b/src/ui/dialogs/scenesettingsdialog.h
@@ -75,7 +75,7 @@ class UI_EXPORT SceneSettingsDialog : public QDialog
 
 private:
     bool validate();
-    void updateDuration();
+    void updateDuration(int index);
 
     Canvas * mTargetCanvas = nullptr;