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

Do not commit gizmo handles if no changes were made #102317

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

Conversation

ryevdokimov
Copy link
Contributor

Unreverts the 3d gizmo handle part of: #87161

#94684 partially reverted the above-mentioned PR, because it ended up breaking SoftBody3D pinned points interaction, but reintroduced the problem in all other 3D gizmos. This PR fixes the problem by making SoftBody3DGizmoPlugin an exception to the mouse movement requirement to interact with gizmo handles.

@ryevdokimov ryevdokimov requested a review from a team as a code owner February 2, 2025 04:18
@ryevdokimov ryevdokimov force-pushed the gizmo-commit-handle-spam-fix branch from 950541e to 85993fc Compare February 2, 2025 04:35
@Chaosus Chaosus added this to the 4.4 milestone Feb 2, 2025
@ryevdokimov ryevdokimov force-pushed the gizmo-commit-handle-spam-fix branch from 85993fc to d71a3c1 Compare February 2, 2025 08:29
@Repiteo Repiteo modified the milestones: 4.4, 4.5 Feb 24, 2025
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.

3 participants