Add loading state to saved trips

ksharma-xyz · Nov 24, 2024 · 3bb6c89 · 3bb6c89
1 parent 3f4e7b5
commit 3bb6c89
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 3 deletions.
diff --git a/...rc/commonMain/kotlin/xyz/ksharma/krail/trip/planner/ui/state/savedtrip/SavedTripsState.kt b/...rc/commonMain/kotlin/xyz/ksharma/krail/trip/planner/ui/state/savedtrip/SavedTripsState.kt
@@ -4,4 +4,7 @@ import kotlinx.collections.immutable.ImmutableList
 import kotlinx.collections.immutable.persistentListOf
 import xyz.ksharma.krail.trip.planner.ui.state.timetable.Trip
 
-data class SavedTripsState(val savedTrips: ImmutableList<Trip> = persistentListOf())
+data class SavedTripsState(
+    val savedTrips: ImmutableList<Trip> = persistentListOf(),
+    val isLoading: Boolean = true,
+)
diff --git a/...ui/src/commonMain/kotlin/xyz/ksharma/krail/trip/planner/ui/savedtrips/SavedTripsScreen.kt b/...ui/src/commonMain/kotlin/xyz/ksharma/krail/trip/planner/ui/savedtrips/SavedTripsScreen.kt
@@ -94,7 +94,7 @@ fun SavedTripsScreen(
                     Spacer(modifier = Modifier.height(12.dp))
                 }
 
-                if (savedTripsState.savedTrips.isEmpty()) {
+                if (savedTripsState.savedTrips.isEmpty() && savedTripsState.isLoading.not()) {
                     item(key = "empty_state") {
                         ErrorMessage(
                             emoji = "🌟",

diff --git a/...src/commonMain/kotlin/xyz/ksharma/krail/trip/planner/ui/savedtrips/SavedTripsViewModel.kt b/...src/commonMain/kotlin/xyz/ksharma/krail/trip/planner/ui/savedtrips/SavedTripsViewModel.kt
@@ -25,6 +25,7 @@ class SavedTripsViewModel(
 
     private fun loadSavedTrips() {
         viewModelScope.launch(context = Dispatchers.IO) {
+            updateUiState { copy(isLoading = true) }
             val trips = mutableSetOf<Trip>()
 
             val savedTrips = sandook.selectAllTrips()
@@ -42,7 +43,7 @@ class SavedTripsViewModel(
                 println("\t SavedTrip: #$index ${trip.fromStopName} -> ${trip.toStopName}")
             }
 
-            updateUiState { copy(savedTrips = trips.toImmutableList()) }
+            updateUiState { copy(savedTrips = trips.toImmutableList(), isLoading = false) }
         }
     }