Skip to content

Commit

Permalink
fix: Disable multiselection after selecting action
Browse files Browse the repository at this point in the history
  • Loading branch information
PhilippeWeidmann committed May 3, 2023
1 parent d81e1a9 commit 7639572
Showing 1 changed file with 11 additions and 4 deletions.
15 changes: 11 additions & 4 deletions Mail/Views/Thread List/ThreadListView.swift
Original file line number Diff line number Diff line change
Expand Up @@ -257,6 +257,7 @@ private struct ThreadListToolbar: ViewModifier {
@ObservedObject var multipleSelectionViewModel: ThreadListMultipleSelectionViewModel

@State private var isShowingSwitchAccount = false
@State private var multipleSelectionActionsTarget: ActionsTarget?

@EnvironmentObject var splitViewManager: SplitViewManager
@EnvironmentObject var navigationDrawerState: NavigationDrawerState
Expand Down Expand Up @@ -347,16 +348,22 @@ private struct ThreadListToolbar: ViewModifier {
.disabled(action == .archive && splitViewManager.selectedFolder?.role == .archive)
}

ActionsPanelButton(threads: Array(multipleSelectionViewModel.selectedItems),
isMultiSelectionEnabled: true) {
ToolbarButtonLabel(text: MailResourcesStrings.Localizable.buttonMore,
icon: MailResourcesAsset.plusActions.swiftUIImage)
ToolbarButton(
text: MailResourcesStrings.Localizable.buttonMore,
icon: MailResourcesAsset.plusActions.swiftUIImage
) {
multipleSelectionActionsTarget = .threads(Array(multipleSelectionViewModel.selectedItems), true)
}
}
.disabled(multipleSelectionViewModel.selectedItems.isEmpty)
}
}
}
.actionsPanel(actionsTarget: $multipleSelectionActionsTarget) {
withAnimation {
multipleSelectionViewModel.isEnabled = false
}
}
.navigationTitle(
multipleSelectionViewModel.isEnabled
? MailResourcesStrings.Localizable.multipleSelectionCount(multipleSelectionViewModel.selectedItems.count)
Expand Down

0 comments on commit 7639572

Please sign in to comment.