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

[WIP] Added optional keybinds support. #381

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

Conversation

n0bodysec
Copy link
Collaborator

@n0bodysec n0bodysec commented Jun 12, 2019

Ghetto and shitty way, but works.

Maybe I can improve this code even more.
Suggestions are appreciated. I don't like my shitty method.

I want to do it without touching SDK files.

Support for things like PR #127.

I need to disable pause menu when pressing "ESC". Any ideas?

@LWSS
Copy link
Owner

LWSS commented Jun 12, 2019

this will break it for anyone using delete

@n0bodysec
Copy link
Collaborator Author

That's the point. Who the fck use DEL anyway? I think DEL is actually bugged anyway.

@FilipeSilvens
Copy link
Collaborator

FilipeSilvens commented Jun 13, 2019

you can just do this in the SetKeyCodeState hook like i do

	if(code == ButtonCode_t::KEY_ESCAPE)
		*SetKeyCodeState::keyOutput = ButtonCode_t::KEY_NONE;
	else
		*SetKeyCodeState::keyOutput = code;

@n0bodysec
Copy link
Collaborator Author

Yeah I know my method is a shit. Thanks FilipeSilvens, I like your method.

@n0bodysec n0bodysec changed the title Added optional keybinds support. [WIP] Added optional keybinds support. Jun 14, 2019
Ghetto and shitty way, but works.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants