Atte149/book-story
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Navigation.kt: Remembered shouldShow for Navigator.composable (might be unnoticeable performance improvement).

Browse source
This commit is contained in:
acclorite 2024-06-20 14:00:02 +03:00
parent 6f985274bf
commit 99de280c56
1 changed files with 10 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

15
app/src/main/java/ua/acclorite/book_story/presentation/data/Navigation.kt
View file

@ -269,11 +269,16 @@ class Navigator @AssistedInject constructor(
backExitAnim: ExitTransition = Transitions.BackSlidingTransitionOut, backExitAnim: ExitTransition = Transitions.BackSlidingTransitionOut,
noinline content: @Composable (screen: S) -> Unit noinline content: @Composable (screen: S) -> Unit
) { ) {
val currentRoute by currentScreen.collectAsState() val currentRoute = currentScreen.collectAsState()
val useBackAnimation by useBackAnimation.collectAsState() val useBackAnimation by useBackAnimation.collectAsState()
val shouldShow by remember {
derivedStateOf {
currentRoute.value == getRoute<S>()
}
}
CustomAnimatedVisibility( CustomAnimatedVisibility(
visible = currentRoute == getRoute<S>(), visible = shouldShow,
enter = if (!useBackAnimation) enterAnim else backEnterAnim, enter = if (!useBackAnimation) enterAnim else backEnterAnim,
exit = if (!useBackAnimation) exitAnim else backExitAnim exit = if (!useBackAnimation) exitAnim else backExitAnim
) { ) {
@ -309,11 +314,11 @@ class Navigator @AssistedInject constructor(
content: @Composable () -> Unit content: @Composable () -> Unit
) { ) {
val activity = LocalContext.current as ComponentActivity val activity = LocalContext.current as ComponentActivity
val currentScreen by currentScreen.collectAsState() val currentScreen = currentScreen.collectAsState()
val useBackAnimation by useBackAnimation.collectAsState() val useBackAnimation by useBackAnimation.collectAsState()
val shouldShow by remember(currentScreen) { val shouldShow by remember {
derivedStateOf { derivedStateOf {
screens.any { it == currentScreen } screens.any { it == currentScreen.value }
} }
} }