fix: Set isShowingDiscovery to false instead of hiding the view

Infomaniak · May 21, 2024 · fcd2840 · fcd2840
1 parent 9247d72
commit fcd2840
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 15 deletions.
diff --git a/Mail/Views/New Message/ComposeMessageView.swift b/Mail/Views/New Message/ComposeMessageView.swift
@@ -231,7 +231,7 @@ struct ComposeMessageView: View {
             initialAttachments = []
 
             if featureFlagsManager.isEnabled(.aiMailComposer) && UserDefaults.shared.shouldPresentAIFeature {
-                aiModel.isShowingDiscovery = true
+                aiModel.isShowingDiscovery = platformDetector.isMac ? false : true
                 return
             }
 
@@ -272,12 +272,10 @@ struct ComposeMessageView: View {
             }
         }
         .discoveryPresenter(isPresented: $aiModel.isShowingDiscovery) {
-            if !platformDetector.isMac {
-                DiscoveryView(item: .aiDiscovery) {
-                    UserDefaults.shared.shouldPresentAIFeature = false
-                } completionHandler: { willShowAIPrompt in
-                    aiModel.isShowingPrompt = willShowAIPrompt
-                }
+            DiscoveryView(item: .aiDiscovery) {
+                UserDefaults.shared.shouldPresentAIFeature = false
+            } completionHandler: { willShowAIPrompt in
+                aiModel.isShowingPrompt = willShowAIPrompt
             }
         }
         .aiPromptPresenter(isPresented: $aiModel.isShowingPrompt) {

diff --git a/Mail/Views/SplitView.swift b/Mail/Views/SplitView.swift
@@ -124,13 +124,11 @@ struct SplitView: View {
             }
         }
         .discoveryPresenter(isPresented: $mainViewState.isShowingSyncDiscovery) {
-            if !platformDetector.isMac {
-                DiscoveryView(item: .syncDiscovery) {
-                    UserDefaults.shared.shouldPresentSyncDiscovery = false
-                } completionHandler: { willSync in
-                    guard willSync else { return }
-                    mainViewState.isShowingSyncProfile = true
-                }
+            DiscoveryView(item: .syncDiscovery) {
+                UserDefaults.shared.shouldPresentSyncDiscovery = false
+            } completionHandler: { willSync in
+                guard willSync else { return }
+                mainViewState.isShowingSyncProfile = true
             }
         }
         .discoveryPresenter(isPresented: $mainViewState.isShowingSetAppAsDefaultDiscovery) {
@@ -167,7 +165,7 @@ struct SplitView: View {
                     try await mailboxManager.refreshAllSignatures()
                 }
                 guard !platformDetector.isDebug else { return }
-                mainViewState.isShowingSyncDiscovery = shouldShowSync()
+                mainViewState.isShowingSyncDiscovery = platformDetector.isMac ? false : shouldShowSync()
             }
         }
         .onOpenURL { url in