diff --git a/Code/Mantid/Vates/ParaviewPlugins/ParaViewWidgets/QtWidgets/DimensionWidget.cpp b/Code/Mantid/Vates/ParaviewPlugins/ParaViewWidgets/QtWidgets/DimensionWidget.cpp index e89d05b4ce5fe3c9d550fbb582c977a83a4da4a3..0d2b755ed562ed6cf1121a5d6fabe488c19d1e36 100644 --- a/Code/Mantid/Vates/ParaviewPlugins/ParaViewWidgets/QtWidgets/DimensionWidget.cpp +++ b/Code/Mantid/Vates/ParaviewPlugins/ParaViewWidgets/QtWidgets/DimensionWidget.cpp @@ -23,7 +23,7 @@ Constructor */ DimensionWidget::DimensionWidget(Mantid::VATES::BinDisplay binDisplay) { - //TODO: The following cascade can be refactored out. + //Prefer dependency injection to the following conditional statements. if(binDisplay == Mantid::VATES::Simple) { m_binWidget = new SimpleBinInputWidget; diff --git a/Code/Mantid/Vates/ParaviewPlugins/ParaViewWidgets/QtWidgets/LowHighStepInputWidget.cpp b/Code/Mantid/Vates/ParaviewPlugins/ParaViewWidgets/QtWidgets/LowHighStepInputWidget.cpp index 0af1f6ae27783edeb80ef61fe9c347ad3b778412..38d995ed1ef8d23da68e65a6a405cfa522edcea4 100644 --- a/Code/Mantid/Vates/ParaviewPlugins/ParaViewWidgets/QtWidgets/LowHighStepInputWidget.cpp +++ b/Code/Mantid/Vates/ParaviewPlugins/ParaViewWidgets/QtWidgets/LowHighStepInputWidget.cpp @@ -37,8 +37,7 @@ LowHighStepInputWidget::LowHighStepInputWidget() connect(m_step, SIGNAL(editingFinished()), this, SLOT(nBinsListener())); - QDoubleValidator* stepValidator = new QDoubleValidator; - stepValidator->setBottom(0); + QDoubleValidator* stepValidator = new QDoubleValidator(0, 10000, 5, this); m_step->setValidator(stepValidator); QHBoxLayout* layout = new QHBoxLayout;