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

Touch keyboard tweaks and fix #975

Merged
merged 2 commits into from
Dec 12, 2019
Merged

Conversation

wadetb
Copy link
Contributor

@wadetb wadetb commented Dec 11, 2019

Hi, this is two separate changes to the touch keyboard. They can be separated and taken independently if you want.

The first is some usability stuff I encountered - making the keyboard appear at startup and after resume, and squelching the keypress that makes it appear.

The second is an issue I saw people talking about in Discord and is a pretty straightforward bug.

Makes the keyboard appear at startup, and after wakeup from suspend or
activity switch. Fixes an odd first impression - "type help for help"
"but how?".

Prevents the first touch that makes the keyboard appear from generating
a key, same for gamepad. Sorry the code is a bit tortured.
When Ctrl+R is used to start run mode, the touch keyboard is disabled.

With no updates, the Ctrl or R keys were never marked as released,
and the Ctrl key being held prevented the Escape key shortcut from
activating when Back was pressed.

Now, all touch state is cleared regardless of the mode.
@StinkerB06
Copy link

Also, can you please change it, so that using input: keyboard metadata will cause the keyboard to show up instead of the gamepad?

@wadetb
Copy link
Contributor Author

wadetb commented Dec 11, 2019

Sure, I'll look into that when I have some more time.

Copy link
Owner

@nesbox nesbox left a comment

Choose a reason for hiding this comment

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

👍

@nesbox nesbox merged commit 744e04d into nesbox:master Dec 12, 2019
@nesbox nesbox added bug enhancement Improvement of existing feature or adding something new labels Dec 12, 2019
@nesbox nesbox added this to the 0.80.0 milestone Dec 12, 2019
@nesbox nesbox removed this from the 0.80.0 milestone Jul 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug enhancement Improvement of existing feature or adding something new
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants