Transform: update transformation while navigating #110465

Germano Cavalcante · 2023-07-25T16:18:23+02:00

2023-07-25 16:18:23 +02:00

Previously, while navigating in the 3D View, the object remained in the
transformation prior to navigation, and was only updated when the
navigation ended.

This commit updates the transform during navigation.

This is only noticed for View Rotate/Orbit and Pan with Trackpad or
NDof (not with the mouse since it follows the initial position of the
object).

Ref #109434

Build:
https://builder.blender.org/download/patch/PR110465/

Previously, while navigating in the 3D View, the object remained in the transformation prior to navigation, and was only updated when the navigation ended. This commit updates the transform during navigation. This is only noticed for View Rotate/Orbit and Pan with Trackpad or NDof (not with the mouse since it follows the initial position of the object). --- Ref #109434 Build: https://builder.blender.org/download/patch/PR110465/

Germano Cavalcante commented

2023-07-25 16:18:35 +02:00

@blender-bot package

Blender Bot commented

2023-07-25 16:18:38 +02:00

Package build started. Download here when ready.

Package build started. [Download here](https://builder.blender.org/download/patch/PR110465) when ready.

Germano Cavalcante referenced this pull request

2023-07-25 16:21:05 +02:00

Transform: use transform Pivot for AutoDepth during navigation #109451

Germano Cavalcante requested review from Julien Kaspar 2023-07-25 16:22:05 +02:00

Julien Kaspar commented

2023-07-31 14:15:04 +02:00

IMO this behavior is less jarring than not updating the transform and having the edited element jump. It would be good to get more opinions on this though.

Also the description says it's not supported for mice but for me it works perfectly with a mouse. Not sure if the decription needs an update.

IMO this behavior is less jarring than not updating the transform and having the edited element jump. It would be good to get more opinions on this though. Also the description says it's not supported for mice but for me it works perfectly with a mouse. Not sure if the decription needs an update.

Julien Kaspar approved these changes 2023-07-31 14:15:11 +02:00

Germano Cavalcante commented

2023-07-31 14:39:44 +02:00

the description says it's not supported for mice

Not really what the description says.
The description mentions that this change is not noticed when panning with the mice.
This is because the mouse cursor is in the position where the object was initially.
So Move, Rotate or Scale transform operations keep the same value in this case.

> the description says it's not supported for mice Not really what the description says. The description mentions that this change is not noticed when panning with the mice. This is because the mouse cursor is in the position where the object was initially. So Move, Rotate or Scale transform operations keep the same value in this case.

Julien Kaspar commented

2023-07-31 14:47:10 +02:00

Ok I understand now. Thanks for clarifying.

It might be a good idea to see the opinions of others in the devtalk.

I think that would be a good idea. It all depends on what users think would make this feature most useful. I could also see how it would be more useful if the axes the object is being translated and rotated in from when the transformation was started is locked while navigating, instead of being updated too.
This needs more feedback from people who'd make use of the feature.

But I think for the purposes of base point snapping this already works much better with these two patches 👍

Ok I understand now. Thanks for clarifying. > It might be a good idea to see the opinions of others in the devtalk. I think that would be a good idea. It all depends on what users think would make this feature most useful. I could also see how it would be more useful if the axes the object is being translated and rotated in from when the transformation was started is locked while navigating, instead of being updated too. This needs more feedback from people who'd make use of the feature. But I think for the purposes of base point snapping this already works much better with these two patches 👍

Germano Cavalcante force-pushed transform_update_navigating from e23a25e476 to aa1f1167ec

2023-07-31 16:09:46 +02:00

Compare

Germano Cavalcante commented

2023-07-31 16:09:54 +02:00

@blender-bot package

Blender Bot commented

2023-07-31 16:09:58 +02:00

Package build started. Download here when ready.

Package build started. [Download here](https://builder.blender.org/download/patch/PR110465) when ready.

Germano Cavalcante force-pushed transform_update_navigating from aa1f1167ec to b5cb09415e

2023-08-02 16:13:36 +02:00

Compare

Germano Cavalcante commented

2023-08-02 16:13:48 +02:00

@blender-bot package

Blender Bot commented

2023-08-02 16:13:52 +02:00

Package build started. Download here when ready.

Package build started. [Download here](https://builder.blender.org/download/patch/PR110465) when ready.

Germano Cavalcante commented

2023-08-03 14:38:34 +02:00

Closing for now.
After discussion in the devtalk, some points were presented that show that perhaps this solution is not ideal.

https://devtalk.blender.org/t/snapping-precision-modeling-improvements/28435

Closing for now. After discussion in the devtalk, some points were presented that show that perhaps this solution is not ideal. https://devtalk.blender.org/t/snapping-precision-modeling-improvements/28435

Germano Cavalcante closed this pull request

2023-08-03 14:38:36 +02:00

buildbot/vexp-code-patch-coordinator Build done.

Details

Pull request closed

Please reopen this pull request to perform a merge.

Sign in to join this conversation.

No reviewers

No Label

Download

What's New

Blender Studio

Manual

Developers Blog

Documentation

Benchmark

Blender Conference

Development Fund

One-time Donations

Transform: update transformation while navigating #110465

Pull request closed