Archived issue #0033513
Visualization - Integration of the ability to zoom with vertical mouse movements.
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.
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
[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
For me it looks ok. Was added enum which allows to zoom in/ zoom out vertically.
Now awaiting for tests on Jenkins,
Will be approved after tests.
(now no remarks)
Will be approved after tests.
(now no remarks)
Commenter 5
need to update git commit and replace "Zooming" with "Visualization"
need to update git commit and replace "Zooming" with "Visualization"
Branch [archived branch] has been updated forcibly by Author.
[revision removed]
[revision removed]
Updated git commit message according to Commenter 6 comment.
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();
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;
+ }
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
[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
Please integrate
OCCT: CR33513_1
Tests:
http://jenkins-test-10.nnov.opencascade.com/view/CR33513-master-Commenter 7/view/ALL/
OCCT: CR33513_1
Tests:
http://jenkins-test-10.nnov.opencascade.com/view/CR33513-master-Commenter 7/view/ALL/
Branch [archived branch] has been deleted by Commenter 4.
[revision removed]
[revision removed]
Branch [archived branch] has been deleted by Commenter 4.
[revision removed]
[revision removed]