Fix USB: flip connected state before calling loadEditor

In USBWorkflow.onConnected, loadEditor() was being called BEFORE
super.onConnected() set _connected = CONNSTATE.connected. The new
post-connect Device Info dialog trigger gates on connectionStatus(),
which requires _connected == connected, so the dialog never opened
for USB users.

Reorder so super.onConnected() (which both flips the state flag and
closes the connect dialog) runs first, then loadEditor() runs with
connectionStatus() true. Also drop the redundant explicit
connectDialog.close() call -- super.onConnected already closes it.

Author: makermelissa-piclaw

js/workflows/usb.js

@@ -45,10 +45,13 @@ class USBWorkflow extends Workflow {
     }
 
     async onConnected(e) {
-        this.connectDialog.close();
-        await this.loadEditor();
+        // super.onConnected sets _connected=CONNSTATE.connected and closes
+        // the connect dialog. Run it first so that loadEditor() (and any
+        // other code that gates on connectionStatus()) sees us as fully
+        // connected.
+        await super.onConnected(e);
         this.debugLog("connected");
-        super.onConnected(e);
+        await this.loadEditor();
     }
 
     async onDisconnected(e, reconnect = true) {