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

Jumping mobs now make any mobs buckled to them jump #16970

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Sun-Soaked
Copy link
Contributor

@Sun-Soaked Sun-Soaked commented Jan 22, 2025

About The Pull Request

Mobs buckled to a jumping mob are now compelled to jump at the same time as their carrier.
This looks much better and allows mobs with a rider to pull off much fancier maneuvers.

Calling the component directly like this is total slop but this is how it's done everywhere else so
Obliterated. Lumi helped me find a better solution, jump component now iterates through mobs buckled to the jumper and forces them to jump.

Why It's Good For The Game

but-heres-the-jumper-jumping-cat

Changelog

🆑
qol: mobs buckled to a jumping mob now jump alongside them
/:cl:

@Sun-Soaked Sun-Soaked changed the title Xenos mounted on a crusher now jump in sync with them Jumping mobs now make any mobs buckled to them jump Jan 24, 2025
code/datums/components/jump.dm Outdated Show resolved Hide resolved
code/datums/components/jump.dm Outdated Show resolved Hide resolved
@Lumipharon Lumipharon added Feature New interesting mechanics with new interesting bugs Requires Testing PRs stable enough for testing, expecting minor changes. Test Merge Candidate This PR has been reviewed and is ready for testing, unless something changes. labels Jan 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature New interesting mechanics with new interesting bugs Requires Testing PRs stable enough for testing, expecting minor changes. Test Merge Candidate This PR has been reviewed and is ready for testing, unless something changes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants