Wouldn't it be better and more intuitive for the user if that content was displayed but with an error message?logari81 wrote: the Constraints list detects a mismatch to the last accepted geometry and hides all its content.
Wouldn't it be better and more intuitive for the user if that content was displayed but with an error message?logari81 wrote: the Constraints list detects a mismatch to the last accepted geometry and hides all its content.
Code: Select all
diff --git a/src/Mod/Sketcher/App/SketchObject.cpp b/src/Mod/Sketcher/App/SketchObject.cpp
index 0626294..86bf07a 100644
--- a/src/Mod/Sketcher/App/SketchObject.cpp
+++ b/src/Mod/Sketcher/App/SketchObject.cpp
@@ -1585,10 +1585,9 @@ void SketchObject::onChanged(const App::Property* prop)
// make sure not to change anything while restoring this object
if (!isRestoring()) {
// if support face was cleared then also clear the external geometry
- if (!Support.getValue()) {
- std::vector<DocumentObject*> obj;
- std::vector<std::string> sub;
- ExternalGeometry.setValues(obj, sub);
+ delConstraintsToExternal();
+ for (int i=0; i < getExternalGeometryCount(); i++) {
+ delExternal(0);
}
}
}