Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Commit transforms done with editor gizmo on tool mode switch. #86930

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ryevdokimov
Copy link
Contributor

@ryevdokimov ryevdokimov commented Jan 7, 2024

Supplementary but not dependent on: #86805 and #86804

This affects 2D and 3D.

Currently you can switch gizmo tool modes while an edit is in progress, the tool will switch, but the previous tools action will continue, which is a bit strange.

2024-09-03.12-44-45.mp4

This PR will prevent this, by committing the transform in the editor when switching tool modes which seems like the more expected and less confusing result.

2024-09-03.12-52-22.mp4

@ryevdokimov ryevdokimov requested a review from a team as a code owner January 7, 2024 16:04
editor/plugins/canvas_item_editor_plugin.cpp Outdated Show resolved Hide resolved
editor/plugins/canvas_item_editor_plugin.cpp Outdated Show resolved Hide resolved
editor/plugins/canvas_item_editor_plugin.cpp Show resolved Hide resolved
editor/plugins/canvas_item_editor_plugin.cpp Outdated Show resolved Hide resolved
editor/plugins/canvas_item_editor_plugin.cpp Outdated Show resolved Hide resolved
editor/plugins/canvas_item_editor_plugin.cpp Outdated Show resolved Hide resolved
editor/plugins/canvas_item_editor_plugin.cpp Outdated Show resolved Hide resolved
editor/plugins/canvas_item_editor_plugin.cpp Outdated Show resolved Hide resolved
@AThousandShips
Copy link
Member

When making several additions from suggestions please use the batch feature, it's next to the commit one 🙂

@ryevdokimov
Copy link
Contributor Author

When making several additions from suggestions please use the batch feature, it's next to the commit one 🙂

Thanks 😅

@ryevdokimov ryevdokimov force-pushed the commit-transforms-on-mode-switch branch 2 times, most recently from 0b273b6 to e683970 Compare January 7, 2024 16:57
@ryevdokimov
Copy link
Contributor Author

ryevdokimov commented Jan 7, 2024

Changes made and tested.

Thanks for your help AThousandShips.

Edit: Nevermind I missed one...

@ryevdokimov ryevdokimov force-pushed the commit-transforms-on-mode-switch branch from e683970 to 0748cbc Compare January 7, 2024 17:11
@ryevdokimov
Copy link
Contributor Author

Ok, I think it's good. Let me know if I missed anything.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants