diff --git a/src/Mod/PartDesign/Gui/TaskGrooveParameters.ui b/src/Mod/PartDesign/Gui/TaskGrooveParameters.ui
index a65fb9ba8..3d9e79ed2 100644
--- a/src/Mod/PartDesign/Gui/TaskGrooveParameters.ui
+++ b/src/Mod/PartDesign/Gui/TaskGrooveParameters.ui
@@ -65,9 +65,6 @@
360.000000000000000
-
- 1
-
diff --git a/src/Mod/PartDesign/Gui/TaskRevolutionParameters.cpp b/src/Mod/PartDesign/Gui/TaskRevolutionParameters.cpp
index 5a97bad59..68f35c428 100644
--- a/src/Mod/PartDesign/Gui/TaskRevolutionParameters.cpp
+++ b/src/Mod/PartDesign/Gui/TaskRevolutionParameters.cpp
@@ -57,7 +57,7 @@ TaskRevolutionParameters::TaskRevolutionParameters(ViewProviderRevolution *Revol
ui->setupUi(proxy);
QMetaObject::connectSlotsByName(this);
- connect(ui->doubleSpinBox, SIGNAL(valueChanged(double)),
+ connect(ui->revolveAngle, SIGNAL(valueChanged(double)),
this, SLOT(onAngleChanged(double)));
connect(ui->axis, SIGNAL(activated(int)),
this, SLOT(onAxisChanged(int)));
@@ -71,7 +71,7 @@ TaskRevolutionParameters::TaskRevolutionParameters(ViewProviderRevolution *Revol
this->groupLayout()->addWidget(proxy);
// Temporarily prevent unnecessary feature recomputes
- ui->doubleSpinBox->blockSignals(true);
+ ui->revolveAngle->blockSignals(true);
ui->axis->blockSignals(true);
ui->checkBoxMidplane->blockSignals(true);
ui->checkBoxReversed->blockSignals(true);
@@ -81,8 +81,7 @@ TaskRevolutionParameters::TaskRevolutionParameters(ViewProviderRevolution *Revol
bool mirrored = pcRevolution->Midplane.getValue();
bool reversed = pcRevolution->Reversed.getValue();
- ui->doubleSpinBox->setDecimals(Base::UnitsApi::getDecimals());
- ui->doubleSpinBox->setValue(l);
+ ui->revolveAngle->setValue(l);
int count=pcRevolution->getSketchAxisCount();
@@ -115,7 +114,7 @@ TaskRevolutionParameters::TaskRevolutionParameters(ViewProviderRevolution *Revol
ui->checkBoxMidplane->setChecked(mirrored);
ui->checkBoxReversed->setChecked(reversed);
- ui->doubleSpinBox->blockSignals(false);
+ ui->revolveAngle->blockSignals(false);
ui->axis->blockSignals(false);
ui->checkBoxMidplane->blockSignals(false);
ui->checkBoxReversed->blockSignals(false);
@@ -188,14 +187,14 @@ void TaskRevolutionParameters::onReversed(bool on)
void TaskRevolutionParameters::onUpdateView(bool on)
{
if (on) {
- PartDesign::Revolution* pcRevolution = static_cast(RevolutionView->getObject());
+ PartDesign::Revolution* pcRevolution = static_cast(RevolutionView->getObject());
pcRevolution->getDocument()->recomputeFeature(pcRevolution);
}
}
double TaskRevolutionParameters::getAngle(void) const
{
- return ui->doubleSpinBox->value();
+ return ui->revolveAngle->value().getValue();
}
QString TaskRevolutionParameters::getReferenceAxis(void) const
@@ -275,7 +274,11 @@ TaskDlgRevolutionParameters::~TaskDlgRevolutionParameters()
void TaskDlgRevolutionParameters::open()
{
-
+ // a transaction is already open at creation time of the pad
+ if (!Gui::Command::hasPendingCommand()) {
+ QString msg = QObject::tr("Edit revolve");
+ Gui::Command::openCommand((const char*)msg.toUtf8());
+ }
}
void TaskDlgRevolutionParameters::clicked(int)
diff --git a/src/Mod/PartDesign/Gui/TaskRevolutionParameters.ui b/src/Mod/PartDesign/Gui/TaskRevolutionParameters.ui
index bef4562c9..3411387ca 100644
--- a/src/Mod/PartDesign/Gui/TaskRevolutionParameters.ui
+++ b/src/Mod/PartDesign/Gui/TaskRevolutionParameters.ui
@@ -49,9 +49,9 @@
-
-
-
- 1
+
+
+ deg
0.000000000000000
@@ -105,6 +105,13 @@
+
+
+ Gui::QuantitySpinBox
+ QWidget
+
+
+