chore: Always restoreState in bottom navigation

Infomaniak · Dec 17, 2024 · d223148 · d223148
1 parent b316896
commit d223148
Showing 1 changed file with 10 additions and 4 deletions.
diff --git a/app/src/main/java/com/infomaniak/swisstransfer/ui/screen/main/components/MainScaffold.kt b/app/src/main/java/com/infomaniak/swisstransfer/ui/screen/main/components/MainScaffold.kt
@@ -113,11 +113,17 @@ private fun NavHostController.navigateToSelectedItem(destination: MainNavigation
         }
         // Avoid multiple copies of the same destination when re-selecting the same item
         launchSingleTop = true
+
         // Restore state when re-selecting a previously selected item
-        val currentBackStackEntry = this@navigateToSelectedItem.currentBackStackEntry
-        val hasDeepLink = currentBackStackEntry?.arguments?.parcelableExtra<Intent>(NavController.KEY_DEEP_LINK_INTENT) != null
-        val isNavigatingToStartDestination = startDestination.route == destination::class.qualifiedName
-        restoreState = !hasDeepLink || !isNavigatingToStartDestination
+        restoreState = true
+
+        // The commented code below was supposed to be an improvement on the line above,
+        // but it doesn't behave the way we'd like. TODO: Find a better solution that works
+
+        // val currentBackStackEntry = [email protected]
+        // val hasDeepLink = currentBackStackEntry?.arguments?.parcelableExtra<Intent>(NavController.KEY_DEEP_LINK_INTENT) != null
+        // val isNavigatingToStartDestination = startDestination.route == destination::class.qualifiedName
+        // restoreState = !hasDeepLink || !isNavigatingToStartDestination
     }
 }