Enabling Normal option for Orientation not working #85229

New Issue

Pratik Borhade · 2021-01-30T15:05:42+01:00

Pratik Borhade commented

2021-01-30 15:05:42 +01:00

System Information
Operating system: Windows-10-10.0.18362-SP0 64 Bits
Graphics card: AMD Radeon(TM) 535 ATI Technologies Inc. 4.5.13540 Core Profile Context 25.20.14002.1

Blender Version
Broken: version: 2.92.0 Alpha , commit date: 2020-12-21 12:33, hash: d2239b685c

Short description of error
Cannot enable Normal option for Orientation

Exact steps for others to reproduce the error

Add -> Mesh -> Monkey
Object -> Transform -> Align to Transform Orientation

Orientation -> Normal

On performing following steps , Orientation will set to Local instead of Normal

Blender 2021-01-30 19-26-16.mp4

**System Information** Operating system: Windows-10-10.0.18362-SP0 64 Bits Graphics card: AMD Radeon(TM) 535 ATI Technologies Inc. 4.5.13540 Core Profile Context 25.20.14002.1 **Blender Version** Broken: version: 2.92.0 Alpha , commit date: 2020-12-21 12:33, hash: `d2239b685c` **Short description of error** Cannot enable `Normal` option for Orientation **Exact steps for others to reproduce the error** - Add -> Mesh -> Monkey - Object -> Transform -> Align to Transform Orientation # Orientation -> Normal On performing following steps , Orientation will set to *Local* instead of *Normal* [Blender 2021-01-30 19-26-16.mp4](https://archive.blender.org/developer/F9605315/Blender_2021-01-30_19-26-16.mp4)

Pratik Borhade commented

2021-01-30 15:05:42 +01:00

Added subscriber: @PratikPB2123

Falk David commented

2021-02-01 09:49:40 +01:00

Added subscribers: @mano-wii, @filedescriptor

Falk David commented

2021-02-01 09:49:40 +01:00

I can confirm this in the latest beta. Looking at the code it seems as though this is intended:

case V3D_ORIENT_NORMAL: {
  if (obedit || (ob && ob->mode & OB_MODE_POSE)) {
    ED_getTransformOrientationMatrix(C, ob, obedit, pivot_point, r_mat);
    return V3D_ORIENT_NORMAL;
  }
  /* no break we define 'normal' as 'local' in Object mode */
  ATTR_FALLTHROUGH;
}```

This was changed in rB515e8e4 by @mano-wii . Could you comment on this?

I can confirm this in the latest beta. Looking at the code it seems as though this is intended: ``` case V3D_ORIENT_NORMAL: { if (obedit || (ob && ob->mode & OB_MODE_POSE)) { ED_getTransformOrientationMatrix(C, ob, obedit, pivot_point, r_mat); return V3D_ORIENT_NORMAL; } /* no break we define 'normal' as 'local' in Object mode */ ATTR_FALLTHROUGH; }``` This was changed in rB515e8e4 by @mano-wii . Could you comment on this?

Germano Cavalcante commented

2021-02-01 14:06:25 +01:00

Normal orientation in object mode has always been local.
Before, it just wasn't so explicit.

I see two solutions for this:

Hide "not possible" orientations.
Return the chosen orientation even with another one being used (this is how it worked before).

Normal orientation in object mode has always been local. Before, it just wasn't so explicit. I see two solutions for this: - Hide "not possible" orientations. - Return the chosen orientation even with another one being used (this is how it worked before).

Pratik Borhade commented

2021-02-01 14:23:03 +01:00

I had seen the code before reporting the issue . Moving to next case ( V3D_ORIENT_LOCAL) with no break statement in previous case(V3D_ORIENT_NORMAL) looks very uncommon (reading comment though) . If it is not an issue , we can close this this task .

I had seen the code before reporting the issue . Moving to next case ( `V3D_ORIENT_LOCAL`) with no break statement in previous case(`V3D_ORIENT_NORMAL`) looks very uncommon (reading comment though) . If it is not an issue , we can close this this task .

Germano Cavalcante commented

2021-02-01 15:10:28 +01:00

Although it is not really a bug, it is in fact strange that the operator returns another orientation than the one chosen.
One of the reasons for this change was to better inform the orientation that is being used in the header.
But since the old versions did not do this, I think it is better to return to the previous behavior.

(Fix coming soon).

Although it is not really a bug, it is in fact strange that the operator returns another orientation than the one chosen. One of the reasons for this change was to better inform the orientation that is being used in the header. But since the old versions did not do this, I think it is better to return to the previous behavior. (Fix coming soon).

blender-admin commented

2021-02-01 15:16:49 +01:00

This issue was referenced by a69983f30e

This issue was referenced by a69983f30e3fd342a605910fa81ae72c25869ff0

Germano Cavalcante commented

2021-02-01 15:17:38 +01:00

Changed status from 'Needs Triage' to: 'Resolved'

Germano Cavalcante closed this issue

2021-02-01 15:17:38 +01:00

Germano Cavalcante self-assigned this 2021-02-01 15:17:38 +01: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

Enabling Normal option for Orientation not working #85229

Orientation -> Normal