-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Allow wieldable guns to cycle in hand #28002
Conversation
…le other interactions
Hey WarMechanic, thanks a bunch for your original PR. Im currently cleaning up gun yaml right now and im actually moving the stuff from base_wieldable.yml into a base_gun.yml with a bunch of other inheritable qualites. You want me to add this change in there? |
Not saying you should close this. This will definitely take less review before merging and its sounds like a good change. If it gets merged before my thing Ill make the changes that I need to. |
This sounds great, I'm pretty sure metalgearsloth had some concerns about value inheritance so that would probably clean up everything nicely. |
Do I understand correctly that this makes the only way to unwield the gun right click? This would be a significant QOL downgrade for reloading, since now to get a free hand to grab your new magazine, instead of just pressing Z and X you have to toggle off harm mode, bring the mouse down to the hand windows to right-click the gun and find Unwield. This is significantly slower and more distracting, requiring the player to divert both their mouse and their gaze away from the action. In the middle of a firefight. I don't think this feature should go through like this without addressing the issue it causes. I have seen it mentioned that switching hands while Wielding could unwield, that may be a good solution? |
I have been informed you can switch to the second hand and press Drop to unwield. But by then I coded the QOL thing I mentioned, anyway |
The current state of unwielding is terrible QoL for shotguns that need you to manually cycle after every shot. Where the natural course would be to interact in-hand to cycle the shot as you used to, you need to alt-click the gun to cycle it which takes much more effort. This PR addresses that, and allows you to Wield, Cycle, Eject and Unwield (through dropping offhand) purely through keybinds.
I think this could use a bit more discussion, because I believe that the off-hand wield could be used in the future for underbarrel weapons like the M-90 GL. |
if that forum suggestion of wielding being a passive thing when you have a free hand was implemented, pressing z would always cycle and there would be no unwield so nothing to conflict with |
I think this would essentially create tech debt, because it would cement a solution where only two-handed weapons can have such alternate fire, and it only supports exactly one alternate fire per weapon. So the moment anyone would want to go further than that, they would need to create a new, more accomodating solution anyway |
we already have a form of alt fire, but it's for battery weapons |
About the PR
Applies #27307 to #26970
Why / Balance
At the moment, interacting in-hand while a gun wields it proceeds to unwield the gun rather than cycle it. Gun interactions can become click interactive and unintuitive, especially with pump action shotguns.
Technical details
Applies the new variable to the wieldable gun parent rather than to multiple guns, attempts to address the concern of applying variables en masse.
Changelog
🆑