From 6727bed35e1853d28bb70bfe0ac93b428ac6c18f Mon Sep 17 00:00:00 2001 From: Bryan Van Deusen Date: Sun, 24 May 2026 22:28:36 -0400 Subject: [PATCH] fix(android): thread navController into inShellDetail extension PlaylistDetailScreen takes navController for its TopAppBar back-button, but the inShellDetail NavGraphBuilder extension was still scoped to just expandPlayer. Added navController as the first parameter and updated the call site. Co-Authored-By: Claude Opus 4.7 (1M context) --- .../java/com/fabledsword/minstrel/nav/MinstrelNavGraph.kt | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/android/app/src/main/java/com/fabledsword/minstrel/nav/MinstrelNavGraph.kt b/android/app/src/main/java/com/fabledsword/minstrel/nav/MinstrelNavGraph.kt index 25b57f8a..72f3a160 100644 --- a/android/app/src/main/java/com/fabledsword/minstrel/nav/MinstrelNavGraph.kt +++ b/android/app/src/main/java/com/fabledsword/minstrel/nav/MinstrelNavGraph.kt @@ -38,7 +38,7 @@ fun MinstrelNavGraph( modifier = modifier.fillMaxSize(), ) { inShellTopLevel(navController, expandPlayer) - inShellDetail(expandPlayer) + inShellDetail(navController, expandPlayer) outsideShell() } } @@ -89,7 +89,10 @@ private fun NavGraphBuilder.inShellTopLevel( } } -private fun NavGraphBuilder.inShellDetail(expandPlayer: () -> Unit) { +private fun NavGraphBuilder.inShellDetail( + navController: NavHostController, + expandPlayer: () -> Unit, +) { composable { backStackEntry -> val route: AlbumDetail = backStackEntry.toRoute() ShellScaffold(onExpandPlayer = expandPlayer) {