From f1a7982a4de58b13eb0b4f30bbe8cb2374f60fef Mon Sep 17 00:00:00 2001 From: James Rich <2199651+jamesarich@users.noreply.github.com> Date: Wed, 24 Jun 2026 11:00:17 -0500 Subject: [PATCH] refactor(core:ui): drop redundant SinglePaneSceneStrategy from NavDisplay (#5934) Co-authored-by: Claude Opus 4.8 --- .../core/ui/component/MeshtasticNavDisplay.kt | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/core/ui/src/commonMain/kotlin/org/meshtastic/core/ui/component/MeshtasticNavDisplay.kt b/core/ui/src/commonMain/kotlin/org/meshtastic/core/ui/component/MeshtasticNavDisplay.kt index c05cb71be..173f27c0b 100644 --- a/core/ui/src/commonMain/kotlin/org/meshtastic/core/ui/component/MeshtasticNavDisplay.kt +++ b/core/ui/src/commonMain/kotlin/org/meshtastic/core/ui/component/MeshtasticNavDisplay.kt @@ -40,7 +40,6 @@ import androidx.navigation3.runtime.NavKey import androidx.navigation3.runtime.rememberSaveableStateHolderNavEntryDecorator import androidx.navigation3.scene.DialogSceneStrategy import androidx.navigation3.scene.Scene -import androidx.navigation3.scene.SinglePaneSceneStrategy import androidx.navigation3.ui.NavDisplay import org.meshtastic.core.navigation.MultiBackstack @@ -129,13 +128,8 @@ fun MeshtasticNavDisplay( backStack.removeLastOrNull() } }, - sceneStrategies = - listOf( - DialogSceneStrategy(), - listDetailSceneStrategy, - supportingPaneSceneStrategy, - SinglePaneSceneStrategy(), - ), + // NavDisplay falls back to SinglePaneSceneStrategy automatically when none of these compute a Scene. + sceneStrategies = listOf(DialogSceneStrategy(), listDetailSceneStrategy, supportingPaneSceneStrategy), sharedTransitionScope = this@SharedTransitionLayout, transitionSpec = meshtasticTransitionSpec(), popTransitionSpec = meshtasticTransitionSpec(),