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(web): undefined reading 'deleteLeft' causing crash in Android app #12494

Open
darcywong00 opened this issue Oct 2, 2024 · 2 comments
Open
Assignees
Milestone

Comments

@darcywong00
Copy link
Contributor

darcywong00 commented Oct 2, 2024

@dinakaranr recorded and reported this issue using the current Keyman for Android 18.0 alpha
(I can repro on 18.0.117-alpha)

Steps to repro:

  1. In the Keyman app with the default sil_euro_latin keyboard
  2. Longpress g -->
  3. Longpress h -->
  4. Longpress j --> ĵ
  5. Flick down on c --> '
  6. Flick down on v --> Javascript error

The chrome dev tools gives the following error

@keymanapp/keyman/we…es/src/common.ts:40 Uncaught (in promise) TypeError: Cannot read properties of undefined (reading 'deleteLeft')
    at buildMergedTransform (@@keymanapp/keyman/web/src/engine/predictive-text/templates/src/common.ts:40:43)
    at ContextTracker.attemptMatchContext (@keymanapp/keyman/web/src/engine/predictive-text/templates/src/common.ts:444:57)
    at _ContextTracker.analyzeState (@keymanapp/keyman/web/src/engine/predictive-text/worker-thread/src/main/correction/context-tracker.…:673:37)
    at @keymanapp/keyman/we…t-helpers.ts:175:48
    at Generator.next (<anonymous>)
    at blob:null/1b8791be-3…-49f3ec17d480:49:63
    at new Promise (<anonymous>)
    at __async (blob:null/1b8791be-3…-49f3ec17d480:33:12)
    at correctAndEnumerate (@keymanapp/keyman/we…ct-helpers.ts:103:4)
    at _ModelCompositor.<anonymous> (@keymanapp/keyman/we…ompositor.ts:122:56)
@darcywong00
Copy link
Contributor Author

darcywong00 commented Oct 2, 2024

His video was too large to attach to GH so I will split it...

Uploading output_video.mp4…

@ermshiperete
Copy link
Contributor

ermshiperete commented Oct 2, 2024

To get the error it is sufficient to do steps 5 and 6. Same thing happens with KeymanWeb test page.

Note that you'll have to switch keyboards to get the error again after it happened.

@mcdurdin mcdurdin added this to the A18S19 milestone Nov 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: No status
Development

No branches or pull requests

4 participants