Skip to content

Commit c22d6cf

Browse files
Product detail and player issue fixed (#53)
1 parent 7355688 commit c22d6cf

File tree

2 files changed

+13
-6
lines changed

2 files changed

+13
-6
lines changed

app/src/main/java/com/techlads/composetv/features/details/ProductDetailsScreen.kt

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
package com.techlads.composetv.features.details
22

3+
import androidx.activity.compose.BackHandler
34
import androidx.compose.foundation.Image
45
import androidx.compose.foundation.background
56
import androidx.compose.foundation.layout.*
@@ -24,12 +25,15 @@ import com.techlads.composetv.widgets.BorderedFocusableItem
2425
import com.techlads.composetv.widgets.Button
2526

2627
@Composable
27-
fun ProductDetailsScreen() {
28-
ProductDetailsContent()
28+
fun ProductDetailsScreen(onBackPressed: () -> Unit) {
29+
ProductDetailsContent(onBackPressed)
2930
}
3031

3132
@Composable
32-
private fun ProductDetailsContent() {
33+
private fun ProductDetailsContent(onBackPressed: () -> Unit) {
34+
35+
BackHandler(onBack = onBackPressed)
36+
3337
Box {
3438
SearchIcon(
3539
modifier = Modifier
@@ -221,6 +225,6 @@ fun ThumbnailImageCard(modifier: Modifier, parent: Int, child: Int) {
221225
@Composable
222226
fun DetailsScreenPrev() {
223227
Material3Theme {
224-
ProductDetailsScreen()
228+
ProductDetailsScreen {}
225229
}
226230
}

app/src/main/java/com/techlads/composetv/navigation/AppNavigation.kt

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ fun AppNavigation(navController: NavHostController, viewModel: HomeViewModel) {
4545
enterTransition = { tabEnterTransition() },
4646
exitTransition = { tabExitTransition() }) {
4747
HomeScreen(viewModel) { _, _ ->
48-
navController.navigate(Screens.Player.title)
48+
navController.navigate(Screens.ProductDetail.title)
4949
}
5050
}
5151

@@ -64,7 +64,10 @@ fun AppNavigation(navController: NavHostController, viewModel: HomeViewModel) {
6464
Screens.ProductDetail.title,
6565
enterTransition = { tabEnterTransition() },
6666
exitTransition = { tabExitTransition() }) {
67-
ProductDetailsScreen()
67+
ProductDetailsScreen(
68+
onBackPressed = {
69+
navController.navigateUp()
70+
})
6871
}
6972
}
7073
}

0 commit comments

Comments
 (0)