Skip to content

Better input options #23

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

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

Better input options #23

wants to merge 13 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Feb 16, 2019

Created a new 'binding' system, that allows you to specify bind strings on the command lines instead of using numbers.

Example:
--moveBinding "left: X, right: B"
(press x or b to move)

Supports some logic too:
--resetBinding "left: joystick & right: b"
(press in the left joystick and b simultaneously to reset playspace)

It's pretty resilient to how the input strings are written. Will account for incorrect upper/lower case letters, varying whitespace and inconsistent separation.

Legacy support:
Old -l and -r commands automatically get converted to the new system.

And added flag --physicsStartOff for those who want it.

Terri00 added 13 commits October 28, 2018 23:53
Full changelog from 0.1.8 to 0.1.9:
+ added binding system
+ added binds:
   --moveBinding
   --togglePhysicsBinding
   --resetBinding

+ added flags:
   --physiscsStartOff

~ changed:
   default -l -r commands are now legacy
   cleaned up ull types to uint64_t
@naelstrof
Copy link
Owner

This is awesome! I'm going to have to look at it a bit later, and verify the documentation and the like.

Super cool though, thanks!

@ghost
Copy link
Author

ghost commented Feb 26, 2019

Nice :) The only thing I am not sure on is the Vive button names. Some of them are probably poorly named since I only have a Rift to work with.
The translations are defined in bindings.hpp in the bindings::controller_aliases map.

Cheers

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.

1 participant