Skip to content

Change default inventory key to E, default Aux1 key to F #16053

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

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

minlemon
Copy link
Contributor

To do

This PR is Ready for Review.

  • Check there are no other places the default keybinding for inventory, Aux1 is specified
  • After merging: update docs page /for-players/controls/

How to test

Launch a new game that uses inventory and Aux1 (e.g. VoxeLibre). Press E for inventory. Hold F to run.

@y5nw y5nw added the UI/UX label Apr 21, 2025
@Zughy Zughy added Feature ✨ PRs that add or enhance a feature Roadmap: Needs approval The change is not part of the current roadmap and needs to be approved by coredevs beforehand labels Apr 21, 2025
@sfan5 sfan5 self-requested a review April 22, 2025 08:18
@lhofhansl
Copy link
Contributor

IMHO... why change the default? If folks want to change it, they have the option.

@minlemon
Copy link
Contributor Author

IMHO... why change the default? If folks want to change it, they have the option.

Please review the discussion in #16052. The short answer is that the default inventory key of "I" cannot be pressed without looking down at the keyboard and Luanti should come with solid defaults. As a case study Minecraft started with inventory key being "I" but switched to "E" after user feedback.

@sfan5 sfan5 added this to the 5.12.0 milestone Apr 26, 2025
@sfan5
Copy link
Collaborator

sfan5 commented Apr 26, 2025

Adding to milestone, because I think we should decide this before release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
@ Client / Controls / Input Feature ✨ PRs that add or enhance a feature Roadmap: Needs approval The change is not part of the current roadmap and needs to be approved by coredevs beforehand UI/UX
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Default inventory key "I" should be changed to something more natural
6 participants