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

Hand Move Offset enhancements #14603

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

Conversation

TheNinjaScout
Copy link

Walk/Run/Crouch animations have a new boolean parameter: "ApplyHandMoveOffsetWhenStationary", set to false by default

When set to true, the hands of the character will move into the position defined by "HandMoveOffset" even when it is standing still.

Additionally, the horizontal value of the Hand Move Offset parameter no longer changes sign when the character moves backwards.

To elaborate on that change mentioned at the bottom of the commit message, this is currently happens on the public Barotrauma branch when you decide you want to make a humanoid character walk with their arms forwards

Barotrauma.2024-09-10.02-16-41.mp4

While here's what happens with the modifications made in this PR (namely, checking to see if the character is flipped to decide the horizontal sign of the hand offset instead of the sign of the horizontal movement value)

Barotrauma.2024-09-10.02-22-16.mp4

And finally, an example of the new "ApplyHandMoveOffsetWhenStationary" parameter at work:

Barotrauma.2024-09-10.02-26-33.mp4

Walk/Run/Crouch animations have a new boolean parameter: "ApplyHandMoveOffsetWhenStationary", set to false by default

When set to true, the hands of the character will move into the position defined by "HandMoveOffset" even when it is standing still.

Additionally, the horizontal value of the Hand Move Offset parameter no longer changes sign when the character moves backwards.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant