Transform: use transform Pivot for AutoDepth during navigation #109451
No reviewers
Labels
No Label
Interest
Alembic
Interest
Animation & Rigging
Interest
Asset Browser
Interest
Asset Browser Project Overview
Interest
Audio
Interest
Automated Testing
Interest
Blender Asset Bundle
Interest
BlendFile
Interest
Collada
Interest
Compatibility
Interest
Compositing
Interest
Core
Interest
Cycles
Interest
Dependency Graph
Interest
Development Management
Interest
EEVEE
Interest
EEVEE & Viewport
Interest
Freestyle
Interest
Geometry Nodes
Interest
Grease Pencil
Interest
ID Management
Interest
Images & Movies
Interest
Import Export
Interest
Line Art
Interest
Masking
Interest
Metal
Interest
Modeling
Interest
Modifiers
Interest
Motion Tracking
Interest
Nodes & Physics
Interest
OpenGL
Interest
Overlay
Interest
Overrides
Interest
Performance
Interest
Physics
Interest
Pipeline, Assets & IO
Interest
Platforms, Builds & Tests
Interest
Python API
Interest
Render & Cycles
Interest
Render Pipeline
Interest
Sculpt, Paint & Texture
Interest
Text Editor
Interest
Translations
Interest
Triaging
Interest
Undo
Interest
USD
Interest
User Interface
Interest
UV Editing
Interest
VFX & Video
Interest
Video Sequencer
Interest
Virtual Reality
Interest
Vulkan
Interest
Wayland
Interest
Workbench
Interest: X11
Legacy
Blender 2.8 Project
Legacy
Milestone 1: Basic, Local Asset Browser
Legacy
OpenGL Error
Meta
Good First Issue
Meta
Papercut
Meta
Retrospective
Meta
Security
Module
Animation & Rigging
Module
Core
Module
Development Management
Module
EEVEE & Viewport
Module
Grease Pencil
Module
Modeling
Module
Nodes & Physics
Module
Pipeline, Assets & IO
Module
Platforms, Builds & Tests
Module
Python API
Module
Render & Cycles
Module
Sculpt, Paint & Texture
Module
Triaging
Module
User Interface
Module
VFX & Video
Platform
FreeBSD
Platform
Linux
Platform
macOS
Platform
Windows
Priority
High
Priority
Low
Priority
Normal
Priority
Unbreak Now!
Status
Archived
Status
Confirmed
Status
Duplicate
Status
Needs Info from Developers
Status
Needs Information from User
Status
Needs Triage
Status
Resolved
Type
Bug
Type
Design
Type
Known Issue
Type
Patch
Type
Report
Type
To Do
No Milestone
No project
No Assignees
4 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: blender/blender#109451
Loading…
Reference in New Issue
No description provided.
Delete Branch "mano-wii/blender:transform_navigate_mouse"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
To avoid the small jumps during the Pan operation, a feature was
implemented in the
ED_view3d_navigation_do
utility that replaces thepoint used for AutoDepth.
As a result, the depth of the pivot point is now used during the
transform.
Also, to ensure minimal disruption to non-AutoDepth users, this feature
is enabled only when "Auto Depth" is enabled.
Ref #109434
Build:
https://builder.blender.org/download/patch/PR109451/
@blender-bot package
Package build started. Download here when ready.
I've used it for a bit and it looks like it fixed all issues I've pointed out in #109434 👍
But it would be good if some more people tested it.
1cea784c20
tod40e10e344
When testing I noticed some bugs in the implementation in main.
I fixed this bug with
ce7ee791b0
andb6a534e41b
.Only
ce7ee791b0
also affects this patch.I'll keep testing.
@blender-bot package
Package build started. Download here when ready.
d40e10e344
to893fe16d93
@blender-bot build
I tried the new build. I have issues with all GRS operations.
Move immediately brings the object to a far away position. (-16711634 m, 4881020 m, -13445089 m) for example. (It seems these numbers change based on the size of viewport. The numbers stay the same if the viewport size stays the same.) Rotate immediately rotates the object some degree along the view axis. Scale immediately scales the object to (0.0, 0.0, 0.0).
These all happen regardless of navigation controls.
893fe16d93
tofb81590128
@persun, thanks for the information and for looking to test.
I still don't have a strong opinion about this change.
For me both approaches are good.
And I can't say which one is more intuitive.
@blender-bot package
Package build started. Download here when ready.
fb81590128
toa2586e5dca
Update:
To avoid small jumps during the Pan operation, a feature was implemented in the
ED_view3d_navigation_do
utility that overwrites the point used forAutoDepth
. So the depth becomes the pivot point of the transformation.Also, in order not to change the behavior too much for users who don't use
AutoDepth
, this feature (of using the depth of the pivot point) is only used when the option "Auto Depth" is enabled.@blender-bot package
Package build started. Download here when ready.
@blender-bot package
Package build started. Download here when ready.
dde1dd023a
tod1de05c91b
For testing, this PR also implements a feature to apply the transform during a navigation.
This removes 100% of the jumps.
But on the other hand, maybe it can be a bit disorienting when Orbiting and Zooming the 3D View.
@blender-bot package
Package build started. Download here when ready.
Transform: Navigate: Disable umproject cursor aproachto Transform: Navigation Tests: Auto Depth to Pivot | Screen mouse offset position | Transform during navigationd1de05c91b
to36edd6f43e
I updated the build and removed the feature to update the transform during navigation.
This feature has been moved to !110465
It might help to test the features separately.
@blender-bot package
Package build started. Download here when ready.
Transform: Navigation Tests: Auto Depth to Pivot | Screen mouse offset position | Transform during navigationto Transform: Navigation Tests: Auto Depth to Pivot | Screen mouse offset positionWorks just like expected with and without Auto Depth enabled.
Germano Cavalcante referenced this pull request2023-07-31 14:39:44 +02:00
36edd6f43e
toebc37c38fd
Transform: Navigation Tests: Auto Depth to Pivot | Screen mouse offset positionto Transform: use transform Pivot for AutoDepth during navigation