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

Added grappling movement provider and fixed disabled/bypassed provider signals #126

Merged
merged 1 commit into from
May 16, 2022

Conversation

Malcolmnixon
Copy link
Collaborator

This pull request implements feature request #122 by adding the grappling movement provider based on code from teddybear082.

Additionally this pull request contains a fix to how movement providers are serviced. The change allows an active movement provider to detect:

  • When they have been disabled while active
  • When they have been preempted by a higher-priority movement provider that has taken exclusive control of movement

In these circumstances the movement provider is able to deactivate itself sending any finished signals - for example to disable movement sound effects. This was found when firing the grappling hook while gliding and the gliding sound continued to play even though grappling motion had taken over.

… signals if disabled or bypassed.

Added grapple movement provider.
@BastiaanOlij
Copy link
Member

I'll give this a test tomorrow, looking forward to swinging around :)

@BastiaanOlij BastiaanOlij linked an issue May 9, 2022 that may be closed by this pull request
@BastiaanOlij
Copy link
Member

Just saw the video, excellent work: https://www.youtube.com/watch?v=4R-h3a0wcPY

Let me know if you're ready for me to merge this Malcolm?

@Malcolmnixon
Copy link
Collaborator Author

Yup, this one's ready to go in. Like everything there's bound to be feature requests and improvements ;)

@BastiaanOlij BastiaanOlij merged commit ca29836 into GodotVR:master May 16, 2022
@Malcolmnixon Malcolmnixon deleted the grappling branch May 16, 2022 22:34
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.

Feature Request - Add Simple "Grapple Hook Movement" Functionality
2 participants