chore: couple of fixes.

This removes the passive indicator rendering since FocusManager handles
it. It also updates part of toolbox navigation to select a first item if
there isn't currently a selection (which state is now possible due to
tab navigation).

Author: BenHenning

src/actions/arrow_navigation.ts

@@ -166,7 +166,11 @@ export class ArrowNavigation {
               return isHandled;
             case Constants.STATE.TOOLBOX:
               // TODO: Move this into cursor?
-              isHandled = toolbox.selectNext();
+              if (!toolbox.getSelectedItem()) {
+                const firstItem = toolbox.getToolboxItems().find((item) => item.isSelectable()) ?? null;
+                toolbox.setSelectedItem(firstItem);
+                isHandled = true;
+              } else isHandled = toolbox.selectNext();
               const selectedItem = toolbox.getSelectedItem();
               if (selectedItem) {
                 Blockly.getFocusManager().focusNode(selectedItem);

src/navigation.ts

@@ -441,7 +441,7 @@ export class Navigation {
     if (cursor && (ignorePopUpDivs || !popUpDivsShowing)) {
       const curNode = cursor.getCurNode();
       if (curNode) {
-        this.passiveFocusIndicator.show(curNode);
+        // this.passiveFocusIndicator.show(curNode);
       }
       // It's initially null so this is a valid state despite the types.
       cursor.setCurNode(null);