General improvements (#68)

* nav animation improvements
* general improvements

Author: psuzn

shared/src/commonMain/kotlin/me/sujanpoudel/playdeals/common/navigation/NavHost.kt

@@ -34,7 +34,7 @@ fun NavHost(navGraph: NavGraph) {
     currentEntry?.also { entry ->
       AnimatedContent(entry, transitionSpec = navigator.transitionSpec) {
         savableStateHolder.SaveableStateProvider(it.id) {
-          entry.destination.content()
+          it.destination.content()
         }
       }
     }

shared/src/commonMain/kotlin/me/sujanpoudel/playdeals/common/ui/screens/home/HomeScreenViewModel.kt

@@ -2,10 +2,7 @@ package me.sujanpoudel.playdeals.common.ui.screens.home
 
 import io.ktor.util.reflect.instanceOf
 import kotlinx.coroutines.delay
-import kotlinx.coroutines.flow.MutableStateFlow
-import kotlinx.coroutines.flow.StateFlow
 import kotlinx.coroutines.flow.combine
-import kotlinx.coroutines.flow.update
 import kotlinx.coroutines.launch
 import me.sujanpoudel.playdeals.common.BuildKonfig
 import me.sujanpoudel.playdeals.common.Screens
@@ -27,9 +24,7 @@ class HomeScreenViewModel(
   private val forexRepository: ForexRepository,
 ) : ViewModel() {
 
-  private val _state = MutableStateFlow(HomeScreenState(lastUpdatedTime = appPreferences.lastUpdatedTime.value))
-  val state = _state as StateFlow<HomeScreenState>
-
+  val state = state(HomeScreenState(lastUpdatedTime = appPreferences.lastUpdatedTime.value))
   val searchTerm = state("")
 
   init {
@@ -51,7 +46,7 @@ class HomeScreenViewModel(
         }
       }
       .collect { deals ->
-        _state.update { state ->
+        state.update { state ->
           state.copy(
             persistentError = if (deals.isNotEmpty()) null else state.persistentError,
             errorOneOff = if (deals.isNotEmpty()) state.persistentError else null,
@@ -68,17 +63,15 @@ class HomeScreenViewModel(
     if (BuildKonfig.MAJOR_RELEASE && appPreferences.getChangelogShownVersion() != BuildKonfig.VERSION_CODE) {
       viewModelScope.launch {
         delay(1000)
-        _state.update {
-          it.copy(
-            destinationOneOff = Screens.CHANGELOG,
-          )
+        state.update {
+          it.copy(destinationOneOff = Screens.CHANGELOG)
         }
       }
     }
   }
 
   fun refreshDeals() {
-    _state.update { state ->
+    state.update { state ->
       state.copy(
         isLoading = state.allDeals.isEmpty(),
         isRefreshing = state.allDeals.isNotEmpty(),
@@ -89,7 +82,7 @@ class HomeScreenViewModel(
 
     viewModelScope.launch {
       val result = dealsRepository.refreshDeals()
-      _state.update { state ->
+      state.update { state ->
         when (result) {
           is Result.Error -> state.copy(
             isLoading = false,
@@ -116,19 +109,19 @@ class HomeScreenViewModel(
   }
 
   fun clearErrorOneOff() {
-    _state.update { state ->
+    state.update { state ->
       state.copy(errorOneOff = null)
     }
   }
 
   fun clearOneOffDestination() {
-    _state.update { state ->
+    state.update { state ->
       state.copy(destinationOneOff = null)
     }
   }
 
   fun toggleFilterItem(item: DealFilterOption) {
-    _state.update { state ->
+    state.update { state ->
       state.copy(
         filterOptions =
           state.filterOptions.map {

shared/src/commonMain/kotlin/me/sujanpoudel/playdeals/common/ui/screens/settings/SettingsScreenViewModel.kt

@@ -1,6 +1,5 @@
 package me.sujanpoudel.playdeals.common.ui.screens.settings
 
-import kotlinx.coroutines.flow.MutableStateFlow
 import kotlinx.coroutines.flow.SharingStarted
 import kotlinx.coroutines.flow.StateFlow
 import kotlinx.coroutines.flow.collectLatest
@@ -28,12 +27,14 @@ class SettingsScreenViewModel(
   val forexRates: StateFlow<List<ForexRateEntity>> = forexRepository.forexRatesFlow()
     .stateIn(viewModelScope, SharingStarted.Eagerly, emptyList())
 
-  val preferredConversion = MutableStateFlow(defaultUsdConversion())
+  val preferredConversion = state(defaultUsdConversion())
 
   init {
     viewModelScope.launch {
       forexRepository.preferredConversionRateFlow()
-        .collectLatest { preferredConversion.value = it }
+        .collectLatest { entity ->
+          preferredConversion.update { entity }
+        }
     }
   }
 

shared/src/commonMain/kotlin/me/sujanpoudel/playdeals/common/viewModel/ViewModel.kt

@@ -3,6 +3,7 @@ package me.sujanpoudel.playdeals.common.viewModel
 import kotlinx.coroutines.flow.MutableStateFlow
 import kotlinx.coroutines.flow.StateFlow
 import kotlinx.coroutines.flow.update
+import kotlin.jvm.JvmInline
 
 @OptIn(ExperimentalStdlibApi::class)
 open class ViewModel() : AutoCloseable {
@@ -41,11 +42,12 @@ open class ViewModel() : AutoCloseable {
 
   sealed interface VMState<T> : StateFlow<T>
 
-  private class VMStateImpl<T> constructor(
+  @JvmInline
+  value class VMStateImpl<T>(
     private val delegate: MutableStateFlow<T>,
   ) : MutableStateFlow<T> by delegate, VMState<T>
 
-  fun <T> VMState<T>.update(function: (T) -> T) {
+  protected fun <T> VMState<T>.update(function: (T) -> T) {
     (this as VMStateImpl<T> as MutableStateFlow<T>).update(function)
   }