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

Game uses default direction buttons for actions instead of user-configured ones #234

Closed
Wuzzy2 opened this issue Mar 21, 2013 · 2 comments

Comments

Projects
None yet
3 participants
@Wuzzy2
Copy link
Contributor

commented Mar 21, 2013

On all actions which require a direction key to be pressed afterwards (like examine) the game only accepts the default direction keys. The customized controls are ignored. This is a major pain in the ass because this means you have two different keybindings for basicly the same thing: direction.

I have discovered this bug because I changed "Move Northwest" from [y] to [z] because I have a QWERTZ keyboard, not a QWERTY keyboard.
Don't get me wrong: For the normal player movement, all the move keys remappings work properly (I have tested it by remapping every move key). But for any action other than moving the player, the game forces you to use the default ones.

This has been tested with following actions:

  • open
  • close
  • smash
  • examine
  • construction
    I am too lazy to test every possible action, probably this issue affects every action which require a direction key except moving.

Of course this creates an inconsistence in the gameplay. The direction keys used by the action keys should be the same than those configured by the player for moving the character.

This bug can still be observed in the version after commit ea4c9b3.

@ethankaminski

This comment has been minimized.

Copy link
Contributor

commented May 14, 2013

This is presumably a result of having two separate input-handling files, input.cpp and keypress.cpp. The former is hardcoded, the latter seems to be where remaps happen.

@minakitty

This comment has been minimized.

Copy link
Contributor

commented Sep 8, 2013

9bc757c still exists

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.