fix: Remove multiple selection animations

Infomaniak · Jun 13, 2023 · a7e6382 · a7e6382
1 parent 7d5e3cc
commit a7e6382
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 19 deletions.
diff --git a/Mail/Components/BottomBarView.swift b/Mail/Components/BottomBarView.swift
@@ -32,7 +32,6 @@ struct BottomBar<Items: View>: ViewModifier {
                 BottomBarView(items: items)
             }
         }
-//        .animation(.nil, value: isVisible)
         .ignoresSafeArea(.keyboard, edges: .bottom)
     }
 }

diff --git a/Mail/Views/Thread List/ThreadListCell.swift b/Mail/Views/Thread List/ThreadListCell.swift
@@ -83,15 +83,13 @@ struct ThreadListCell: View {
     }
 
     private func didLongPressCell() {
-        withAnimation {
-            multipleSelectionViewModel.feedbackGenerator.prepare()
-            multipleSelectionViewModel.isEnabled.toggle()
-            if multipleSelectionViewModel.isEnabled {
-                @InjectService var matomo: MatomoUtils
-                matomo.track(eventWithCategory: .multiSelection, action: .longPress, name: "enable")
-                multipleSelectionViewModel.feedbackGenerator.impactOccurred()
-                multipleSelectionViewModel.toggleSelection(of: thread)
-            }
+        multipleSelectionViewModel.feedbackGenerator.prepare()
+        multipleSelectionViewModel.isEnabled.toggle()
+        if multipleSelectionViewModel.isEnabled {
+            @InjectService var matomo: MatomoUtils
+            matomo.track(eventWithCategory: .multiSelection, action: .longPress, name: "enable")
+            multipleSelectionViewModel.feedbackGenerator.impactOccurred()
+            multipleSelectionViewModel.toggleSelection(of: thread)
         }
     }
 }

diff --git a/Mail/Views/Thread List/ThreadListModifiers.swift b/Mail/Views/Thread List/ThreadListModifiers.swift
@@ -76,9 +76,7 @@ struct ThreadListToolbar: ViewModifier {
                     if multipleSelectionViewModel.isEnabled {
                         Button(MailResourcesStrings.Localizable.buttonCancel) {
                             matomo.track(eventWithCategory: .multiSelection, name: "cancel")
-                            withAnimation {
-                                multipleSelectionViewModel.isEnabled = false
-                            }
+                            multipleSelectionViewModel.isEnabled = false
                         }
                     } else {
                         if isCompactWindow {
@@ -160,9 +158,7 @@ struct ThreadListToolbar: ViewModifier {
                 .disabled(multipleSelectionViewModel.selectedItems.isEmpty)
             }
             .actionsPanel(actionsTarget: $multipleSelectionActionsTarget) {
-                withAnimation {
-                    multipleSelectionViewModel.isEnabled = false
-                }
+                multipleSelectionViewModel.isEnabled = false
             }
             .navigationTitle(
                 multipleSelectionViewModel.isEnabled

diff --git a/Mail/Views/Thread List/ThreadListMultipleSectionViewModel.swift b/Mail/Views/Thread List/ThreadListMultipleSectionViewModel.swift
@@ -100,9 +100,7 @@ import SwiftUI
         default:
             break
         }
-        withAnimation {
-            isEnabled = false
-        }
+        isEnabled = false
     }
 
     private func setActions() {