chore: Add close button to RestoreEmailsView (#1381)

Mail/Utils/AdaptivePanelViewModifier.swift

@@ -30,6 +30,7 @@ struct AdaptivePanelViewModifier<Item: Identifiable, PanelContent: View>: ViewMo
 
     @Binding var item: Item?
     @ViewBuilder var panelContent: (Item) -> PanelContent
+
     func body(content: Content) -> some View {
         content
             .popover(item: $item) { item in

Mail/Utils/SheetOrAlertPanel.swift

@@ -0,0 +1,52 @@
+/*
+ Infomaniak Mail - iOS App
+ Copyright (C) 2024 Infomaniak Network SA
+
+ This program is free software: you can redistribute it and/or modify
+ it under the terms of the GNU General Public License as published by
+ the Free Software Foundation, either version 3 of the License, or
+ (at your option) any later version.
+
+ This program is distributed in the hope that it will be useful,
+ but WITHOUT ANY WARRANTY; without even the implied warranty of
+ MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ GNU General Public License for more details.
+
+ You should have received a copy of the GNU General Public License
+ along with this program.  If not, see <http://www.gnu.org/licenses/>.
+ */
+
+import MailCore
+import SwiftUI
+import SwiftUIBackports
+
+extension View {
+    func sheetOrAlertPanel<ModalContent: View>(isPresented: Binding<Bool>,
+                                               @ViewBuilder modalContent: @escaping () -> ModalContent) -> some View {
+        modifier(SheetOrAlertPanel(isPresented: isPresented, modalContent: modalContent))
+    }
+}
+
+struct SheetOrAlertPanel<ModalContent: View>: ViewModifier {
+    @Environment(\.isCompactWindow) private var isCompactWindow
+
+    @Binding var isPresented: Bool
+
+    @ViewBuilder let modalContent: () -> ModalContent
+
+    func body(content: Content) -> some View {
+        content
+            .sheet(isPresented: Binding(get: { isCompactWindow && isPresented }, set: { isPresented = $0 })) {
+                if #available(iOS 16.0, *) {
+                    modalContent()
+                        .modifier(SelfSizingPanelViewModifier())
+                } else {
+                    modalContent()
+                        .modifier(SelfSizingPanelBackportViewModifier())
+                }
+            }
+            .customAlert(isPresented: Binding(get: { !isCompactWindow && isPresented }, set: { isPresented = $0 })) {
+                modalContent()
+            }
+    }
+}

Mail/Views/Bottom sheets/Actions/HeaderCloseButtonView.swift

@@ -32,7 +32,6 @@ struct HeaderCloseButtonView: View {
                 .foregroundStyle(MailTextStyle.header2.color)
                 .frame(maxWidth: .infinity)
         }
-        .padding(.horizontal, value: .regular)
         .padding(.trailing, IKIcon.Size.small.rawValue)
         .padding(.bottom, value: .regular)
     }

Mail/Views/Bottom sheets/Actions/ReportJunkView.swift

@@ -38,6 +38,7 @@ struct ReportJunkView: View {
                 HeaderCloseButtonView(title: MailResourcesStrings.Localizable.actionReportJunk) {
                     dismiss()
                 }
+                .padding(.horizontal, value: .regular)
             }
 
             ForEach(actions) { action in

Mail/Views/Bottom sheets/RestoreEmailsView.swift

@@ -26,6 +26,9 @@ import SwiftUI
 struct RestoreEmailsView: View {
     @EnvironmentObject var mailboxManager: MailboxManager
 
+    @Environment(\.dismiss) private var dismiss
+    @Environment(\.isCompactWindow) private var isCompactWindow
+
     @State private var selectedDate = ""
     @State private var availableDates = [String]()
     @State private var pickerNoSelectionText = MailResourcesStrings.Localizable.loadingText
@@ -34,30 +37,30 @@ struct RestoreEmailsView: View {
     @LazyInjectService private var snackbarPresenter: SnackBarPresentable
 
     var body: some View {
-        VStack(alignment: .leading) {
+        VStack(alignment: .leading, spacing: 0) {
             Text(MailResourcesStrings.Localizable.restoreEmailsTitle)
                 .textStyle(.bodyMedium)
-                .padding(.bottom, 16)
+                .padding(.bottom, value: .regular)
 
             Text(MailResourcesStrings.Localizable.restoreEmailsText)
                 .textStyle(.bodySecondary)
-                .padding(.bottom, 10)
+                .padding(.bottom, value: .small)
 
             LargePicker(title: MailResourcesStrings.Localizable.restoreEmailsBackupDate,
                         noSelectionText: pickerNoSelectionText,
                         selection: $selectedDate,
                         items: availableDates.map(mapDates))
-                .padding(.bottom, 24)
+                .padding(.bottom, value: .medium)
                 .onChange(of: selectedDate) { _ in
                     matomo.track(eventWithCategory: .restoreEmailsBottomSheet, action: .input, name: "selectDate")
                 }
 
             ModalButtonsView(primaryButtonTitle: MailResourcesStrings.Localizable.buttonConfirmRestoreEmails,
-                             secondaryButtonTitle: nil,
                              primaryButtonEnabled: !availableDates.isEmpty,
-                             primaryButtonAction: restoreEmails)
+                             primaryButtonAction: restoreEmails,
+                             secondaryButtonAction: dismiss.callAsFunction)
         }
-        .padding(.horizontal, UIPadding.bottomSheetHorizontal)
+        .padding(.horizontal, isCompactWindow ? UIPadding.bottomSheetHorizontal : 0)
         .task {
             await tryOrDisplayError {
                 let backupsList = try await mailboxManager.apiFetcher.listBackups(mailbox: mailboxManager.mailbox).backups

Mail/Views/Menu Drawer/Items/MenuDrawerItemsListView.swift

@@ -59,7 +59,7 @@ struct MenuDrawerItemsAdvancedListView: View {
                 ) {
                     isShowingRestoreMails = true
                 }
-                .floatingPanel(isPresented: $isShowingRestoreMails) {
+                .sheetOrAlertPanel(isPresented: $isShowingRestoreMails) {
                     RestoreEmailsView()
                 }
             }