Remove redundant descriptions from key bindings #16220
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
y5nw
approved these changes
Jun 3, 2025
| keymap_toggle_fog (Toggle fog) key SYSTEM_SCANCODE_60 | ||
|
|
||
| # Key for toggling the display of debug info. | ||
| keymap_toggle_debug (Toggle debug info) key SYSTEM_SCANCODE_62 | ||
|
|
||
| # Key for toggling the display of the profiler. Used for development. |
There was a problem hiding this comment.
I wonder whether we can also remove the description for this one since; "toggle profiler" is IMO already clear enough.
Co-authored-by: y5nw <37980625+y5nw@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Goal
Goal: Remove redundant descriptions like "key for moving forward" on the keybind for "Move Forward". Leave important information like "Will also disable autoforward, when active." on keybinds that may need further clarification.
Resolves #16168
To do
Someone should probably look through the settings that were changed and double check, what might be self explanatory to one person might not be to others. Please let me know if you want any of the descriptions added back!
This PR is Ready for Review.
How to test
Build and launch the project.
Check the settings menu and see that the keybinds that are self explanatory have no information string associated with them.
Profit!