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

Consistency in exiting menus #16765

Closed
ifreund opened this issue May 19, 2016 · 7 comments
Closed

Consistency in exiting menus #16765

ifreund opened this issue May 19, 2016 · 7 comments
Labels
Good First Issue This is a good first issue for a new contributor Info / User Interface Game - player communication, menus, etc.

Comments

@ifreund
Copy link
Contributor

ifreund commented May 19, 2016

The new menu that pops up when examining a toilet gives options to drink the liquid or pour it on the ground. However, unlike other menus in the game it has no cancel option. I think this should be added to be consistent with other menus, such as butchering or repairing clothing.

@mugling
Copy link
Contributor

mugling commented May 19, 2016

Try escape key, this is consistent with many other menus

@ifreund
Copy link
Contributor Author

ifreund commented May 19, 2016

I know that escape key works, I'm just saying that q for cancel should work as well

@mugling
Copy link
Contributor

mugling commented May 19, 2016

That behavior would cause problems in menu's that assign hotkeys based upon invlets (as q and c are both valid invlets). Esc doesn't have this problem

@ifreund
Copy link
Contributor Author

ifreund commented May 19, 2016

Then, for consistency, would it make more sense to remove the q cancel option from butchering and repairing? Or replace it with Esc cancel on all menus?

@mugling
Copy link
Contributor

mugling commented May 19, 2016

Yes, I think we should aim for consistency and using Esc guarantees no conflicts with existing hotkeys (however they are allocated)

@ifreund ifreund changed the title Lack of cancel option in new liquid menu Consistency in exiting menus May 20, 2016
@ifreund
Copy link
Contributor Author

ifreund commented May 20, 2016

So moving forward, should we just remove the cancel option from menus that currently have it? Or replace it with ⎋? (I know that is unicode not ascii so that would be annoying.) If we remove the cancel option would it be unclear how to exit a menu without choosing a listed option?

@mugling mugling added Good First Issue This is a good first issue for a new contributor Info / User Interface Game - player communication, menus, etc. labels Jun 4, 2016
@illi-kun
Copy link
Contributor

Particular case of #12112.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Good First Issue This is a good first issue for a new contributor Info / User Interface Game - player communication, menus, etc.
Projects
None yet
Development

No branches or pull requests

3 participants