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

PartDesign: Adopt primitive preview

Browse Source 
The preview was developed when the "one feature only" visibility rule was not yet introduced. This commit maks it work again.
This commit is contained in:
Stefan Tröger
2016-04-09 10:54:26 +02:00
parent 303578efc6
commit 97a51686c7
4 changed files with 25 additions and 18 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
src/Mod/PartDesign/Gui/TaskPrimitiveParameters.cpp
View File

@@ -621,10 +621,7 @@ TaskPrimitiveParameters::TaskPrimitiveParameters(ViewProviderPrimitive* Primitiv
//make sure the relevant things are visible
cs_visibility = vp->isVisible();
vp->Visibility.setValue(true);
if(prm->BaseFeature.getValue()) {
Gui::Application::Instance->activeDocument()->getViewProvider(prm->BaseFeature.getValue())->setVisible(true);
}
parameter = new TaskDatumParameters(vp);
Content.push_back(parameter);
@@ -659,12 +656,7 @@ bool TaskPrimitiveParameters::accept()
ViewProviderDatumCoordinateSystem* vp = static_cast<ViewProviderDatumCoordinateSystem*>(
Gui::Application::Instance->activeDocument()->getViewProvider(cs));
vp->setVisible(cs_visibility);
auto* prm = static_cast<PartDesign::FeaturePrimitive*>(vp_prm->getObject());
if(prm->BaseFeature.getValue()) {
Gui::Application::Instance->activeDocument()->getViewProvider(prm->BaseFeature.getValue())->setVisible(false);
}
connection.disconnect();
return true;
}
@@ -679,15 +671,9 @@ bool TaskPrimitiveParameters::reject() {
ViewProviderDatumCoordinateSystem* vp = static_cast<ViewProviderDatumCoordinateSystem*>(
Gui::Application::Instance->activeDocument()->getViewProvider(cs));
if(vp) {
if(vp)
vp->setVisible(cs_visibility);
auto* prm = static_cast<PartDesign::FeaturePrimitive*>(vp_prm->getObject());
if(prm->BaseFeature.getValue()) {
Gui::Application::Instance->activeDocument()->getViewProvider(prm->BaseFeature.getValue())->setVisible(false);
}
}
connection.disconnect();
return true;
}