DiscussionsIssue archiveOCCT:Visualization

Archived issue #0033513

Visualization - Integration of the ability to zoom with vertical mouse movements.

Open CASCADEOCCT:Visualizationverified11 public notes

Search issues

Description

Currently zoom is only working when user moving mouse horizontally, vertical movements aren't affecting zoom functionality.

Steps to reproduce

Not required

Public activity

11 archived notes

Participants are labeled by their role within this record.

01Commenter 2
Branch [archived branch] has been created by Author.

[revision removed]


Detailed log of new commits:

Author: Author
Date: Tue Oct 24 14:20:10 2023 +0000

    0033513: Zooming - Only working when mouse move horizontally
02Commenter 3
For me it looks ok. Was added enum which allows to zoom in/ zoom out vertically.
03Commenter 4
Now awaiting for tests on Jenkins,
Will be approved after tests.
(now no remarks)
04Commenter 4
Commenter 5
need to update git commit and replace "Zooming" with "Visualization"
05Commenter 2
Branch [archived branch] has been updated forcibly by Author.

[revision removed]
06Author
Updated git commit message according to Commenter 6 comment.
07Commenter 4
Commenter 5 please add some comments to your git message.

I guess it is possible to merge if you will create a new variable
const double aDelta = myMouseActiveGesture == AIS_MouseGesture_Zoom ? aDelta.x() : aDelta.y();
+    case AIS_MouseGesture_ZoomVertical:
+    {
+      if (!myToAllowZooming)
+      {
+        break;
+      }
+      const double aZoomTol = theIsEmulated
+                            ? double(myTouchToleranceScale) * myTouchZoomThresholdPx
+                            : 0.0;
+      if (double (Abs (aDelta.y())) > aZoomTol)
+      {
+        if (UpdateZoom (Aspect_ScrollDelta (aDelta.y())))
+        {
+          toUpdateView = true;
+        }
+        myMouseProgressPoint = thePoint;
+      }
+      break;
+    }
08Commenter 2
Branch [archived branch] has been created by Commenter 7.

[revision removed]


Detailed log of new commits:

Author: Author
Date: Tue Oct 24 15:20:10 2023 +0100

    0033513: Visualization - Integration of the ability to zoom with vertical mouse movements
    
    Added a new gesture for zooming after vertical mouse movement - AIS_MouseGesture_ZoomVertical
09Commenter 7
10Commenter 2
Branch [archived branch] has been deleted by Commenter 4.

[revision removed]
11Commenter 2
Branch [archived branch] has been deleted by Commenter 4.

[revision removed]