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

TTT: Fix enter button for disguiser #1297

Merged
merged 1 commit into from Dec 17, 2016

Conversation

Projects
None yet
4 participants
@markusmarkusz
Contributor

markusmarkusz commented Dec 16, 2016

Calling WEPS.DisguiseToggle(ply) in server state doesn't make any sense.
It has to be client side.

@robotboy655 robotboy655 added the TTT label Dec 16, 2016

@Kefta

This comment has been minimized.

Contributor

Kefta commented Dec 16, 2016

It used to work -- what commit broke it?

@markusmarkusz

This comment has been minimized.

Contributor

markusmarkusz commented Dec 16, 2016

It seems like that this is a issue for a long time. (This function is the same function like 4 years ago.)

@Kefta

This comment has been minimized.

Contributor

Kefta commented Dec 16, 2016

Yeah, I don't remember it working properly since GM12. Always thought it was my numpad

@svdm

This comment has been minimized.

Collaborator

svdm commented Dec 17, 2016

Judging from the comment it used to be in the KeyRelease hook. I vaguely remember something changing about (numpad) key handling/bindings in a gmod update long ago, probably was moved out at that point and broken either then or later.

@svdm svdm merged commit ade3a7e into Facepunch:master Dec 17, 2016

@markusmarkusz markusmarkusz deleted the markusmarkusz:patch-9 branch Feb 26, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment