fix: account for ephemeral focus in enter precondition

Previously bugs could occur if the field editor contained a button but did
prevent keydown events propagating.

Author: microbit-matt-hillsdon

src/actions/enter.ts

@@ -64,7 +64,8 @@ export class EnterAction {
             const targetWorkspace = workspace.isFlyout
               ? workspace.targetWorkspace
               : workspace;
-            return !!targetWorkspace && !targetWorkspace.isReadOnly();
+            if (!targetWorkspace) return false;
+            return this.navigation.canCurrentlyEdit(targetWorkspace);
           }
           default:
             return false;
@@ -111,6 +112,8 @@ export class EnterAction {
    * @returns True if the enter action should be handled.
    */
   private shouldHandleEnterForWS(workspace: WorkspaceSvg): boolean {
+    if (!this.navigation.canCurrentlyNavigate(workspace)) return false;
+
     const cursor = workspace.getCursor();
     const curNode = cursor?.getCurNode();
     if (!curNode) return false;

src/navigation.ts

@@ -96,7 +96,12 @@ export class Navigation {
    * @returns The state of the given workspace.
    */
   getState(): Constants.STATE {
-    const focusedTree = Blockly.getFocusManager().getFocusedTree();
+    const focusManager = Blockly.getFocusManager();
+    if (focusManager.ephemeralFocusTaken()) {
+      return Constants.STATE.NOWHERE;
+    }
+
+    const focusedTree = focusManager.getFocusedTree();
     if (focusedTree instanceof Blockly.WorkspaceSvg) {
       if (focusedTree.isFlyout) {
         return Constants.STATE.FLYOUT;
@@ -837,11 +842,7 @@ export class Navigation {
       workspace.targetWorkspace ??
       workspace
     ).keyboardAccessibilityMode;
-    return (
-      !!accessibilityMode &&
-      this.getState() !== Constants.STATE.NOWHERE &&
-      !Blockly.getFocusManager().ephemeralFocusTaken()
-    );
+    return !!accessibilityMode && this.getState() !== Constants.STATE.NOWHERE;
   }
 
   /**
@@ -854,7 +855,7 @@ export class Navigation {
    * @returns whether keyboard navigation and editing is currently allowed.
    */
   canCurrentlyEdit(workspace: Blockly.WorkspaceSvg) {
-    return this.canCurrentlyNavigate(workspace) && !workspace.options.readOnly;
+    return this.canCurrentlyNavigate(workspace) && !workspace.isReadOnly();
   }
 
   /**

test/webdriverio/test/basic_test.ts

@@ -375,15 +375,14 @@ suite('Keyboard navigation on Fields', function () {
     await keyRight(this.browser);
     // Enter to choose.
     await this.browser.keys(Key.Enter);
-    await this.browser.pause(1000);
-
-    // Check that browser focus is back on the colour block, not e.g. a widget
-    // or dropdown div. Not sufficient to check with the focus manager.
-    // TODO: fix and flip.
-    chai.assert.isFalse(
-      await this.browser.execute(() =>
-        document.activeElement?.classList.contains('blocklyActiveFocus'),
-      ),
+
+    // Focus seems to take longer than a single pause to settle.
+    await this.browser.waitUntil(
+      () =>
+        this.browser.execute(() =>
+          document.activeElement?.classList.contains('blocklyActiveFocus'),
+        ),
+      {timeout: 1000},
     );
   });
 });