Make isDisplayingValidationErrors based on the validationResults

StanfordSpezi · Nov 3, 2023 · c961e57 · c961e57
1 parent bd04a27
commit c961e57
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 7 deletions.
diff --git a/Sources/SpeziValidation/ValidationEngine.swift b/Sources/SpeziValidation/ValidationEngine.swift
@@ -56,11 +56,13 @@ public class ValidationEngine: Identifiable {
 
     /// Flag that indicates if ``displayedValidationResults`` returns any ``FailedValidationResult``.
     @MainActor public var isDisplayingValidationErrors: Bool {
+        let gotResults = !validationResults.isEmpty
+
         if configuration.contains(.hideFailedValidationOnEmptySubmit) {
-            return !inputValid && (source == .manual || !inputWasEmpty)
+            return gotResults && (source == .manual || !inputWasEmpty)
         }
 
-        return !inputValid
+        return gotResults
     }
 
 

diff --git a/Sources/SpeziValidation/ValidationModifier.swift b/Sources/SpeziValidation/ValidationModifier.swift
@@ -40,6 +40,10 @@ struct ValidationModifier<FocusValue: Hashable>: ViewModifier {
             .onChange(of: input) {
                 validation.submit(input: input, debounce: true)
             }
+            .onSubmit(of: .text) {
+                // here we just make sure that we submit it without a debounce
+                validation.submit(input: input)
+            }
     }
 }
 

diff --git a/Sources/SpeziValidation/Views/VerifiableTextField.swift b/Sources/SpeziValidation/Views/VerifiableTextField.swift
@@ -42,11 +42,6 @@ public struct VerifiableTextField<FieldLabel: View, FieldFooter: View>: View {
                     SecureField(text: $text, label: { label })
                 }
             }
-                .onSubmit {
-                    // the validation modifier automatically submits the text,
-                    // here we just make sure that we submit it without a debounce
-                    validationEngine.submit(input: text)
-                }
 
             HStack {
                 ValidationResultsView(results: validationEngine.displayedValidationResults)