problem with turntable orbit #47263

New Issue

item412 · 2016-01-28T11:31:39+01:00

item412 commented

2016-01-28 11:31:39 +01:00

System Information
win7 64

Blender Version
Broken: 2.76

Short description of error

If you tilt view (shift+num4/num6) orbit for rotation remains world Z, not local. From tilted point of view that makes no sense and looks like its broken.

Exact steps for others to reproduce the error
press shift+num6 several times and try to rotate view horizontally

**System Information** win7 64 **Blender Version** Broken: 2.76 **Short description of error** If you tilt view (shift+num4/num6) orbit for rotation remains world Z, not local. From tilted point of view that makes no sense and looks like its broken. **Exact steps for others to reproduce the error** press shift+num6 several times and try to rotate view horizontally

item412 commented

2016-01-28 11:31:39 +01:00

Changed status to: 'Open'

item412 commented

2016-01-28 11:31:39 +01:00

Added subscriber: @item412

Bastien Montagne commented

2016-01-29 16:04:52 +01:00

Added subscribers: @ideasman42, @mont29

Bastien Montagne commented

2016-01-29 16:04:52 +01:00

This is dead easy to fix (see patch below), @ideasman42 any specific reason to have current behavior?

Patch:

diff --git a/source/blender/editors/space_view3d/view3d_edit.c b/source/blender/editors/space_view3d/view3d_edit.c
index 0f40765..15fc4ca 100644
--- a/source/blender/editors/space_view3d/view3d_edit.c
+++ b/source/blender/editors/space_view3d/view3d_edit.c
@@ -4005,8 +4005,8 @@ static int vieworbit_exec(bContext *C, wmOperator *op)
 					angle = -angle;
 				}
 
-				/* z-axis */
-				axis_angle_to_quat_single(quat_mul, 'Z', angle);
+				/* View Y-axis */
+				axis_angle_to_quat(quat_mul, rv3d->viewinv[1], angle);
 			}
 			else {
 
@@ -4014,7 +4014,7 @@ static int vieworbit_exec(bContext *C, wmOperator *op)
 					angle = -angle;
 				}
 
-				/* horizontal axis */
+				/* View X-axis */
 				axis_angle_to_quat(quat_mul, rv3d->viewinv[0], angle);
 			}

This is dead easy to fix (see patch below), @ideasman42 any specific reason to have current behavior? Patch: ``` diff --git a/source/blender/editors/space_view3d/view3d_edit.c b/source/blender/editors/space_view3d/view3d_edit.c index 0f40765..15fc4ca 100644 --- a/source/blender/editors/space_view3d/view3d_edit.c +++ b/source/blender/editors/space_view3d/view3d_edit.c @@ -4005,8 +4005,8 @@ static int vieworbit_exec(bContext *C, wmOperator *op) angle = -angle; } - /* z-axis */ - axis_angle_to_quat_single(quat_mul, 'Z', angle); + /* View Y-axis */ + axis_angle_to_quat(quat_mul, rv3d->viewinv[1], angle); } else { @@ -4014,7 +4014,7 @@ static int vieworbit_exec(bContext *C, wmOperator *op) angle = -angle; } - /* horizontal axis */ + /* View X-axis */ axis_angle_to_quat(quat_mul, rv3d->viewinv[0], angle); } ```

Bastien Montagne commented

2016-01-29 16:54:33 +01:00

Problem here is that you sometimes want current behavior, e.g. when 'looking down' without any tilt, rotating around global Z axis is most desired, we do not want to rotate around view Y axis then...

item412 commented

2016-01-29 23:59:08 +01:00

I agree, for modeling thats ok, but for sculpting/painting ability to tilt and rotate around view Y is very usefull. I think we need then switch (like in zbrush) in user preferences.

Campbell Barton commented

2016-01-30 02:49:30 +01:00

@mont29, think patched behavior is more correct/consistent (none of the other orbit keys use global axis).
However I expect some users prefer current behavior.

Interactive orbit defaults to turntable, so think its OK to change this, so num-keys all use relative axis.

@mont29, think patched behavior is more correct/consistent (none of the other orbit keys use global axis). However I expect some users prefer current behavior. Interactive orbit defaults to turntable, so think its OK to change this, so num-keys all use relative axis.

blender-admin commented

2016-01-30 19:07:36 +01:00

This issue was referenced by fe0ca82b23

This issue was referenced by fe0ca82b232dcab29030c4a20b8bd1b033d1d346

Bastien Montagne commented

2016-01-30 19:09:46 +01:00

Changed status from 'Open' to: 'Resolved'

Bastien Montagne closed this issue

2016-01-30 19:09:46 +01:00

Robert M commented

2016-02-01 21:10:45 +01:00

Added subscriber: @robertm

Robert M commented

2016-02-01 21:10:45 +01:00

