From 7beab639292ca96cf4881aae1195a7e3d7db4043 Mon Sep 17 00:00:00 2001
From: Eric Ayers <ericzundel@gmail.com>
Date: Fri, 11 Jul 2025 23:32:50 -0400
Subject: [PATCH] Conditionally change button text on USB connect

 - Change the text in the UI button for selecting a filesystem
   mount point based on platform.  Putting the pathname on the
   filesystem could be confusing for Windows or ChromeOS users
   where they don't have the concept of a single filesystem
   tree the way *NIX filesystems are used to.
---
 index.html             |  2 +-
 js/common/utilities.js | 21 +++++++++++++++++++++
 js/workflows/usb.js    |  7 ++++++-
 3 files changed, 28 insertions(+), 2 deletions(-)

diff --git a/index.html b/index.html
index 4f10992..ff7c0a8 100644
--- a/index.html
+++ b/index.html
@@ -308,7 +308,7 @@ <h1>Select Serial Device</h1>
                     <h1>Select USB Host Folder</h1>
                     <p>Select the root folder of your device. This is typically the CIRCUITPY Drive on your computer unless you renamed it. If your device does not appear as a drive on your computer, it will need to have the USB Host functionality enabled.</p>
                     <p>
-                        <button class="purple-button hidden" id="useHostFolder">Use <span id="workingFolder"></span></button>
+                        <button class="purple-button hidden" id="useHostFolder"><span id="workingFolder"></span></button>
                         <button class="purple-button first-item" id="selectHostFolder">Select New Folder</button>
                     </p>
                 </div>
diff --git a/js/common/utilities.js b/js/common/utilities.js
index 25105f7..0b9f881 100644
--- a/js/common/utilities.js
+++ b/js/common/utilities.js
@@ -56,6 +56,25 @@ function isLocal() {
     return (isMdns() || location.hostname == "localhost" || isIp()) && (location.pathname == "/code/");
 }
 
+// Test to see if browser is running on Microsoft Windows OS
+function isMicrosoftWindows() {
+    // Newer test on Chromium
+    if (navigator.userAgentData?.platform === "Windows") {
+        return true;
+    } else if (navigator.userAgent.includes("Windows")) {
+        return true;
+    }
+    return false;
+}
+
+// Test to see if browser is running on Microsoft Windows OS
+function isChromeOs() {
+    if (navigator.userAgent.includes("CrOS")) {
+        return true;
+    }
+    return false;
+}
+
 // Parse out the url parameters from the current url
 function getUrlParams() {
     // This should look for and validate very specific values
@@ -146,6 +165,8 @@ export {
     isMdns,
     isIp,
     isLocal,
+    isMicrosoftWindows,
+    isChromeOs,
     getUrlParams,
     getUrlParam,
     timeout,
diff --git a/js/workflows/usb.js b/js/workflows/usb.js
index 36605ad..7c65f63 100644
--- a/js/workflows/usb.js
+++ b/js/workflows/usb.js
@@ -4,6 +4,7 @@ import {GenericModal, DeviceInfoModal} from '../common/dialogs.js';
 import {FileOps} from '@adafruit/circuitpython-repl-js'; // Use this to determine which FileTransferClient to load
 import {FileTransferClient as ReplFileTransferClient} from '../common/repl-file-transfer.js';
 import {FileTransferClient as FSAPIFileTransferClient} from '../common/fsapi-file-transfer.js';
+import { isChromeOs, isMicrosoftWindows } from '../common/utilities.js';
 
 let btnRequestSerialDevice, btnSelectHostFolder, btnUseHostFolder, lblWorkingfolder;
 
@@ -247,7 +248,11 @@ class USBWorkflow extends Workflow {
         console.log("New folder name:", folderName);
         if (folderName) {
             // Set the working folder label
-            lblWorkingfolder.innerHTML = folderName;
+            if (isMicrosoftWindows() || isChromeOs()) {
+                lblWorkingfolder.innerHTML = "OK";
+            } else {
+                lblWorkingfolder.innerHTML = `Use ${folderName}`;
+            }
             btnUseHostFolder.classList.remove("hidden");
             btnSelectHostFolder.innerHTML = "Select Different Folder";
             btnSelectHostFolder.classList.add("inverted");