Invert setDrawerLockMode() parameter

Infomaniak · Jan 23, 2024 · 578429e · 578429e
1 parent 8436edc
commit 578429e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/app/src/main/java/com/infomaniak/mail/ui/MainActivity.kt b/app/src/main/java/com/infomaniak/mail/ui/MainActivity.kt
@@ -429,13 +429,13 @@ class MainActivity : BaseActivity() {
         SentryDebug.addNavigationBreadcrumb(destination.displayName, arguments)
         trackDestination(destination)
 
-        setDrawerLockMode(destination.id == R.id.threadListFragment)
+        setDrawerLockMode(isLocked = destination.id != R.id.threadListFragment)
 
         previousDestinationId = destination.id
     }
 
-    fun setDrawerLockMode(isUnlocked: Boolean) {
-        val drawerLockMode = if (isUnlocked) DrawerLayout.LOCK_MODE_UNLOCKED else DrawerLayout.LOCK_MODE_LOCKED_CLOSED
+    fun setDrawerLockMode(isLocked: Boolean) {
+        val drawerLockMode = if (isLocked) DrawerLayout.LOCK_MODE_LOCKED_CLOSED else DrawerLayout.LOCK_MODE_UNLOCKED
         binding.drawerLayout.setDrawerLockMode(drawerLockMode)
     }
 

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
@@ -144,7 +144,7 @@ class ThreadListMultiSelection {
     }
 
     private fun lockDrawerAndSwipe(isMultiSelectOn: Boolean) = with(threadListFragment) {
-        (requireActivity() as MainActivity).setDrawerLockMode(!isMultiSelectOn)
+        (requireActivity() as MainActivity).setDrawerLockMode(isLocked = isMultiSelectOn)
         if (isMultiSelectOn) {
             binding.threadsList.apply {
                 disableSwipeDirection(DirectionFlag.LEFT)