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.

Sign up for GitHub

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

Jump to bottom

[Manual] Clarify use of delta with move_and_slide #9060

Merged
merged 1 commit into from
Mar 6, 2024
Merged

[Manual] Clarify use of delta with move_and_slide #9060

merged 1 commit into from
Mar 6, 2024

Conversation

AThousandShips
Copy link
Member

Added details to clarify that while velocity should not be multiplied by delta, gravity still should.

@AThousandShips
[Manual] Clarify use of delta with move_and_slide
c6bf5d4 
Added details to clarify that while `velocity` should not be multiplied
by `delta`, `gravity` still should.
@AThousandShips AThousandShips added enhancement area:manual Issues and PRs related to the Manual/Tutorials section of the documentation topic:physics cherrypick:4.1 cherrypick:4.2 labels Mar 6, 2024
@AThousandShips AThousandShips requested a review from skyace65 March 6, 2024 11:48
@mhilbrunner mhilbrunner merged commit 1367bdb into godotengine:master Mar 6, 2024
1 check passed
@AThousandShips AThousandShips deleted the gravity_note branch March 6, 2024 14:24
@mhilbrunner
Copy link
Member

Thanks! Merged.

@AThousandShips
Copy link
Member Author

Thank you!

@mhilbrunner
Copy link
Member

Cherry-picked to 4.2 in #9647.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Piralein Piralein Piralein approved these changes

@mhilbrunner mhilbrunner mhilbrunner approved these changes

@skyace65 skyace65 Awaiting requested review from skyace65

Assignees
No one assigned
Labels
area:manual Issues and PRs related to the Manual/Tutorials section of the documentation cherrypick:4.1 enhancement topic:physics
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

CharacterBody2D move_and_slide example is using delta, despite warning not to use it
3 participants
@AThousandShips @mhilbrunner @Piralein