Why not make this an option? I used the old orbit behavior to quickly setup the camera for axonometric projection (useful for isometric games and tile rendering).
Now this is much more complicated.

Why not make this an option? I used the old orbit behavior to quickly setup the camera for axonometric projection (useful for isometric games and tile rendering). Now this is much more complicated.

item412 commented

2016-02-02 09:54:22 +01:00

Option to rotate also with middle mouse, not only nums. There should be two turntables: world Z and view Y

Moon Caine commented

2016-03-01 18:07:59 +01:00

Added subscriber: @mooncaine

Moon Caine commented

2016-03-01 18:07:59 +01:00

This breaks an existing, expected behavior. User should have an option somehow to choose the orbit behavior.

Leon Eckardt commented

2016-03-09 17:49:54 +01:00

Added subscriber: @Leon95

Leon Eckardt commented

2016-03-09 17:49:54 +01:00

used NUM 4/6 sometimes on laptop if i don't have my mouse around. current behaviour is not really helpful in replacing the MMB function. I vote also for user preferences option.
Users even filed it as bug ^^

used NUM 4/6 sometimes on laptop if i don't have my mouse around. current behaviour is not really helpful in replacing the MMB function. I vote also for user preferences option. Users even filed it as bug ^^

Carsten commented

2016-03-14 12:14:01 +01:00

Added subscriber: @CarstenWartmann

Carsten commented

2016-03-14 12:14:01 +01:00

Wow, thats a drastic change for me after >15Years of Blender. I don't like it. Even Experienced user will rotate quickly into a view which is useless but not recoverable with turning back.

Please make a Preference for the old behaiviour.

Carsten

Wow, thats a drastic change for me after >15Years of Blender. I don't like it. Even Experienced user will rotate quickly into a view which is useless but not recoverable with turning back. Please make a Preference for the old behaiviour. Carsten

Olson Burry commented

2016-03-23 02:44:50 +01:00

Added subscriber: @OliverBurn

Olson Burry commented

2016-03-23 02:44:50 +01:00

Consistency is one thing, but when it breaks an objectively useful behaviour I will argue against consistency in favour of some common sense.

This change is horrible and I want the 2.76 behaviour back.

Consistency is one thing, but when it breaks an objectively useful behaviour I will argue against consistency in favour of some common sense. This change is horrible and I want the 2.76 behaviour back.

Paul R commented

2016-03-23 17:29:56 +01:00

Added subscriber: @intracube

Paul R commented

2016-03-23 17:29:57 +01:00

Another vote for this being an option with the default set to the long-standing behaviour.

Kind of surprised this has suddenly changed considering the old way was expected and useful.

Edit: Just noticed the proposal to revert in D1868

Another vote for this being an option with the default set to the long-standing behaviour. Kind of surprised this has suddenly changed considering the old way was expected and useful. Edit: Just noticed the proposal to revert in [D1868](https://archive.blender.org/developer/D1868)

Quentin Wenger commented

2016-03-23 20:22:42 +01:00

Added subscriber: @QuentinWenger

Quentin Wenger commented

2016-03-23 20:22:42 +01:00

D1868 was an attempt of mine to add an option for both cases.

It has however been decided today in IRC that best was to revert the incrimined commit.

I thus closed my revision and 2.77a will see good ol' behavior back.

[D1868](https://archive.blender.org/developer/D1868) was an attempt of mine to add an option for both cases. It has however been decided today in IRC that best was to revert the incrimined commit. I thus closed my revision and 2.77a will see good ol' behavior back.

Paul R commented

2016-03-24 16:53:28 +01:00

In #47263#365751, @QuentinWenger wrote:
D1868 was an attempt of mine to add an option for both cases.

It has however been decided today in IRC that best was to revert the incrimined commit.

I thus closed my revision and 2.77a will see good ol' behavior back.

Yep, I read the IRC backlog :)

Your patch looks like it would've been a good alternative.

> In #47263#365751, @QuentinWenger wrote: > [D1868](https://archive.blender.org/developer/D1868) was an attempt of mine to add an option for both cases. > > It has however been decided today in IRC that best was to revert the incrimined commit. > > I thus closed my revision and 2.77a will see good ol' behavior back. Yep, I read the IRC backlog :) Your patch looks like it would've been a good alternative.

blender-admin commented

2016-03-25 12:09:59 +01:00

This issue was referenced by 7219988258

This issue was referenced by 72199882587b20b7b72b5205cb84372fdc8da12c

Bastien Montagne commented

2016-03-25 12:16:31 +01:00

'fix' was reverted. Not re-opening the task, since it’s not a bug but rather a design issue…

Sign in to join this conversation.

No Label

Download

What's New

Blender Studio

Manual

Developers Blog

Documentation

Benchmark

Blender Conference

Development Fund

One-time Donations

problem with turntable orbit #47263