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

'Search by key' button for the Keybind menu #6070

Merged
merged 2 commits into from
Apr 7, 2024

Conversation

SabreML
Copy link
Member

@SabreML SabreML commented Apr 2, 2024

About the pull request

Adds a 'search by key' button to the Keybind menu, which scrolls the window to the first entry bound to the inputted key.

Ideally this would somehow loop through everything that the key is bound to, but I haven't been able to figure that out yet.

Explain why it's good for the game

It's easy to search for the name of a keybind if you already know what you're looking for, but if for example you're trying to figure out what the 'F' key is doing, or whether or not a key you want to bind is already in use, then currently you need to look through the whole list manually.

Testing Photographs and Procedure

Screenshots & Videos
xWgBOBDqTk.mp4

Changelog

🆑
ui: Added a 'search by key' button to the Keybind menu.
/:cl:

@github-actions github-actions bot added the UI deletes nanoui/html label Apr 2, 2024
Copy link
Member

@harryob harryob left a comment

Choose a reason for hiding this comment

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

looks and works fine - but we might want to add a tooltip to the button to make it a little more intuitive for what it does

@Drulikar Drulikar added this pull request to the merge queue Apr 7, 2024
Merged via the queue into cmss13-devs:master with commit b0884d3 Apr 7, 2024
26 checks passed
cm13-github added a commit that referenced this pull request Apr 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
UI deletes nanoui/html
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants