[Navigation Material] Update Docs and Sample to use new API for addin…

…g custom Navigators (#746) * Update Navigation to 2.4.0-alpha10 * Update Docs and Sample to use new API for adding custom Navigators Fixes #744 * Remove unneeded add statement
google · Sep 30, 2021 · a285f41 · a285f41
1 parent bbab100
commit a285f41
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 11 deletions.
diff --git a/docs/navigation-material.md b/docs/navigation-material.md
@@ -18,9 +18,8 @@ This features composable bottom sheet destinations.
     ```kotlin
     @Composable
     fun MyApp() {
-        val navController = rememberNavController()
         val bottomSheetNavigator = rememberBottomSheetNavigator()
-        navController.navigatorProvider += bottomSheetNavigator
+        val navController = rememberNavController(bottomSheetNavigator)
     }
     ```
 
@@ -29,9 +28,8 @@ This features composable bottom sheet destinations.
     ```kotlin
     @Composable
     fun MyApp() {
-        val navController = rememberNavController()
         val bottomSheetNavigator = rememberBottomSheetNavigator()
-        navController.navigatorProvider += bottomSheetNavigator
+        val navController = rememberNavController(bottomSheetNavigator)
         ModalBottomSheetLayout(bottomSheetNavigator) {
             NavHost(navController, Destinations.Home) {
                // We'll define our graph here in a bit!
@@ -45,9 +43,8 @@ This features composable bottom sheet destinations.
     ```kotlin
     @Composable
     fun MyApp() {
-        val navController = rememberNavController()
         val bottomSheetNavigator = rememberBottomSheetNavigator()
-        navController.navigatorProvider += bottomSheetNavigator
+        val navController = rememberNavController(bottomSheetNavigator)
         ModalBottomSheetLayout(bottomSheetNavigator) {
             NavHost(navController, Destinations.Home) {
                composable(route = "home") {

diff --git a/...c/androidTest/java/com/google/accompanist/navigation.material/BottomSheetNavigatorTest.kt b/...c/androidTest/java/com/google/accompanist/navigation.material/BottomSheetNavigatorTest.kt
@@ -198,10 +198,9 @@ internal class BottomSheetNavigatorTest {
         val sheetState = ModalBottomSheetState(ModalBottomSheetValue.Hidden)
 
         composeTestRule.setContent {
-            navController = rememberNavController()
+            navigator = remember { BottomSheetNavigator(sheetState) }
+            navController = rememberNavController(navigator)
             if (savedState != null) navController.restoreState(savedState)
-            navigator = remember(navController) { BottomSheetNavigator(sheetState) }
-            navController.navigatorProvider += navigator
             if (compositionState == 0) {
                 ModalBottomSheetLayout(
                     bottomSheetNavigator = navigator

diff --git a/...e/src/main/java/com/google/accompanist/sample/navigation/material/BottomSheetNavSample.kt b/...e/src/main/java/com/google/accompanist/sample/navigation/material/BottomSheetNavSample.kt
@@ -58,9 +58,8 @@ private object Destinations {
 @OptIn(ExperimentalMaterialNavigationApi::class)
 @Composable
 fun BottomSheetNavDemo() {
-    val navController = rememberNavController()
     val bottomSheetNavigator = rememberBottomSheetNavigator()
-    navController.navigatorProvider += bottomSheetNavigator
+    val navController = rememberNavController(bottomSheetNavigator)
 
     ModalBottomSheetLayout(bottomSheetNavigator) {
         NavHost(navController, Destinations.Home) {