Add mesh, align to view, un check align to view. not updating. #28805

New Issue

Martin Lindelöf · 2011-10-04T20:33:09+02:00

Martin Lindelöf commented

2011-10-04 20:33:09 +02:00

%%%1. add mesh
2. check align to view (mesh aligns to view)
3. uncheck align to view (mesh doesn't refresh, neither does X;Y;Z rotation inputs)

revision 40789
mac osx 10.7 lion
%%%

%%%1. add mesh 2. check align to view (mesh aligns to view) 3. uncheck align to view (mesh doesn't refresh, neither does X;Y;Z rotation inputs) revision 40789 mac osx 10.7 lion %%%

Martin Lindelöf commented

2011-10-04 20:33:09 +02:00

Changed status to: 'Open'

Campbell Barton commented

2011-10-05 10:22:02 +02:00

%%%this is surprisingly not easy to solve since there are 2 settings which modify the object rotation and one overrides the other.

If rotation is set, view align is ignored.
If view align is set, rotation is set from view.

problem is that setting view align sets rotation so when its clicked off it just uses the rotation set by the view.

Not that fussy on how this is done but can see its a bit stupid.

I'm tempted to have an align to view operator and remove it from the add-object operators.

This way the user preference is used by the operator, and if the user wants to align to the view in any other context its an explicit action.

Another way to solve is to have the view align not initialize the operators rotation, so the object will be rotated to the view but rotation will read 0,0,0. this is confusing in a different way but at least clicking off 'View Align' will reset the object.%%%

%%%this is surprisingly not easy to solve since there are 2 settings which modify the object rotation and one overrides the other. - If rotation is set, view align is ignored. - If view align is set, rotation is set from view. problem is that setting view align sets rotation so when its clicked off it just uses the rotation set by the view. Not that fussy on how this is done but can see its a bit stupid. I'm tempted to have an align to view operator and remove it from the add-object operators. This way the user preference is used by the operator, and if the user wants to align to the view in any other context its an explicit action. Another way to solve is to have the view align not initialize the operators rotation, so the object will be rotated to the view but rotation will read 0,0,0. this is confusing in a different way but at least clicking off 'View Align' will reset the object.%%%

Martin Lindelöf commented

2011-10-05 11:41:43 +02:00

%%%and I just thought a update() function was left out ^^
because if you hover the inputs you see the X;Y;Z shows the view rotation as you said.

last mentioned solution, wouldn't that be confusing? the added mesh has a obvious rotation, but XYZ says all 0,0,0?

could a temporary 3d vector variable be created at operator call? this 3d vector has 0,0,0 rotation.
and if the align-to-view is un checked the mesh wil be rotated according to this temporary 3d vector ?

so rotation is never set, it's always read either from 3d viewport rotation, or this temporary 3d vector.%%%

%%%and I just thought a update() function was left out ^^ because if you hover the inputs you see the X;Y;Z shows the view rotation as you said. last mentioned solution, wouldn't that be confusing? the added mesh has a obvious rotation, but XYZ says all 0,0,0? could a temporary 3d vector variable be created at operator call? this 3d vector has 0,0,0 rotation. and if the align-to-view is un checked the mesh wil be rotated according to this temporary 3d vector ? so rotation is never set, it's always read either from 3d viewport rotation, or this temporary 3d vector.%%%

Martin Lindelöf commented

2011-10-05 11:44:38 +02:00

%%%and if user is inputing XYZ rotation, he's actually rotating this 3d vector, and mesh is reading it's XYZ rotation.%%%

Brecht Van Lommel commented

2011-10-05 14:20:39 +02:00

%%%I think align to view should be part of the operator, there's many operators that work this way, getting their default parameters from the user preferences. I think the problem is in ignoring align to view when the rotation is set. To make that more clear, the rotation values could be grayed out in the UI when align to view is enabled.%%%

Campbell Barton commented

2011-10-06 02:25:22 +02:00

%%%@brecht, the problem with always using the view align option is the object will always snap to the view rotation, even when changing other settings, maybe this is OK though it used to do this and IMHO was odd/annoying.

Since you add the object (View align on in userprefs), rotate the view, change the location --- object faces view again.
%%%

%%%@brecht, the problem with always using the view align option is the object will always snap to the view rotation, even when changing other settings, maybe this is OK though it used to do this and IMHO was odd/annoying. Since you add the object (View align on in userprefs), rotate the view, change the location --- object faces view again. %%%

Sergey Sharybin commented

2011-10-06 14:13:27 +02:00

%%%Prepared patch for this issue: http://www.pasteall.org/25391/diff
IMO, storing rotation to prevent re-aligning when adjusting other settings reduces amount of confusing thing. So it was only issue of re-setting stored rotation when toggling Align To View option.

Prefer Campbell/Brecht to verify way of resetting rotation before commit.%%%

%%%Prepared patch for this issue: http://www.pasteall.org/25391/diff IMO, storing rotation to prevent re-aligning when adjusting other settings reduces amount of confusing thing. So it was only issue of re-setting stored rotation when toggling Align To View option. Prefer Campbell/Brecht to verify way of resetting rotation before commit.%%%

Brecht Van Lommel commented

2011-10-06 14:45:07 +02:00

%%%Campbell, indeed, I didn't consider that.

Sergey, this looks like a good fix to me.%%%

%%%Campbell, indeed, I didn't consider that. Sergey, this looks like a good fix to me.%%%

Sergey Sharybin commented

2011-10-06 14:52:13 +02:00

%%%Patch commited to svn rev40831.
Thanks for the report, closing.%%%

%%%Patch commited to svn rev40831. Thanks for the report, closing.%%%

Sergey Sharybin commented

2011-10-06 14:52:13 +02:00

Changed status from 'Open' to: 'Resolved'

Sergey Sharybin closed this issue

2011-10-06 14:52:13 +02:00

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

Add mesh, align to view, un check align to view. not updating. #28805