highlighting of point constraints

src/Mod/Surface/Gui/TaskFilling.cpp

@@ -421,9 +421,6 @@ bool FillingPanel::accept()
                                    editedObject->InitialFace.getSubValues()));
     this->vp->highlightReferences(ViewProviderFilling::Face, links, false);
 
-    Gui::Command::commitCommand();
-    Gui::Command::doCommand(Gui::Command::Gui,"Gui.ActiveDocument.resetEdit()");
-    Gui::Command::updateActive();
     return true;
 }
 
@@ -441,9 +438,6 @@ bool FillingPanel::reject()
     selectionMode = None;
     Gui::Selection().rmvSelectionGate();
 
-    Gui::Command::abortCommand();
-    Gui::Command::doCommand(Gui::Command::Gui,"Gui.ActiveDocument.resetEdit()");
-    Gui::Command::updateActive();
     return true;
 }
 
@@ -828,12 +822,28 @@ void TaskFilling::open()
 
 bool TaskFilling::accept()
 {
-    return widget1->accept();
+    bool ok = widget1->accept();
+    if (ok) {
+        widget2->reject();
+        Gui::Command::commitCommand();
+        Gui::Command::doCommand(Gui::Command::Gui,"Gui.ActiveDocument.resetEdit()");
+        Gui::Command::updateActive();
+    }
+
+    return ok;
 }
 
 bool TaskFilling::reject()
 {
-    return widget1->reject();
+    bool ok = widget1->reject();
+    if (ok) {
+        widget2->reject();
+        Gui::Command::abortCommand();
+        Gui::Command::doCommand(Gui::Command::Gui,"Gui.ActiveDocument.resetEdit()");
+        Gui::Command::updateActive();
+    }
+
+    return ok;
 }
 
 }

src/Mod/Surface/Gui/TaskFillingVertex.cpp

@@ -189,6 +189,12 @@ void FillingVertexPanel::open()
     Gui::Selection().clearSelection();
 }
 
+void FillingVertexPanel::reject()
+{
+    this->vp->highlightReferences(ViewProviderFilling::Vertex,
+        editedObject->Points.getSubListValues(), false);
+}
+
 void FillingVertexPanel::clearSelection()
 {
     Gui::Selection().clearSelection();
@@ -290,7 +296,7 @@ void FillingVertexPanel::onSelectionChanged(const Gui::SelectionChanges& msg)
                 }
             }
             this->vp->highlightReferences(ViewProviderFilling::Vertex,
-                editedObject->Points.getSubListValues(), false);
+                editedObject->Points.getSubListValues(), true);
         }
 
         editedObject->recomputeFeature();

src/Mod/Surface/Gui/TaskFillingVertex.h

@@ -61,6 +61,7 @@ public:
     ~FillingVertexPanel();
 
     void open();
+    void reject();
     void checkOpenCommand();
     void setEditedObject(Surface::Filling* obj);
 

src/Mod/Surface/Gui/TaskFillingVertex.ui

@@ -36,9 +36,6 @@
           <property name="text">
            <string>Add Vertex</string>
           </property>
-          <property name="checkable">
-           <bool>true</bool>
-          </property>
           <property name="checked">
            <bool>false</bool>
           </property>
@@ -55,9 +52,6 @@
           <property name="text">
            <string>Remove Vertex</string>
           </property>
-          <property name="checkable">
-           <bool>true</bool>
-          </property>
          </widget>
         </item>
        </layout>