Add ability to transform nodes relative to the active node#104233
Open
ryevdokimov wants to merge 1 commit intogodotengine:masterfrom
Open
Add ability to transform nodes relative to the active node#104233ryevdokimov wants to merge 1 commit intogodotengine:masterfrom
active node#104233ryevdokimov wants to merge 1 commit intogodotengine:masterfrom
Conversation
ryevdokimov
force-pushed
the
relative-transform
branch
from
e1c9e94 to
934d09f
Compare
ryevdokimov
force-pushed
the
relative-transform
branch
2 times, most recently
from
66901c2 to
0e94194
Compare
ryevdokimov
force-pushed
the
relative-transform
branch
from
0e94194 to
c0f5215
Compare
ryevdokimov
force-pushed
the
relative-transform
branch
from
c0f5215 to
647c526
Compare
Calinou
reviewed
Jul 8, 2025
Member
Calinou
left a comment
Calinou
left a comment
There was a problem hiding this comment.
Tested locally, it works as expected. Code looks good to me.
Some feedback:
- The default shortcut (A) will conflict with the physical Q shortcut from #103533 on AZERTY keyboards. I suggest making it use physical A, so that it becomes Q on AZERTY to avoid conflicts.
- The feature's state is not persisted when disabling Use Local Space. This also means that if you use Blender-style manipulation shortcuts, it's impossible to make use of the feature as you always have to go through global space before going through local space (by pressing the manipulation shortcut a second time).
- The tooltip should mention that the feature is only effective when Use Local Space is enabled (so users know why it appears grayed out by default).
ryevdokimov
force-pushed
the
relative-transform
branch
from
647c526 to
ff79c80
Compare
Contributor
Author
|
Feedback addressed. Also fixed saving state when project is reloaded. |
ryevdokimov
force-pushed
the
relative-transform
branch
from
ff79c80 to
a2b25bd
Compare
ryevdokimov
force-pushed
the
relative-transform
branch
from
a2b25bd to
db680c5
Compare
ryevdokimov
force-pushed
the
relative-transform
branch
from
db680c5 to
6b0246c
Compare
ryevdokimov
force-pushed
the
relative-transform
branch
2 times, most recently
from
0690825 to
e6d081b
Compare
ryevdokimov
force-pushed
the
relative-transform
branch
from
e6d081b to
49158e6
Compare
TokageItLab
requested changes
Nov 23, 2025
Member
There was a problem hiding this comment.
If we're following Blender's gui, I think we should also allow setting the Origin separately. However, I understand the concern that this might add too many features.
Even so, the current this PR implementation seems inconsistent regarding position, rotation, and scaling, making it unclear what the intended purpose is. The icons are also unclear.
So as a proposal, how about implementing most operations the same as Global, but only setting the Gizmo's transform to Local:
- Ignore EachOrigin
- Set the Gizmo's position to match the leader object's Origin, not the average position
- Set the Gizmo's basis to match the leader object's Basis
Also, I think it's okay to make this Local Gizmo + Global Operation the default.
Then, representing the current Each Origin + Local Operation with the following icons would make the behavior clearer:
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
#102176 introduced the ability to select an
active nodeto determine the basis of the transform gizmo. This PR adds the ability to transform nodes relative to that basis now. This will the improve the usability of transforming several nodes at once.A new tool button has been added:
This button is only enabled when the adjacent
Use Local Spacetool is enabled, and the functionality is automatically disabled when switching back to global space.2025-03-16.17-29-11.mp4