From cd56f7f71ae2d2128d6fed923db4b3ba637270ca Mon Sep 17 00:00:00 2001
From: Robin Davies <rerdavies@gmail.com>
Date: Sat, 12 Mar 2022 16:02:35 -0500
Subject: [PATCH] No browser save of  wifi config password fixes #32

---
 react/src/NoChangePassword.tsx | 1 +
 react/src/WifiConfigDialog.tsx | 9 ++++++---
 2 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/react/src/NoChangePassword.tsx b/react/src/NoChangePassword.tsx
index ba41c74..eb13e58 100644
--- a/react/src/NoChangePassword.tsx
+++ b/react/src/NoChangePassword.tsx
@@ -90,6 +90,7 @@ class NoChangePassword extends React.Component<NoChangePasswordProps, NoChangePa
         {
             e.target.value = "(Unchanged)";
         }
+        e.currentTarget.setAttribute("readonly","true");
     }
 
     render() {
diff --git a/react/src/WifiConfigDialog.tsx b/react/src/WifiConfigDialog.tsx
index 9ae1f78..e073b01 100644
--- a/react/src/WifiConfigDialog.tsx
+++ b/react/src/WifiConfigDialog.tsx
@@ -207,9 +207,10 @@ const WifiConfigDialog = withStyles(styles, { withTheme: true })(
             } else {
                 wifiConfigSettings.wifiWarningGiven = true;
                 this.preventPasswordPrompt();
+                // let preventPasswordPrompt changes settle (it's HARD to prevent a password prompt)
                 setTimeout(()=> {
                     this.props.onOk(wifiConfigSettings);
-                });
+                },100);
             }
         }
     
@@ -219,9 +220,10 @@ const WifiConfigDialog = withStyles(styles, { withTheme: true })(
         let passwordInput = this.refPassword.current;
         if (passwordInput)
         {
-            passwordInput.value = "";
             passwordInput.type = "text";
+            passwordInput.value = "";
         }
+        this.setState({newPassword: ""});
     }
     handleChannelChange(e: any)
     {
@@ -247,9 +249,10 @@ const WifiConfigDialog = withStyles(styles, { withTheme: true })(
 
         const handleClose = () => {
             this.preventPasswordPrompt();
+            // let preventPasswordPrompt changes settle (it's HARD to prevent a password prompt)
             setTimeout(()=> {
                 onClose();
-            });
+            },100);
         };
 
         return (