-
-
Notifications
You must be signed in to change notification settings - Fork 108
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
fix(web): fix osk touch-focus tracking #11705
Conversation
Fixes: #11704 Also addresses _part_ of KEYMAN-WEB-HY, but not all variants.
User Test ResultsTest specification and instructions
Test Artifacts
|
Test Results
|
@@ -13,10 +13,10 @@ export default class TouchEventPromiseMap { | |||
} | |||
|
|||
public maintainTouches(list: TouchList) { | |||
let keys = [].concat(this.map.keys()); | |||
let keys = Array.from(this.map.keys()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does #11732 have any impact on this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is outside of the worker.
We keep es6-shim around on Android and use it for devices with outdated WebViews; it'll cover this case well enough.
Changes in this pull request will be available for download in Keyman version 18.0.51-alpha |
Fixes: #11704
Fixes: KEYMAN-WEB-HY. Turns out that the original variant of that issue was resolved with #11505, leaving only the current form in place. (Note: is not currently commit-tagged; I realized this later.)
The offending code attained its current form in #11462, wherein I converted this class from using a
Record<,>
Object-based "map" to an actual ES6Map
.Before:
Object.keys()
.After:
map.keys()
.Object.keys()
actually does return an array, whilemap.keys()
does not. Fortunately, this means that the bug this fixes should only occur for 18.0-alpha.User Testing
TEST_ATTEMPT_REPRO: Using Keyman for Android, attempt to reproduce #11704: