Merge remote-tracking branch 'origin/master' into fix/android/arrows-…

…longpress-menu
keymanapp · Aug 21, 2024 · 10f46f3 · 10f46f3
2 parents dc434a7 + 1d42c6e
commit 10f46f3
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 7 deletions.
diff --git a/web/src/engine/main/src/keymanEngine.ts b/web/src/engine/main/src/keymanEngine.ts
@@ -61,6 +61,10 @@ export default class KeymanEngine<
       return;
     }
 
+    if(!this.core.languageProcessor.mayCorrect) {
+      event.keyDistribution = [];
+    }
+
     if(this.keyEventRefocus) {
       // Do anything needed to guarantee that the outputTarget stays active (`app/browser`: maintains focus).
       // (Interaction with the OSK may have de-focused the element providing active context;

diff --git a/web/src/engine/osk/src/visualKeyboard.ts b/web/src/engine/osk/src/visualKeyboard.ts
@@ -929,13 +929,6 @@ export default class VisualKeyboard extends EventEmitter<EventMap> implements Ke
    * @returns
    */
   getSimpleTapCorrectionDistances(input: InputSample<KeyElement, string>, keySpec?: ActiveKey): Map<ActiveKeyBase, number> {
-    // TODO: It'd be nice to optimize by keeping these off when unused, but the wiring
-    //       necessary would get in the way of modularization at the moment.
-    // let keyman = com.keyman.singleton;
-    // if (!keyman.core.languageProcessor.mayCorrect) {
-    //   return null;
-    // }
-
     // Note:  if subkeys are active, they will still be displayed at this time.
     let touchKbdPos = this.getTouchCoordinatesOnKeyboard(input);
     let layerGroup = this.layerGroup.element;  // Always has proper dimensions, unlike kbdDiv itself.