kindred/create
1
1
Fork 0
Code Issues 53 Pull Requests Actions Packages Projects 9 Releases 2 Wiki Activity

[PD] TaskExtrudeParameters: simplify code

Browse Source 
to avoid several time the same translatable string
This commit is contained in:
Uwe
2022-07-20 03:16:56 +02:00
parent 0e22ada9dd
commit 98dae1b217
4 changed files with 15 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
src/Mod/PartDesign/Gui/TaskExtrudeParameters.cpp
View File

@@ -50,7 +50,7 @@ TaskExtrudeParameters::TaskExtrudeParameters(ViewProviderSketchBased *SketchBase
// we need a separate container widget to add all controls to
proxy = new QWidget(this);
ui->setupUi(proxy);
ui->lineFaceName->setPlaceholderText(tr("No face selected"));
handleLineFaceNameNo();
Gui::ButtonGroup* group = new Gui::ButtonGroup(this);
group->addButton(ui->checkBoxMidplane);
@@ -683,10 +683,10 @@ void TaskExtrudeParameters::getReferenceAxis(App::DocumentObject*& obj, std::vec
void TaskExtrudeParameters::onButtonFace(const bool checked)
{
if (!checked && ui->lineFaceName->text().isEmpty())
ui->lineFaceName->setPlaceholderText(tr("No face selected"));
else if (checked && ui->lineFaceName->text().isEmpty())
handleLineFaceName(); // sets placeholder text
if (!checked)
handleLineFaceNameNo();
else
handleLineFaceNameClick(); // sets placeholder text
// to distinguish that this is the direction selection
selectionFace = true;
@@ -892,9 +892,14 @@ void TaskExtrudeParameters::translateModeList(int)
// implement in sub-class
}
void TaskExtrudeParameters::handleLineFaceName(void)
void TaskExtrudeParameters::handleLineFaceNameClick(void)
{
ui->lineFaceName->setPlaceholderText(tr("Click on a face in the model"));
}
void TaskExtrudeParameters::handleLineFaceNameNo(void)
{
ui->lineFaceName->setPlaceholderText(tr("No face selected"));
}
#include "moc_TaskExtrudeParameters.cpp"

3
src/Mod/PartDesign/Gui/TaskExtrudeParameters.h
View File

@@ -118,7 +118,8 @@ protected:
virtual void updateUI(int index);
void updateDirectionEdits(void);
void setDirectionMode(int index);
void handleLineFaceName(void);
void handleLineFaceNameClick(void);
void handleLineFaceNameNo(void);
private:
void tryRecomputeFeature();

2
src/Mod/PartDesign/Gui/TaskPadParameters.cpp
View File

@@ -106,7 +106,7 @@ void TaskPadParameters::onModeChanged(int index)
pcPad->Type.setValue("UpToFace");
if (ui->lineFaceName->text().isEmpty()) {
ui->buttonFace->setChecked(true);
handleLineFaceName(); // sets placeholder text
handleLineFaceNameClick(); // sets placeholder text
}
break;
case Modes::TwoDimensions:

2
src/Mod/PartDesign/Gui/TaskPocketParameters.cpp
View File

@@ -116,7 +116,7 @@ void TaskPocketParameters::onModeChanged(int index)
ui->lengthEdit->setValue(0.0);
if (ui->lineFaceName->text().isEmpty()) {
ui->buttonFace->setChecked(true);
handleLineFaceName(); // sets placeholder text
handleLineFaceNameClick(); // sets placeholder text
}
break;
case Modes::TwoDimensions: