Fix navigation on search screen

alexymumo · Apr 1, 2024 · 3427344 · 3427344
1 parent 2064dd4
commit 3427344
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 3 deletions.
diff --git a/presentation/src/main/java/com/alexmumo/presentation/detail/DetailScreen.kt b/presentation/src/main/java/com/alexmumo/presentation/detail/DetailScreen.kt
@@ -150,6 +150,7 @@ fun DetailScreen(
                         .fillMaxWidth()
                         .padding(all = 2.dp),
                     color = Color.White,
+                    maxLines = 20
                 )
                 Row(
                     verticalAlignment = Alignment.CenterVertically,

diff --git a/presentation/src/main/java/com/alexmumo/presentation/search/SearchScreen.kt b/presentation/src/main/java/com/alexmumo/presentation/search/SearchScreen.kt
@@ -47,6 +47,8 @@ import androidx.hilt.navigation.compose.hiltViewModel
 import androidx.navigation.NavController
 import androidx.navigation.compose.rememberNavController
 import com.alexmumo.presentation.components.NewsCard
+import com.alexmumo.presentation.navigation.NavItem
+import timber.log.Timber
 
 @Composable
 fun SearchScreen(
@@ -63,7 +65,8 @@ fun SearchScreen(
         onSearchTextChange = { text ->
             viewModel.setSearchString(text)
         },
-        searchState = searchState
+        searchState = searchState,
+        navController = navController
     )
 }
 
@@ -73,7 +76,8 @@ fun SearchContent(
     onSearch: (String) -> Unit,
     currentString: String,
     onSearchTextChange: (String) -> Unit,
-    searchState: SearchState
+    searchState: SearchState,
+    navController: NavController
 ) {
     Column(
         modifier = Modifier
@@ -89,7 +93,14 @@ fun SearchContent(
         Spacer(modifier = Modifier.height(5.dp))
         LazyColumn {
             items(searchState.data) { article ->
-                NewsCard(onNavigate = {}, article = article)
+                NewsCard(onNavigate = {
+                    navController.currentBackStackEntry?.savedStateHandle?.set(
+                        key = "news",
+                        value = article
+                    )
+                    navController.navigate(NavItem.Detail.route)
+                    Timber.tag("Logged ${NavItem.Detail.route}")
+                }, article = article)
             }
         }
     }