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

bug(android): Holding backspace appears to desync context between web and app. #7172

Closed
Tracked by #7161
mcdurdin opened this issue Aug 30, 2022 · 2 comments · Fixed by #7156
Closed
Tracked by #7161

bug(android): Holding backspace appears to desync context between web and app. #7172

mcdurdin opened this issue Aug 30, 2022 · 2 comments · Fixed by #7156

Comments

@mcdurdin
Copy link
Member

Describe the bug

From a team review of the Keyman for Android UX (#7161)

Holding backspace appears to desync context between web and app. After the first backspace, subsequent letters are not deleted from the textarea, but holding for a few seconds shows the keyboard going to upper case, suggesting web thinks it is at start of context. After that, even tapping the backspace key then does nothing, suggesting same thing.

Per @jahorton, does not appear to repro on iOS (15.0.268-stable).

Expected behavior

Backspace should work. Holding backspace should delete more rapidly.

Screenshots

Screen_Recording_20220831-060939_Keyman.mp4

Related issues


Keyman for Windows/macOS/Linux/iPhone/iPad/Android:

  • Device: Samsung A90 5G
  • OS: Android 12
  • Keyman version: 15.0.269-stable
@mcdurdin
Copy link
Member Author

I tried removing onLongPress from KMKeyboard and the problem went away. So it's a conflict between the Android gesture recognizer and the KeymanWeb code. Next question is: when did this arise -- and why? I may do a bisect to find out.

@mcdurdin
Copy link
Member Author

mcdurdin commented Sep 1, 2022

Fixed by #7156.

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.

1 participant