From: nds <natalia.donis@opencascade.com>
Date: Mon, 13 Apr 2015 08:36:17 +0000 (+0300)
Subject: The face filter is not necessary on sketch label activation if there is a selected... 
X-Git-Tag: V_1.1.0~41^2
X-Git-Url: http://git.salome-platform.org/gitweb/?a=commitdiff_plain;h=488ec1949fc439f8d77960cdb3b4682ddc05adda;p=modules%2Fshaper.git

The face filter is not necessary on sketch label activation if there is a selected plane.
---

diff --git a/src/PartSet/PartSet_WidgetSketchLabel.cpp b/src/PartSet/PartSet_WidgetSketchLabel.cpp
index 676094940..2bf6d00aa 100644
--- a/src/PartSet/PartSet_WidgetSketchLabel.cpp
+++ b/src/PartSet/PartSet_WidgetSketchLabel.cpp
@@ -237,6 +237,9 @@ bool PartSet_WidgetSketchLabel::setSelectionCustom(const ModuleBase_ViewerPrs& t
 void PartSet_WidgetSketchLabel::activateCustom()
 {
   std::shared_ptr<GeomAPI_Pln> aPlane = plane();
+  if (aPlane.get())
+    return;
+
   bool aBodyIsVisualized = false;
   XGUI_Displayer* aDisp = myWorkshop->displayer();
   QObjectPtrList aDisplayed = aDisp->displayedObjects();
@@ -249,14 +252,7 @@ void PartSet_WidgetSketchLabel::activateCustom()
     }
   }
 
-  if (aPlane || aBodyIsVisualized) {
-    //setSketchingMode();
-    // In order to avoid Opening/Closing of context too often
-    // it can be useful for a delay on the property panel filling
-    // it is possible that it is not necessary anymore, but it requires a check
-    //mySelectionTimer->start(20);
-    //setSketchingMode();
-  } else {
+  if (!aBodyIsVisualized) {
     // We have to select a plane before any operation
     showPreviewPlanes();
   }