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

Add hold esc for menu in options. #3731

Closed
LovelyA72 opened this issue Nov 16, 2018 · 4 comments · Fixed by #6166
Closed

Add hold esc for menu in options. #3731

LovelyA72 opened this issue Nov 16, 2018 · 4 comments · Fixed by #6166

Comments

@LovelyA72
Copy link
Contributor

LovelyA72 commented Nov 16, 2018

I asked some of my friends about this new feature. Most of them said it will be a little bit hard for old players to get used to this behavior. And it might causes note missing(or level fail). If there's a switch in options menu that can enable/disable this feature,it will be better.
Concern from #3686

@peppy
Copy link
Sponsor Member

peppy commented Nov 18, 2018

The addition of this is to dissuade pausing. We may add hotkeys to quick exit (like quick retry) to alleviate this.

@jorolf
Copy link
Contributor

jorolf commented Nov 19, 2018

How would you implement that button?
Just a single press quitting the player? Hold to quit? Maybe double tap?

  • A single press to quit is way to easy to hit accidentally (with pause this wasn't that much of an issue because you could continue)
  • Hold to quit is exactly what we're trying to bypass in the first place
  • Double tap seems like a good alternative but it is quite hidden for new/old players, you would have to mention that somewhere specifically

@peppy
Copy link
Sponsor Member

peppy commented Nov 20, 2018

Hold to quit, yes.

@LovelyA72
Copy link
Contributor Author

LovelyA72 commented Nov 21, 2018

I think there needs to have at least one way to instantly stop the current level instead of holding a key.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants