Use standard validation display for operator id

mavlink · Dec 21, 2024 · d606c2b · d606c2b
1 parent 6f7af6d
commit d606c2b
Showing 1 changed file with 12 additions and 2 deletions.
diff --git a/src/UI/preferences/RemoteIDSettings.qml b/src/UI/preferences/RemoteIDSettings.qml
@@ -342,8 +342,6 @@ SettingsPage {
             SettingsGroupLayout {
                 heading:            qsTr("Operator ID")
                 Layout.fillWidth:   true
-                outerBorderColor: (_regionOperation === RemoteIDSettings.RegionOperation.EU || remoteIDSettings.sendOperatorID.value) ?
-                                      (_activeRID && !_remoteIDManager.operatorIDGood ? qgcPal.colorRed : defaultBorderColor) : defaultBorderColor
 
                 FactCheckBoxSlider {
                     text:               qsTr("Broadcast%1").arg(isEURegion ? " (EU Required)" : "")
@@ -375,12 +373,24 @@ SettingsPage {
                     }
 
                     QGCTextField {
+                        id:                     operatorIDTextField
                         Layout.preferredWidth:  textFieldWidth
                         Layout.fillWidth:       true
                         text:                   operatorIDFact.valueString
                         visible:                operatorIDFact.visible
                         maximumLength:          20                  // Maximum defined by Mavlink definition of OPEN_DRONE_ID_OPERATOR_ID message
 
+                        property bool operatorIDInvalid: ((_regionOperation === RemoteIDSettings.RegionOperation.EU || remoteIDSettings.sendOperatorID.value) &&
+                                                            _activeRID && !_remoteIDManager.operatorIDGood)
+
+                        onOperatorIDInvalidChanged: {
+                            if (operatorIDInvalid) {
+                                operatorIDTextField.showValidationError(qsTr("Invalid Operator ID"), operatorIDFact.valueString, false /* preventViewSwitch */)
+                            } else {
+                                operatorIDTextField.clearValidationError(false /* preventViewSwitch */)
+                            }
+                        }
+
                         onTextChanged: {
                             operatorIDFact.value = text
                             if (_activeVehicle) {