Maniphest T63470

Origin to 3D Cursor does not respect Cursor rotation
Closed, Archived

Assigned To
Sebastian Parborg (zeddb)
Authored By
Jacob Max Fober (Fubax)
Apr 10 2019, 9:03 PM
Tags
  • BF Blender
Subscribers
Brecht Van Lommel (brecht)
Habib Gahbiche (zazizizou)
Henrique Corrêa (HeCorr)
Jacob Max Fober (Fubax)
Martin Capitanio (capnm)
William Reynish (billreynish)
Tokens
"Heartbreak" token, awarded by lopoIsaac.

Description

System Information
Operating system: Windows-7-6.1.7601-SP1 64 Bits
Graphics card: GeForce GT 1030/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 385.12

Blender Version
Broken: version: 2.80 (sub 55), branch: blender2.7, commit date: 2019-04-09 18:12, hash: rB009dbc2bc9fb
Worked: (optional)

Short description of error
When you set object origin to 3D cursor it only accounts for Cursor XYZ location, discarding its rotation

Exact steps for others to reproduce the error

  1. Rotate and move 3D Cursor in the 'N' panel.
  2. Select default Cube and goto Object > Set Origin > Origin to 3D Cursor.

[Based on the default startup]

Event Timeline

Jacob Max Fober (Fubax) created this task.Apr 10 2019, 9:03 PM
Sebastian Parborg (zeddb) claimed this task.Apr 11 2019, 4:11 PM
Sebastian Parborg (zeddb) lowered the priority of this task from 90 to 50.
Habib Gahbiche (zazizizou) added a subscriber: Habib Gahbiche (zazizizou).Apr 28 2019, 3:41 PM

It might make sense to consider cursor rotation for 'Snap Selection to Cursor' but I don't think the origin itself has a rotation, so I wouldn't classify this behavior as a bug.

If this turns out to be really a bug I can work on it. For linked objects I would suggest to rotate them as well according to the set Pivot Point.

Jacob Max Fober (Fubax) added a comment.Apr 29 2019, 12:22 AM

From a user point-of-view It seems logic to have origin axis rotated as 3D cursor. Since there's no other direct option to do that.

Imagine situation, where you import 3D Scan of an object that's off axis and rotated.

The only way to set origin to some flat surface now is:

  1. Project 3D cursor to rotated object geometry
  2. Create Empty object
  3. Set Transform Orientation to 3D Cursor
  4. Select Empty object and go to Object > Transform > Align to Transform Orientation
  5. Parent rotated object under the Empty with "Keep transform"
  6. Reset empty rotation and position
  7. Clear Parent for rotated object (and keep transform)
  8. Select Rotated object and go to Object > Apply > All transforms
Sebastian Parborg (zeddb) added a comment.Apr 29 2019, 11:43 AM

@Habib Gahbiche (zazizizou) it is not really a bug per se. It's more a consistency in the UI. Now that we have rotation on the 3d cursor. It would be logical to at least have the option to use it for snapping.
If you want to make the snapping take into account the 3D cursor rotation, then by all means do so! :)

Habib Gahbiche (zazizizou) added a comment.Apr 29 2019, 8:43 PM

@Jacob Max Fober (Fubax) I'm not sure how Set Origin to Cursor would solve your problem. You could just clear rotation and location after importing if the information is there, otherwise you'll need to do something like

  1. Go to edit mode
  2. Select some flat surface
  3. Snape Cursor to Selection
  4. Go to object mode
  5. Set Origin to 3D Cursor.

These steps don't change the rotation of the object though, but your suggestion of Set origin to 3D cursor would.

@Sebastian Parborg (zeddb) ok thanks. I agree with snapping so I'll start working on that :)

William Reynish (billreynish) added a subscriber: William Reynish (billreynish).Apr 29 2019, 8:47 PM

While I agree it would be nice if it were added, I am not sure we can really call it a bug - it's just a missing feature of the snapping operator.

Jacob Max Fober (Fubax) added a comment.EditedApr 29 2019, 9:40 PM

@Habib Gahbiche (zazizizou)
Well with Set Origin to Cursor accounting for cursor rotation the process would look like:

  1. Select Cursor tool (make sure Surface Project is enabled it the Top Bar)
  2. Click on some flat surface
  3. Set Origin to 3D Cursor

@William Reynish (billreynish)
I understand your point of view. I'm not sure either if it's really a bug =) Maybe new category (not real bug)

Martin Capitanio (capnm) added a subscriber: Martin Capitanio (capnm).EditedApr 29 2019, 10:14 PM

While I agree it would be nice if it were added, I am not sure we can really call it a bug - it's just a missing feature of the snapping operator.

From the user's point of view, most of us would certainly think aligning with the cursor
and ignoring the cursor rotation is a bug.

The same problem is when you add a new object. Anyone I asked would expect the
new object to be aligned with the cursor (unless he chooses the "Align to View" option).

Habib Gahbiche (zazizizou) added a comment.May 1 2019, 1:14 PM
In T63470#668136, @Martin Capitanio (capnm) wrote:

While I agree it would be nice if it were added, I am not sure we can really call it a bug - it's just a missing feature of the snapping operator.

From the user's point of view, most of us would certainly think aligning with the cursor
and ignoring the cursor rotation is a bug.

The same problem is when you add a new object. Anyone I asked would expect the
new object to be aligned with the cursor (unless he chooses the "Align to View" option).

This refers to snapping and not setting the origin to 3D cursor. I think we all agree on that.

@Jacob Max Fober (Fubax) ok I thought by flat surface you meant a face of your object. However I still think that setting the origin to 3D cursor should not change the object's position or rotation (so keep the current behavior) and instead change snapping to consider cursor rotation.

Jacob Max Fober (Fubax) added a comment.May 1 2019, 1:57 PM
In T63470#669238, @Habib Gahbiche (zazizizou) wrote:

(...) (so keep the current behavior) and instead change snapping to consider cursor rotation.

Yes, that's exactly what I had in mind. Thanks

PS: Maybe idea for another tool:
Move Origin, while geometry stays stationary in World Space.

Habib Gahbiche (zazizizou) added a comment.May 25 2019, 8:54 PM

I submitted a patch that handles snap selected to cursor in object mode: D4897. Also feel free to leave some feedback on blenderartists

Brecht Van Lommel (brecht) changed the task status from Unknown Status to Unknown Status.May 29 2019, 3:47 AM
Brecht Van Lommel (brecht) added a subscriber: Brecht Van Lommel (brecht).

We should review this patch for inclusion in 2.81, as it was submitted after the freeze for 2.80. But there is no bug here, it's a feature request.

Lopo Isaac (lopoIsaac) awarded a token.Sep 24 2020, 6:28 PM
Henrique Corrêa (HeCorr) added a subscriber: Henrique Corrêa (HeCorr).Mar 23 2022, 9:34 PM

I can't believe we're in 2022 and snapping a bone or an object to the 3D cursor doesn't affect it's rotation.