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

Add Mass Distribution, Deactivation, Solver inspector property groups #77943

Merged
merged 1 commit into from
Aug 17, 2023

Conversation

aaronfranke
Copy link
Member

@aaronfranke aaronfranke commented Jun 7, 2023

RigidBody(2D/3D) is a complex class with many properties, so I think it's worth making some sections.

  • The Mass Distribution section has properties for how mass is distributed: center of mass and inertia.
  • The Deactivation section has properties for sleeping, locking, and freezing.
  • The Solver section has properties for how the physics simulation solves/calculates things, like integration, continuous collision, and if the contact monitor is enabled.
  • The mass, physics material, and gravity scale properties stay top-level since they are common to use and easy to understand for novice users.

When expanded, it looks like this:

Screenshot 2023-08-06 at 9 28 05 PM Screenshot 2023-08-06 at 9 30 50 PM

When collapsed, it looks like this, very tidy:

Screenshot 2023-08-06 at 9 28 23 PM Screenshot 2023-08-06 at 9 31 00 PM

@YuriSizov YuriSizov requested a review from a team July 7, 2023 12:27
@aaronfranke aaronfranke changed the title Add Deactivation and Mass Distribution inspector property groups Add Mass Distribution, Deactivation, Solver inspector property groups Aug 7, 2023
Copy link
Member

@akien-mga akien-mga left a comment

Choose a reason for hiding this comment

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

Seems fine to me.

@akien-mga akien-mga merged commit e6e7925 into godotengine:master Aug 17, 2023
@akien-mga
Copy link
Member

Thanks!

@aaronfranke aaronfranke deleted the mass-dist branch August 17, 2023 15:05
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