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

Allow customization of text input UI hotkeys #57266

Merged
merged 6 commits into from
May 1, 2022

Conversation

Qrox
Copy link
Contributor

@Qrox Qrox commented Apr 29, 2022

Summary

Interface "Allow customization of text input UI hotkeys using either ? or F1"

Purpose of change

Allow customizing the hotkeys of text input UIs.

Describe the solution

Various text input UIs in the game now handle input using actions instead of hardcoded hotkeys.

String input popup and string editor window now use F1 as the hotkey to bring up the keybindings menu to avoid conflicting with text input, but uilist, keybindings menu, and world creation menu still use the previous default of ?.

The hotkeys one most likely would want to change are those for the "clear text", "paste", and "input from file" actions, but other hotkeys are still changed into actions in case someone cannot type the default hotkeys.

Testing

Tested in the tiles and curses builds and in all the affected menus (string input popup, string editor window, keybindings menu, world creation menu, and uilist).

@github-actions github-actions bot added [C++] Changes (can be) made in C++. Previously named `Code` [JSON] Changes (can be) made in JSON Character / World Generation Issues and enhancements concerning stages of creating a character or a world Info / User Interface Game - player communication, menus, etc. json-styled JSON lint passed, label assigned by github actions astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions labels Apr 29, 2022
@dseguin dseguin merged commit 5489918 into CleverRaven:master May 1, 2022
@Qrox Qrox deleted the string-input-context branch May 2, 2022 05:38
} else if( action == "PAGE_UP" ) {
if( filtered_registered_actions.empty() ) {
// do nothing
} if( scroll_offset >= display_height ) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

else is missing here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions [C++] Changes (can be) made in C++. Previously named `Code` Character / World Generation Issues and enhancements concerning stages of creating a character or a world Info / User Interface Game - player communication, menus, etc. [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants