Merge pull request #3502 from dubstar-04/fixes/op-visibility

[Path] - Don't reset the operations visibility state during document recompute
FreeCAD · May 25, 2020 · 388bfbd · 388bfbd
2 parents f1354fe + 760685e
commit 388bfbd
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/Mod/Path/PathCommands.py b/src/Mod/Path/PathCommands.py
@@ -150,7 +150,10 @@ def IsActive(self):
 
     def Activated(self):
         for sel in FreeCADGui.Selection.getSelectionEx():
-            PathScripts.PathDressup.baseOp(sel.Object).Active = not(PathScripts.PathDressup.baseOp(sel.Object).Active)
+            op = PathScripts.PathDressup.baseOp(sel.Object)
+            op.Active = not op.Active
+            op.ViewObject.Visibility = op.Active
+
         FreeCAD.ActiveDocument.recompute()
 
 

diff --git a/src/Mod/Path/PathScripts/PathOp.py b/src/Mod/Path/PathScripts/PathOp.py
@@ -472,9 +472,6 @@ def execute(self, obj):
         '''
         PathLog.track()
 
-        if obj.ViewObject:
-            obj.ViewObject.Visibility = obj.Active
-
         if not obj.Active:
             path = Path.Path("(inactive operation)")
             obj.Path = path