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 roll state to getPedMoveState while crouching or crawling #1635

Closed
wants to merge 3 commits into from

Conversation

Unde-R
Copy link
Contributor

@Unde-R Unde-R commented Aug 24, 2020

Looks like the movement was declared but no code for It.

@patrikjuvonen patrikjuvonen added the enhancement New feature or request label Sep 14, 2020
@StrixG StrixG added this to the Confirmed Issues milestone Oct 25, 2020
@qaisjp qaisjp removed this from the Confirmed Issues milestone Dec 7, 2020
@patrikjuvonen patrikjuvonen self-requested a review January 16, 2021 22:48
Copy link
Contributor

@patrikjuvonen patrikjuvonen left a comment

Choose a reason for hiding this comment

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

At the moment this shows MOVEMENTSTATE_ROLL is activated when the ped is not actually rolling.

Steps to reproduce:

  • Have fists (or any non-roll inducing weapon) equipped
  • Hold right mouse and walk right/left, it shows roll while you're crawling

Furthermore: what are potential roll-preventing features in GTA/MTA? At least while you're in vehicle getPedMoveState is not available, but how about when you force an animation while crouched and try to roll? What other potential things can break this? I suggest try to break this check in as many ways as possible, thanks 👍

A test resource would be good too :)

Additionally, if you stop pressing any of the keys, the movement state returned is no longer roll, even though the roll animation is still playing.

This doesn't cover all cases it seems.

@patrikjuvonen patrikjuvonen marked this pull request as draft January 17, 2021 11:11
@github-actions
Copy link
Contributor

github-actions bot commented Jan 7, 2022

This draft pull request is stale because it has been open for at least 90 days with no activity. Please continue on your draft pull request or it will be closed in 30 days automatically.

@github-actions github-actions bot added the stale Inactive for over 90 days, to be closed label Jan 7, 2022
@github-actions github-actions bot closed this Feb 7, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Feb 7, 2022

This draft pull request was closed because it has been marked stale for 30 days with no activity.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request stale Inactive for over 90 days, to be closed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants