don't auto-select any devices at page boot

noahm · Apr 16, 2024 · e4dd4fc · e4dd4fc
1 parent c7e9652
commit e4dd4fc
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 4 deletions.
diff --git a/ui/pad-coms.ts b/ui/pad-coms.ts
@@ -11,10 +11,10 @@ export async function promptSelectDevice() {
     return;
   }
 
-  await open_smx_device(devices[0]);
+  await open_smx_device(devices[0], true);
 }
 
-export async function open_smx_device(dev: HIDDevice) {
+export async function open_smx_device(dev: HIDDevice, autoSelect = false) {
   if (!dev.opened) {
     await dev.open();
   }
@@ -34,11 +34,13 @@ export async function open_smx_device(dev: HIDDevice) {
       [serial]: stage, // TODO: Is there a better way to handle this?
     };
   });
-  uiState.set(selectedStageSerial$, serial);
   uiState.set(
     nextStatusTextLine$,
     `status: device opened: ${dev.productName}:P${stage.info?.player}:${stage.info?.serial}`,
   );
+  if (autoSelect) {
+    uiState.set(selectedStageSerial$, serial);
+  }
 }
 
 /**

diff --git a/ui/ui.tsx b/ui/ui.tsx
@@ -15,7 +15,7 @@ import {
 import { StageTest } from "./stage/stage-test.tsx";
 import { TypedSelect } from "./common/typed-select.tsx";
 
-export function UI() {
+function usePreviouslyPairedDevices() {
   useEffect(() => {
     // once, on load, get paired devices and attempt connection
     if (browserSupported) {
@@ -27,6 +27,10 @@ export function UI() {
       );
     }
   }, []);
+}
+
+export function UI() {
+  usePreviouslyPairedDevices();
 
   return (
     <>