Revert center of rotation change after pan #3310
Conversation
PaulHax
force-pushed
the
center-rotation
branch
from
a938190 to
b087e01
Compare
Folks camera control needs are diverse. Extention is the way. Kitchen sink camera controls are a trap. But InteractionStyleManipulator is a monster already, so maybe I'm missing something and we should put a option in there? |
From my side, I didn’t understand why the behaviour changed. It has been like this for a long time and didn’t seem to bother anyone. I get that the new behaviour can be useful, but perhaps it would have been better to add it as an option rather than changing the default. |
| // Also allow pan with shift + left button | ||
| const shiftPanManipulator = vtkMouseCameraTrackballPanManipulator.newInstance(); | ||
| shiftPanManipulator.setButton(1); | ||
| shiftPanManipulator.setShift(true); |
There was a problem hiding this comment.
This should have an option to enable or disable center of rotation modification. The default should remain the way it was before.
PaulHax
force-pushed
the
center-rotation
branch
2 times, most recently
from
6bdc658 to
71890ad
Compare
Add vtkMouseCameraTrackballPanManipulatorAutoCenter closes #3309
PaulHax
force-pushed
the
center-rotation
branch
from
71890ad to
78d2d04
Compare
PaulHax
changed the title
|
I rolled back the InteractorStyleManipulator behavior. Added a |
Rolls back InteractorStyleManipulator center of rotation behaior to not update on pan.
Adds
vtkMouseCameraTrackballPanManipulatorAutoCenterfor the auto update on center of rotation for mouse only input.Test with
npm run example InteractorStyleManipulatorCompare to
npm run example InteractorStyleManipulatorRotateCameraCentercloses #3309