From 706bd14880cb79ea770c54b576484cf6d3fa72f8 Mon Sep 17 00:00:00 2001
From: mzernova <mzernova@opencascade.com>
Date: Fri, 31 Jan 2025 23:19:19 +0000
Subject: [PATCH] Fix sensitivity

---
 src/AIS/AIS_Manipulator.cxx     | 13 ++++++++++---
 src/AIS/AIS_Manipulator.hxx     |  5 +----
 src/OpenGl/OpenGl_Structure.cxx | 10 ++++++++--
 tests/vselect/bugs/bug32750     |  4 ++--
 4 files changed, 21 insertions(+), 11 deletions(-)

diff --git a/src/AIS/AIS_Manipulator.cxx b/src/AIS/AIS_Manipulator.cxx
index 604f0eb73a..e839f744a1 100644
--- a/src/AIS/AIS_Manipulator.cxx
+++ b/src/AIS/AIS_Manipulator.cxx
@@ -646,6 +646,13 @@ Standard_Boolean AIS_Manipulator::ObjectTransformation(const Standard_Integer  t
         mySectorGroup->SetTransformation(aTrsf);
       }
 
+      // Change value of an angle if it should have different sign.
+      if (anAngle * myPrevState < 0 && Abs(anAngle) < M_PI_2)
+      {
+        Standard_Real aSign = myPrevState > 0 ? -1.0 : 1.0;
+        anAngle             = aSign * (M_PI * 2 - anAngle);
+      }
+
       gp_Trsf aNewTrsf;
       aNewTrsf.SetRotation(aCurrAxis, anAngle);
       theTrsf *= aNewTrsf;
@@ -720,14 +727,14 @@ Standard_Boolean AIS_Manipulator::ProcessDragging(const Handle(AIS_InteractiveCo
       return Standard_True;
     }
     case AIS_DragAction_Stop: {
-      // at the end of transformation redisplay for updating sensitive areas
       StopTransform(true);
       if (mySkinMode == ManipulatorSkin_Flat)
       {
         mySectorGroup->Clear();
       }
-      if (aCtx->IsDisplayed(this))
+      else if (aCtx->IsDisplayed(this))
       {
+        // at the end of transformation redisplay for updating sensitive areas
         aCtx->Redisplay(this, true);
       }
       return Standard_True;
@@ -1504,7 +1511,7 @@ void AIS_Manipulator::ComputeSelection(const Handle(SelectMgr_Selection)& theSel
           // define sensitivity by point
           Handle(Select3D_SensitivePoint) aPnt =
             new Select3D_SensitivePoint(anOwner, myAxes[anIt].ScalerCubePosition());
-          aPnt->SetSensitivityFactor(15);
+          aPnt->SetSensitivityFactor(aHighSensitivity);
           theSelection->Add(aPnt);
         }
         // enlarge sensitivity by triangulation
diff --git a/src/AIS/AIS_Manipulator.hxx b/src/AIS/AIS_Manipulator.hxx
index 7b9530b4a5..925d67b07c 100644
--- a/src/AIS/AIS_Manipulator.hxx
+++ b/src/AIS/AIS_Manipulator.hxx
@@ -688,10 +688,7 @@ protected: //! @name Auxiliary classes to fill presentation with proper primitiv
 
     void SetIndent(const Standard_ShortReal theValue) { myIndent = theValue; }
 
-    Standard_ShortReal Size() const
-    {
-      return myLength + myBoxSize + myDiskThickness + myIndent * 2.0f;
-    }
+    Standard_ShortReal Size() const { return myInnerRadius + myDiskThickness + myIndent * 2; }
 
     Standard_ShortReal InnerRadius() const { return myInnerRadius + myIndent * 2.0f; }
 
diff --git a/src/OpenGl/OpenGl_Structure.cxx b/src/OpenGl/OpenGl_Structure.cxx
index f0a5957e45..2508964081 100644
--- a/src/OpenGl/OpenGl_Structure.cxx
+++ b/src/OpenGl/OpenGl_Structure.cxx
@@ -407,7 +407,10 @@ void OpenGl_Structure::renderGeometry(const Handle(OpenGl_Workspace)& theWorkspa
     const OpenGl_Group* aGroup = aGroupIter.Value();
 
     const gp_Trsf& aTrsf = aGroup->Transformation();
-    applyTransformation(aCtx, aTrsf, Standard_True);
+    if (aTrsf.Form() != gp_Identity)
+    {
+      applyTransformation(aCtx, aTrsf, Standard_True);
+    }
 
     const Handle(Graphic3d_TransformPers)& aTrsfPers = aGroup->TransformPersistence();
     if (!aTrsfPers.IsNull())
@@ -425,7 +428,10 @@ void OpenGl_Structure::renderGeometry(const Handle(OpenGl_Workspace)& theWorkspa
       aCtx->ApplyModelViewMatrix();
     }
 
-    applyTransformation(aCtx, aTrsf, Standard_False);
+    if (aTrsf.Form() != gp_Identity)
+    {
+      applyTransformation(aCtx, aTrsf, Standard_False);
+    }
   }
 }
 
diff --git a/tests/vselect/bugs/bug32750 b/tests/vselect/bugs/bug32750
index 36cf6dbdc5..a982d6bd2d 100644
--- a/tests/vselect/bugs/bug32750
+++ b/tests/vselect/bugs/bug32750
@@ -18,8 +18,8 @@ vzoom 0.5
 vmanipulator m -attach b
 
 #set mouse coordinates for actions
-set mouse_pick {205 155}
-set mouse_drag1 {205 55}
+set mouse_pick {206 155}
+set mouse_drag1 {206 55}
 set mouse_pick2 {250 10}
 set mouse_pick3 {250 200}