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

specialOne/Two/Three alias for f/g/h #2

Merged
merged 1 commit into from Jan 14, 2017
Merged

Conversation

Silverfeelin
Copy link
Owner

Input options specialOne, specialTwo and specialThree added.

  • These values reflect the values for the input options f, g and h respectively.

This increases clarity for the end user, since users may not have their controls for the special actions set to F, G and H.
The old input options are still present, to keep the mod compatible with older implementations. Even though they are still present, it is discouraged to use f, g and/or h.

Example:
User has PlayerTechAction2 bound to V rather than G.
The keybind would still read g, so the user expects they must press G to call the function, instead of the correct V.

Input options `specialOne`, `specialTwo` and `specialThree` added. These values reflect the values for the input options `f`, `g` and `h` respectively.

This is done to increase clarity, since users may not have their controls for the special actions set to <kbd>f</kbd>, <kbd>g</kbd> and <kbd>h</kbd>. The old input options are still present, to keep the mod compatible with older implementations. It is discouraged to use `f`, `g` and/or `h` at this point.

Example: User has special action two bound to <kbd>V</kbd> rather than <kbd>G</kbd>. The keybind would still read `g`, so the user expects they must press <kbd>g</kbd> to call the function.
@Silverfeelin Silverfeelin merged commit 899f48f into master Jan 14, 2017
@Silverfeelin Silverfeelin deleted the Silverfeelin-patch-1 branch January 14, 2017 20:11
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.

None yet

1 participant