NewsScreen: Use LaunchedEffect to load article

Signed-off-by: Aayush Gupta <[email protected]>

app/src/main/java/io/aayush/relabs/ui/screens/news/NewsScreen.kt

@@ -1,8 +1,5 @@
 package io.aayush.relabs.ui.screens.news
 
-import android.net.Uri
-import android.util.Log
-import androidx.browser.customtabs.CustomTabsIntent
 import androidx.compose.foundation.isSystemInDarkTheme
 import androidx.compose.foundation.layout.fillMaxSize
 import androidx.compose.foundation.layout.padding
@@ -16,7 +13,11 @@ import androidx.compose.material3.TopAppBar
 import androidx.compose.material3.TopAppBarDefaults
 import androidx.compose.material3.rememberTopAppBarState
 import androidx.compose.runtime.Composable
+import androidx.compose.runtime.LaunchedEffect
 import androidx.compose.runtime.getValue
+import androidx.compose.runtime.mutableStateOf
+import androidx.compose.runtime.remember
+import androidx.compose.runtime.setValue
 import androidx.compose.ui.Modifier
 import androidx.compose.ui.input.nestedscroll.nestedScroll
 import androidx.compose.ui.platform.LocalContext
@@ -56,6 +57,11 @@ fun NewsScreen(
  ) {
  val articles: List<Article> by viewModel.feed.collectAsStateWithLifecycle()
  val context = LocalContext.current
+ var clicked by remember { mutableStateOf("") }
+
+ LaunchedEffect(key1 = clicked) {
+ if (clicked.isNotBlank()) viewModel.loadArticle(context, clicked)
+ }
 
  LazyColumn(modifier = Modifier.padding(it)) {
  items(articles) { article ->
@@ -79,15 +85,7 @@ fun NewsScreen(
  ).toString(),
  author = stringResource(id = R.string.posted_by, article.author ?: ""),
  date = article.pubDate ?: "",
- onClicked = {
- try {
- CustomTabsIntent.Builder()
- .build()
- .launchUrl(context, Uri.parse(article.link))
- } catch (exception: Exception) {
- Log.e(TAG, "Failed to open profile", exception)
- }
- }
+ onClicked = { clicked = article.link ?: "" }
  )
  }
  }

app/src/main/java/io/aayush/relabs/ui/screens/news/NewsViewModel.kt

@@ -1,5 +1,9 @@
 package io.aayush.relabs.ui.screens.news
 
+import android.content.Context
+import android.net.Uri
+import android.util.Log
+import androidx.browser.customtabs.CustomTabsIntent
 import androidx.lifecycle.ViewModel
 import androidx.lifecycle.viewModelScope
 import com.prof.rssparser.Article
@@ -12,9 +16,12 @@ import javax.inject.Inject
 
 @HiltViewModel
 class NewsViewModel @Inject constructor(
- private val rssNewsRepository: RSSNewsRepository
+ private val rssNewsRepository: RSSNewsRepository,
+ private val customTabIntent: CustomTabsIntent
 ) : ViewModel() {
 
+ private val TAG = NewsViewModel::class.java.simpleName
+
  private val _feed = MutableStateFlow<List<Article>>(emptyList())
  val feed = _feed.asStateFlow()
 
@@ -27,4 +34,12 @@ class NewsViewModel @Inject constructor(
  _feed.value = rssNewsRepository.getMobileFeed().getOrDefault(emptyList())
  }
  }
+
+ fun loadArticle(context: Context, link: String) {
+ try {
+ customTabIntent.launchUrl(context, Uri.parse(link))
+ } catch (exception: Exception) {
+ Log.e(TAG, "Failed to open profile", exception)
+ }
+ }
 }

app/src/main/java/io/aayush/relabs/utils/CommonModule.kt

@@ -2,6 +2,7 @@ package io.aayush.relabs.utils
 
 import android.content.Context
 import android.content.SharedPreferences
+import androidx.browser.customtabs.CustomTabsIntent
 import dagger.Module
 import dagger.Provides
 import dagger.hilt.InstallIn
@@ -23,4 +24,10 @@ object CommonModule {
  fun provideSharedPrefInstance(@ApplicationContext context: Context): SharedPreferences {
  return context.getSharedPreferences(context.packageName, Context.MODE_PRIVATE)
  }
+
+ @Singleton
+ @Provides
+ fun provideCustomTabIntent(): CustomTabsIntent {
+ return CustomTabsIntent.Builder().build()
+ }
 }