search

keymanapp / keyman

Keyman cross platform input methods system running on Android, iOS, Linux, macOS, Windows and mobile and desktop web
https://keyman.com/
Other
372 stars 102 forks source link

bug(web): sticky backspace button in Keyman Android 17.0.307-beta #11223

Closed alex-larkin closed 1 week ago

alex-larkin commented 1 month ago

Describe the bug

Sometimes after selecting text and deleting it, the Backspace button remains active, continuing to quickly delete whatever text was before the selection, one character at a time. This seems to happen when pressing Backspace slightly longer than a tap, but not for too long.

The active Backspace button can be deactivated by touching it again (but often not before it's too late).

See video:

https://drive.google.com/file/d/1vtxc2-0JskRnKSBC3ibM1gnq6EHm4_bL/view?usp=sharing (Especially from 0:27 to the end.)

This is my project folder, if needed:

https://drive.google.com/file/d/1jIJqt_a1e3Gn9WS6iePrsg07xjTHE-F2/view?usp=sharing

Reproduce the bug

  1. Open a keyboard run by Keyman Beta for Android 17.0.307
  2. type out a string of characters
  3. Select the last bit of the characters with a long tap
  4. Tap Backspace. (Try a slightly longer tap. But not too long of a tap.)
  5. Sometimes, the Backspace button will remain active, deleting the selection, and then the text before it, character by character.
  6. This will continue until all the text is deleted or until tapping Backspace again, whichever happens first.

I was able to reproduce this behavior about 3 out of 6 times.

Expected behavior

Tapping Backspace only triggers one deleting event. The key should not remain active.

Related issues

No response

Keyman apps

Keyman version

17.0.307

Operating system

Android 12

Device

Sansung Galaxy S21

Target application

The notepad in the Keyman Beta app

Browser

No response

Keyboard name

teclado_quechua

Keyboard version

0.9

Language name

que

Additional context

No response

jahorton commented 1 month ago

I'm curious if this might be related to #11221 - as if the backspace key handling is terminating too early. I'll add a test to #11245 for this, to explore the possibility.

alex-larkin commented 1 month ago

Thanks!

jahorton commented 3 weeks ago

Could you try again with the latest version and let us know if the behavior is fixed now? Thanks.

alex-larkin commented 3 weeks ago

Thanks for your work. I just tested with keyman-17.0.311.apk beta. While I did not observe the same sticky behavior, I did observe that all keyboards exhibited intermittent non-funcionality for backspace key. Tapping the backspace key would make it flash blue, but nothing would be deleted. Especially when the cursor was at or towards the end of the text.

No issues observed with the keyboard named Winchus in the recording, which is effectively the same as "Teclado Quechua" but running on the current version 16 production release.

See this screen recording:

https://drive.google.com/file/d/1-Dr00-JJgVP04iZfPbXxU6gGbvBZ8Jqm/view?usp=drivesdk

Thanks so much.

jahorton commented 3 weeks ago

Thanks for your work. I just tested with keyman-17.0.311.apk beta. While I did not observe the same sticky behavior, I did observe that all keyboards exhibited intermittent non-funcionality for backspace key. Tapping the backspace key would make it flash blue, but nothing would be deleted. Especially when the cursor was at or towards the end of the text.

No issues observed with the keyboard named Winchus in the recording, which is effectively the same as "Teclado Quechua" but running on the current version 16 production release.

See this screen recording:

https://drive.google.com/file/d/1-Dr00-JJgVP04iZfPbXxU6gGbvBZ8Jqm/view?usp=drivesdk

Thanks so much.

About this: we've recently noticed a bug with a non-functional backspace that might be related, though it's hard to tell if this is the same thing you're experiencing without full details about your document. I do see that it's fairly large and does seem to have been copied and pasted from a source with links, so... there's a decent chance this may be relevant. Does #11182 seem like it could apply to your scenario? We're working on a fix for issue that already with #11206.

jahorton commented 3 weeks ago

Either way, if it's not too much to ask, a replica of your document at the start of the video would be helpful for attempting to reproduce this ourselves. Even if it is a match for #11182, it'd still be useful for testing our pending fix.

Noting some of the text visible in the video... is this basically an except of https://help.keyman.com/keyboard/ekwtamil99uniext/1.0/chart.pdf? Granted, it's hard to tell where in the PDF the excerpt starts and ends.

jahorton commented 3 weeks ago

11206 got merged yesterday, which is the fix I referred to with the prior comment. Could you try again with the latest beta build and see if that fixes things? Also, it'd help to have specific data to test with if you can reliably reproduce it yourself.

jahorton commented 1 week ago

We believe that this has been fixed at this point, and no further reports have come in. We'll go ahead and close this issue - we can always link it in any new bug reports in the worst-case.

alex-larkin commented 1 week ago

Thanks so much! I'm sorry for not having responded. I was in the most of a move and then my wife got sick. (Doing better now!) Sounds good.