Merge pull request #1702 from Infomaniak/deactivate-multi-select-if-no-selection

KevinBoulongne · web-flow · commit ea38c1c6ac09 · 2024-02-16T06:39:24.000+01:00
Deactivate multiselect if we don't have any selected mail
diff --git a/app/src/main/java/com/infomaniak/mail/ui/main/folder/ThreadListMultiSelection.kt b/app/src/main/java/com/infomaniak/mail/ui/main/folder/ThreadListMultiSelection.kt
@@ -128,9 +128,13 @@ class ThreadListMultiSelection {
         }
 
         mainViewModel.selectedThreadsLiveData.observe(viewLifecycleOwner) { selectedThreads ->
-            updateSelectedCount(selectedThreads)
-            updateSelectAllLabel()
-            updateMultiSelectActionsStatus(selectedThreads)
+            if (selectedThreads.isEmpty()) {
+                mainViewModel.isMultiSelectOn = false
+            } else {
+                updateSelectedCount(selectedThreads)
+                updateSelectAllLabel()
+                updateMultiSelectActionsStatus(selectedThreads)
+            }
         }
     }
 

Original file line number	Diff line number	Diff line change
`@@ -128,9 +128,13 @@ class ThreadListMultiSelection {`
`128`	`128`	`}`
`129`	`129`
`130`	`130`	`mainViewModel.selectedThreadsLiveData.observe(viewLifecycleOwner) { selectedThreads ->`
`131`		`- updateSelectedCount(selectedThreads)`
`132`		`- updateSelectAllLabel()`
`133`		`- updateMultiSelectActionsStatus(selectedThreads)`
	`131`	`+ if (selectedThreads.isEmpty()) {`
	`132`	`+ mainViewModel.isMultiSelectOn = false`
	`133`	`+ } else {`
	`134`	`+ updateSelectedCount(selectedThreads)`
	`135`	`+ updateSelectAllLabel()`
	`136`	`+ updateMultiSelectActionsStatus(selectedThreads)`
	`137`	`+ }`
`134`	`138`	`}`
`135`	`139`	`}`
`136`	`140`