Add config option for "camera" rotation navigation#956
Add config option for "camera" rotation navigation#956phkahler merged 1 commit intosolvespace:masterfrom
Conversation
SS rotates the model when middle button dragging while some users expect this operation to rotate the camera where left-right and up-down directions are reversed instead. This adds that option.
|
An option to turn the turntable can be found in the config. |
|
@hinell That option switches to a different style of rotation. The "camera" rotation is very different and acts as though your mouse is moving the camera rather than the model/sketch. So dragging to the right causes you to view the model from the right instead of rotating the model to the right which would show you the left side of it. At least that my understanding. Moving the model seems very natural to me and the idea of moving the camera would be confusing, but I can see where camera movement seems natural to some people and moving the model like we do now would be very confusing. We'll revisit this soon. When this PR came along none of this was clear to me ;-) |
|
Note to self look at my pending reviews more often. |
| if(SS.turntableNav) { // lock the Z to vertical | ||
| double sign = SS.cameraNav ? -1.0 : 1.0; | ||
| double s = 0.3*(PI/180)*scale*sign; // degrees per pixel | ||
| if(SS.turntableNav) { // lock the Z to vertical |
There was a problem hiding this comment.
And also I think this fixes the CTRL-middle drag "hack" of turntable navigation.
#888 (comment)
#1307 (comment)
|
As much as I hate it, I know people who want it find the existing behavior equally terrible ;-) It works fine for me on Linux and it's a really small PR! |
4 participants
issue #325 requests an alternative to middle-button mouse navigation/rotation. What I think the user is asking for is an option to manipulate the camera instead of model when rotating, i.e. reversing up/down and left/right directions. This PR implements that feature (independent of turntable navigation) as a config option called "camera mouse navigation" (name could use some work). It drives me a little bit crazy but I think some are used to this and would welcome this option.