From a6b468f1d297161e4ffd4f09e471bf5b39405ed9 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ga=C3=ABtan=20Muller?= Date: Wed, 29 Nov 2023 15:52:56 +0100 Subject: [PATCH] Update "Examples" as well --- .../pillarbox/demo/tv/examples/Examples.kt | 47 +++++++++++++++---- 1 file changed, 37 insertions(+), 10 deletions(-) diff --git a/pillarbox-demo-tv/src/main/java/ch/srgssr/pillarbox/demo/tv/examples/Examples.kt b/pillarbox-demo-tv/src/main/java/ch/srgssr/pillarbox/demo/tv/examples/Examples.kt index 53b2f8677..4655c2621 100644 --- a/pillarbox-demo-tv/src/main/java/ch/srgssr/pillarbox/demo/tv/examples/Examples.kt +++ b/pillarbox-demo-tv/src/main/java/ch/srgssr/pillarbox/demo/tv/examples/Examples.kt @@ -16,6 +16,7 @@ import androidx.compose.runtime.derivedStateOf import androidx.compose.runtime.getValue import androidx.compose.runtime.mutableIntStateOf import androidx.compose.runtime.remember +import androidx.compose.runtime.rememberCoroutineScope import androidx.compose.runtime.setValue import androidx.compose.ui.Alignment import androidx.compose.ui.ExperimentalComposeUiApi @@ -38,6 +39,7 @@ import androidx.compose.ui.text.style.TextAlign import androidx.compose.ui.text.style.TextOverflow import androidx.compose.ui.tooling.preview.Preview import androidx.compose.ui.unit.dp +import androidx.navigation.NavHostController import androidx.navigation.NavType import androidx.navigation.compose.NavHost import androidx.navigation.compose.composable @@ -46,6 +48,7 @@ import androidx.navigation.navArgument import androidx.tv.foundation.lazy.grid.TvGridCells import androidx.tv.foundation.lazy.grid.TvLazyVerticalGrid import androidx.tv.foundation.lazy.grid.itemsIndexed +import androidx.tv.foundation.lazy.grid.rememberTvLazyGridState import androidx.tv.material3.Card import androidx.tv.material3.ExperimentalTvMaterial3Api import androidx.tv.material3.MaterialTheme @@ -55,6 +58,7 @@ import ch.srgssr.pillarbox.demo.shared.data.Playlist import ch.srgssr.pillarbox.demo.shared.ui.NavigationRoutes import ch.srgssr.pillarbox.demo.tv.ui.theme.PillarboxTheme import ch.srgssr.pillarbox.demo.tv.ui.theme.paddings +import kotlinx.coroutines.launch /** * Examples home @@ -81,6 +85,7 @@ fun ExamplesHome( ExamplesSection( modifier = modifier, items = playlists, + navController = navController, onItemClick = { index, _ -> navController.navigate("${NavigationRoutes.homeSample}/$index") } @@ -114,6 +119,7 @@ fun ExamplesHome( ExamplesSection( title = playlist.title, items = playlist.items, + navController = navController, onItemClick = { _, item -> onItemSelected(item) } @@ -154,6 +160,7 @@ private fun ExamplesSection( modifier: Modifier = Modifier, title: String? = null, items: List, + navController: NavHostController, onItemClick: (index: Int, item: T) -> Unit, content: @Composable (item: T) -> Unit ) { @@ -166,15 +173,7 @@ private fun ExamplesSection( } Column( - modifier = modifier - .onPreviewKeyEvent { - if (it.key == Key.DirectionUp && it.type == KeyEventType.KeyDown && isOnFirstRow) { - focusedIndex = -1 - focusManager.moveFocus(FocusDirection.Up) - } else { - false - } - }, + modifier = modifier, verticalArrangement = Arrangement.spacedBy(MaterialTheme.paddings.baseline) ) { if (title != null) { @@ -184,9 +183,37 @@ private fun ExamplesSection( ) } + val coroutineScope = rememberCoroutineScope() + val scrollState = rememberTvLazyGridState() + TvLazyVerticalGrid( columns = TvGridCells.Fixed(columnCount), - modifier = Modifier.focusRestorer(), + modifier = Modifier + .focusRestorer() + .onPreviewKeyEvent { + if (it.key == Key.DirectionUp && it.type == KeyEventType.KeyDown && isOnFirstRow) { + focusedIndex = -1 + focusManager.moveFocus(FocusDirection.Up) + } else if (it.key == Key.Back && it.type == KeyEventType.KeyDown) { + if (!isOnFirstRow) { + focusedIndex = 0 + + coroutineScope.launch { + scrollState.animateScrollToItem(focusedIndex) + } + + true + } else if (navController.previousBackStackEntry == null) { + focusedIndex = -1 + focusManager.moveFocus(FocusDirection.Up) + } else { + false + } + } else { + false + } + }, + state = scrollState, contentPadding = PaddingValues(vertical = MaterialTheme.paddings.baseline), verticalArrangement = Arrangement.spacedBy(MaterialTheme.paddings.baseline), horizontalArrangement = Arrangement.spacedBy(MaterialTheme.paddings.baseline)