Make sure we do not automatically dismiss an user initiated account s…

…heet presentation
StanfordSpezi · Nov 18, 2024 · c6c5cd9 · c6c5cd9
1 parent 0a0ea7b
commit c6c5cd9
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 3 deletions.
diff --git a/TemplateApplication/Account/AccountSheet.swift b/TemplateApplication/Account/AccountSheet.swift
@@ -12,6 +12,8 @@ import SwiftUI
 
 
 struct AccountSheet: View {
+    private let dismissAfterSignIn: Bool
+
     @Environment(\.dismiss) var dismiss
 
     @Environment(Account.self) private var account
@@ -33,7 +35,9 @@ struct AccountSheet: View {
                     }
                 } else {
                     AccountSetup { _ in
-                        dismiss() // we just signed in, dismiss the account setup sheet
+                        if dismissAfterSignIn {
+                            dismiss() // we just signed in, dismiss the account setup sheet
+                        }
                     } header: {
                         AccountSetupHeader()
                     }
@@ -50,13 +54,17 @@ struct AccountSheet: View {
         }
     }
 
-    var closeButton: some ToolbarContent {
+    @ToolbarContentBuilder private var closeButton: some ToolbarContent {
         ToolbarItem(placement: .cancellationAction) {
             Button("Close") {
                 dismiss()
             }
         }
     }
+
+    init(dismissAfterSignIn: Bool = true) {
+        self.dismissAfterSignIn = dismissAfterSignIn
+    }
 }
 
 

diff --git a/TemplateApplication/HomeView.swift b/TemplateApplication/HomeView.swift
@@ -37,7 +37,7 @@ struct HomeView: View {
             .tabViewStyle(.sidebarAdaptable)
             .tabViewCustomization($tabViewCustomization)
             .sheet(isPresented: $presentingAccount) {
-                AccountSheet()
+                AccountSheet(dismissAfterSignIn: false) // presentation was user initiated, do not automatically dismiss
             }
             .accountRequired(!FeatureFlags.disableFirebase && !FeatureFlags.skipOnboarding) {
                 AccountSheet()