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

Making smart holstering and trying to keybind a reload button [do NOT merge] #3229

Closed
wants to merge 3 commits into from

Conversation

OverDriveZ
Copy link
Contributor

About The Pull Request

This does a lot of things.
To begin with, this is supposed to unholster your main weapon using a keybind, prioritising between the following slots in this order:
-Armor slot,
-Belt slot,
-Holster.

If the same keybind is being pushed again, and there's a weapon in your hands, then the system will return to your free hand the speedloader (magazine or clip, whatever it's needed) with the highest amount of ammo. (Maybe making it smart to fetch the right kind of ammo the gun needs?

Here's a simple example of what I did:

de449bc2f1d2413e19a20c4dc3317471.mp4

To-Do Checklist

  • Making it fetch a weapon with a priority.
  • Making it fetch the fullest ammo clip.
  • Making it fetch the matching kind of ammo for the gun you have in your other hand.

Changelog

🆑
add: Adds smart holstering.
/:cl:

@OverDriveZ OverDriveZ marked this pull request as draft September 30, 2023 21:50
@Tk420634
Copy link
Contributor

Tk420634 commented Oct 2, 2023

smolstering

@OverDriveZ
Copy link
Contributor Author

Iteration II does this now:

8d35504506048a5d0786d5dd78375d86.mp4

This now lets you akimbo stuff
@OverDriveZ
Copy link
Contributor Author

Now it lets you akimbo

a54afa4ac02c99c56cd28956cdd41603.mp4

@OverDriveZ OverDriveZ closed this Oct 3, 2023
@OverDriveZ
Copy link
Contributor Author

professionist made a better version of this

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.

2 participants