fix: Do not open scheduled draft on long press (#2167)

Infomaniak · Feb 20, 2025 · b77ba3c · b77ba3c
2 parents d02b5e2 + fb3279b
commit b77ba3c
Showing 1 changed file with 4 additions and 8 deletions.
diff --git a/app/src/main/java/com/infomaniak/mail/ui/main/folder/ThreadListAdapter.kt b/app/src/main/java/com/infomaniak/mail/ui/main/folder/ThreadListAdapter.kt
@@ -259,15 +259,15 @@ class ThreadListAdapter @Inject constructor(
 
         multiSelection?.let { listener ->
             selectionCardView.setOnLongClickListener {
-                onThreadClickWithAbilityToOpenMultiSelection(thread, position, listener, TrackerAction.LONG_PRESS)
+                onThreadClickWithAbilityToOpenMultiSelection(thread, listener, TrackerAction.LONG_PRESS)
                 true
             }
             expeditorAvatar.apply {
                 setOnClickListener {
-                    onThreadClickWithAbilityToOpenMultiSelection(thread, position, listener, TrackerAction.CLICK)
+                    onThreadClickWithAbilityToOpenMultiSelection(thread, listener, TrackerAction.CLICK)
                 }
                 setOnLongClickListener {
-                    onThreadClickWithAbilityToOpenMultiSelection(thread, position, listener, TrackerAction.LONG_PRESS)
+                    onThreadClickWithAbilityToOpenMultiSelection(thread, listener, TrackerAction.LONG_PRESS)
                     true
                 }
             }
@@ -319,14 +319,10 @@ class ThreadListAdapter @Inject constructor(
 
     private fun CardviewThreadItemBinding.onThreadClickWithAbilityToOpenMultiSelection(
         thread: Thread,
-        position: Int,
         listener: MultiSelectionListener<Thread>,
         action: TrackerAction,
     ) {
-        if (isMultiselectDisabledInThisFolder) {
-            onThreadClicked(thread, position)
-            return
-        }
+        if (isMultiselectDisabledInThisFolder) return
 
         val hasOpened = openMultiSelectionIfClosed(listener, action)
         toggleMultiSelectedThread(thread, shouldUpdateSelectedUi = !hasOpened)