From c67cd9bb19e0090c4af0dd67ca381a0228d2bb40 Mon Sep 17 00:00:00 2001
From: Floris Bos <bos@je-eigen-domein.nl>
Date: Sat, 27 Mar 2021 11:04:13 +0100
Subject: [PATCH] Color password field red if user enabled SSH but forget
 password

Ref #188
---
 OptionsPopup.qml | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/OptionsPopup.qml b/OptionsPopup.qml
index 587db9a..fa47de7 100644
--- a/OptionsPopup.qml
+++ b/OptionsPopup.qml
@@ -169,13 +169,14 @@ Popup {
 
                             Text {
                                 text: qsTr("Set password for 'pi' user:")
-                                color: parent.enabled ? "black" : "grey"
+                                color: parent.enabled ? (fieldUserPassword.indicateError ? "red" : "black") : "grey"
                             }
                             TextField {
                                 id: fieldUserPassword
                                 echoMode: TextInput.Password
                                 Layout.minimumWidth: 200
                                 property bool alreadyCrypted: false
+                                property bool indicateError: false
 
                                 onTextEdited: {
                                     if (alreadyCrypted) {
@@ -184,6 +185,9 @@ Popup {
                                         alreadyCrypted = false
                                         clear()
                                     }
+                                    if (indicateError) {
+                                        indicateError = false
+                                    }
                                 }
                             }
                         }
@@ -341,8 +345,9 @@ Popup {
             Button {
                 text: qsTr("SAVE")
                 onClicked: {
-                    if (radioPasswordAuthentication.checked && fieldUserPassword.text.length == 0)
+                    if (chkSSH.checked && radioPasswordAuthentication.checked && fieldUserPassword.text.length == 0)
                     {
+                        fieldUserPassword.indicateError = true
                         fieldUserPassword.forceActiveFocus()
                         return
                     }