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

Fix scale sensitivity for 3D objects #52685

Merged
merged 1 commit into from
Sep 15, 2021

Conversation

lentsius-bark
Copy link
Contributor

The master version of #52665

Scaling 3D objects has been broken for a long time, it works well only when scale values stay around their original ones. Meaning that if you try scaling an objects that already at scale 30, the scaling function becomes unusable. Simple as that.

With this PR the motion vector is divided by the distance of the mouse click from the object. This is neat as it gives users the ability to choose scaling sensitivity in a very simple way: "The further from the object's center you mouse is, the slower it scales and vice versa"

@lentsius-bark lentsius-bark requested a review from a team as a code owner September 14, 2021 22:13
@lentsius-bark lentsius-bark changed the title Fix issues with scaling Node3Ds Fix scale sensitivity for 3D objects Sep 14, 2021
@Calinou Calinou added this to the 4.0 milestone Sep 14, 2021
Copy link
Member

@fire fire left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

\o/

@akien-mga akien-mga merged commit 5139f18 into godotengine:master Sep 15, 2021
@akien-mga
Copy link
Member

Thanks! And congrats on your first second merged Godot contribution 🎉

@lentsius-bark
Copy link
Contributor Author

@akien-mga thanks! The first one was from the days when I didn't even know what a "double" was. Remember doing a heavy google search.

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.

5 participants