Provide LocalSavedStateRegistryOwner in navigation-compose [179469431]

Fixed

Feature Request

Status Update

No update yet.

Description

an...@google.com

created issue #1

Feb 5, 2021 12:46PM

https://android-review.googlesource.com/c/platform/frameworks/support/+/1550025 we added LocalSavedStateRegistryOwner and I forgot that we now need to provide this ambient in NavHost.
Could you please add it?

Comments

jb...@google.com <jb...@google.com> Feb 5, 2021 04:40PM

Accepted by jb...@google.com.

ap...@google.com <ap...@google.com> #2Feb 9, 2021 12:12AM

Project: platform/frameworks/support
Branch: androidx-main

commit ae370934d2d0416b6f555755ef66be4abe51b42d
Author: Jeremy Woods <jbwoods@google.com>
Date: Fri Feb 05 10:49:38 2021

Provider SavedStateRegistryOwners to back stack entries

With the new addition of the LocalSavedStateRegistryOwner we can now
provide the NavBackStackEntry's SavedStateRegistryOwner to each
Composable destination.

RelNote: "Composable NavDestinations can now access the
NavBackStackEntry's SavedStateRegistryOwner by using the
LocalSavedStateRegistryOwner CompositionLocal."
Test: NavHostTest.savedStateRegistryOwnerTest
Bug: 179469431

Change-Id: I435d70ef0cfa8059e0d490cd5e4ed141b690ec59

M navigation/navigation-compose/src/androidTest/java/androidx/navigation/compose/NavHostTest.kt
M navigation/navigation-compose/src/main/java/androidx/navigation/compose/NavHost.kt

https://android-review.googlesource.com/1577946

jb...@google.com <jb...@google.com> #3Feb 9, 2021 10:00PM

Marked as fixed.

This has been added internally and will be available in Navigation Compose 1.0.0-alpha08.

Issue 179469431

Description

Issue summary

Comments

jb...@google.com <jb...@google.com> Feb 5, 2021 04:40PM

ap...@google.com <ap...@google.com> #2Feb 9, 2021 12:12AM

jb...@google.com <jb...@google.com> #3Feb 9, 2021 10:00PM

Add comment

Issue metadata