refactor: extract nav method

Author: jvsena42

app/src/main/java/to/bitkit/ui/ContentView.kt

@@ -1385,6 +1385,16 @@ fun NavController.navigateToHome() {
     }
 }
 
+/**
+ * Navigates to the specified route only if not already on that route.
+ */
+inline fun <reified T : Any> NavController.navigateIfNotCurrent(route: T) {
+    val isOnRoute = currentBackStackEntry?.destination?.hasRoute<T>() ?: false
+    if (!isOnRoute) {
+        navigate(route)
+    }
+}
+
 fun NavController.navigateToSettings() = navigate(
     route = Routes.Settings,
 )

app/src/main/java/to/bitkit/ui/components/DrawerMenu.kt

@@ -39,12 +39,13 @@ import androidx.compose.ui.unit.dp
 import androidx.compose.ui.unit.sp
 import androidx.compose.ui.zIndex
 import androidx.navigation.NavController
+import androidx.navigation.NavDestination.Companion.hasRoute
 import androidx.navigation.compose.rememberNavController
 import kotlinx.coroutines.launch
 import to.bitkit.R
 import to.bitkit.ui.Routes
+import to.bitkit.ui.navigateIfNotCurrent
 import to.bitkit.ui.navigateToHome
-import to.bitkit.ui.navigateToSettings
 import to.bitkit.ui.screens.wallets.HomeRoutes
 import to.bitkit.ui.shared.util.blockPointerInputPassthrough
 import to.bitkit.ui.shared.util.clickableAlpha
@@ -102,16 +103,16 @@ fun DrawerMenu(
             drawerState = drawerState,
             onClickAddWidget = {
                 if (!hasSeenWidgetsIntro) {
-                    rootNavController.navigate(Routes.WidgetsIntro)
+                    rootNavController.navigateIfNotCurrent(Routes.WidgetsIntro)
                 } else {
-                    rootNavController.navigate(Routes.AddWidget)
+                    rootNavController.navigateIfNotCurrent(Routes.AddWidget)
                 }
             },
             onClickShop = {
                 if (!hasSeenShopIntro) {
-                    rootNavController.navigate(Routes.ShopIntro)
+                    rootNavController.navigateIfNotCurrent(Routes.ShopIntro)
                 } else {
-                    rootNavController.navigate(Routes.ShopDiscover)
+                    rootNavController.navigateIfNotCurrent(Routes.ShopDiscover)
                 }
             },
         )
@@ -141,7 +142,11 @@ private fun Menu(
             label = stringResource(R.string.wallet__drawer__wallet),
             iconRes = R.drawable.ic_coins,
             onClick = {
-                rootNavController.navigateToHome()
+                val isOnHome = rootNavController.currentBackStackEntry
+                    ?.destination?.hasRoute<Routes.Home>() ?: false
+                if (!isOnHome) {
+                    rootNavController.navigateToHome()
+                }
                 scope.launch { drawerState.close() }
             },
             modifier = Modifier.testTag("DrawerWallet")
@@ -152,9 +157,9 @@ private fun Menu(
             iconRes = R.drawable.ic_heartbeat,
             onClick = {
                 if (walletNavController != null) {
-                    walletNavController.navigate(HomeRoutes.AllActivity)
+                    walletNavController.navigateIfNotCurrent(HomeRoutes.AllActivity)
                 } else {
-                    rootNavController.navigate(Routes.Home)
+                    rootNavController.navigateIfNotCurrent(Routes.Home)
                 }
                 scope.launch { drawerState.close() }
             },
@@ -199,7 +204,7 @@ private fun Menu(
             label = stringResource(R.string.wallet__drawer__settings),
             iconRes = R.drawable.ic_settings,
             onClick = {
-                rootNavController.navigateToSettings()
+                rootNavController.navigateIfNotCurrent(Routes.Settings)
                 scope.launch { drawerState.close() }
             },
             modifier = Modifier.testTag("DrawerSettings")
@@ -212,15 +217,17 @@ private fun Menu(
             modifier = Modifier
                 .fillMaxWidth()
                 .clickableAlpha {
-                    rootNavController.navigate(Routes.AppStatus)
+                    rootNavController.navigateIfNotCurrent(Routes.AppStatus)
                     scope.launch { drawerState.close() }
                 }
         ) {
             AppStatus(
                 showText = true,
                 showReady = true,
                 color = Colors.Black,
-                modifier = Modifier.padding(vertical = 16.dp).testTag("DrawerAppStatus")
+                modifier = Modifier
+                    .padding(vertical = 16.dp)
+                    .testTag("DrawerAppStatus")
             )
         }
     }