Fix behaviour of double inputs when pressing enter

Author: stfnp

gui/source/pre/views/ArrowMassView.cpp

@@ -90,7 +90,7 @@ ArrowMassView::ArrowMassView(QAbstractItemModel* model, QPersistentModelIndex in
 
     // Keep model up to date on changes
     QObject::connect(selectionBox, &QComboBox::currentIndexChanged, this, updateModel);
-    QObject::connect(massEdit, &DoubleSpinBox::valueChanged, this, updateModel);
-    QObject::connect(massPerForceEdit, &DoubleSpinBox::valueChanged, this, updateModel);
-    QObject::connect(massPerEnergyEdit, &DoubleSpinBox::valueChanged, this, updateModel);
+    QObject::connect(massEdit, &DoubleSpinBox::contentModified, this, updateModel);
+    QObject::connect(massPerForceEdit, &DoubleSpinBox::contentModified, this, updateModel);
+    QObject::connect(massPerEnergyEdit, &DoubleSpinBox::contentModified, this, updateModel);
 }

gui/source/pre/views/DrawLengthView.cpp

@@ -86,5 +86,5 @@ DrawLengthView::DrawLengthView(QAbstractItemModel* model, QPersistentModelIndex
 
     // Keep model up to date on changes
     QObject::connect(selectionBox, &QComboBox::currentIndexChanged, this, updateModel);
-    QObject::connect(lengthEdit, &DoubleSpinBox::valueChanged, this, updateModel);
+    QObject::connect(lengthEdit, &DoubleSpinBox::contentModified, this, updateModel);
 }

gui/source/pre/widgets/DoubleSpinBox.cpp

@@ -19,7 +19,9 @@ DoubleSpinBox::DoubleSpinBox(const Quantity& quantity, const DoubleRange& range,
     // Prevent catching focus when scrolling, https://stackoverflow.com/a/19382766
     setFocusPolicy(Qt::StrongFocus);
 
-    QObject::connect(lineEdit(), &QLineEdit::editingFinished, this, &DoubleSpinBox::contentModified);    // Signal modification by user when the text was edited
+    // Signal modification by user when the text was edited
+    // QDoubleSpinBox::valueChanged is not used because it is triggered during typing, leading to unnecessary updates of the model.
+    QObject::connect(this, &QDoubleSpinBox::editingFinished, this, &DoubleSpinBox::contentModified);
     QObject::connect(&quantity, &Quantity::unitChanged, this, &DoubleSpinBox::updateUnit);
     updateUnit();
 }