in conv list: show loading spinner as long as request is running

Problem
The shimmer loading skeleton (and the pull-to-refresh spinner) were dismissed after the first emission from the local Room database — even if that emission was an empty list. The network fetch still running in the background was invisible to the user, causing an empty-state flash followed by the actual data appearing.
Changes
ConversationsListViewModel.kt
Removed _isShimmerVisible: MutableStateFlow(true) (which was toggled off prematurely on any DB emission).
Added _isLoadingRooms: MutableStateFlow(true) — set to true when getRooms() starts, false only after the repository Job fully completes (both local DB and network sync are done).
isShimmerVisible is now a combine of _isLoadingRooms and getRoomsStateFlow: shimmer is visible while loading is in progress AND the room list is still empty. This means:
Initial load with no cached data → shimmer stays until the network fetch finishes.
Subsequent pull-to-refresh with existing data → shimmer doesn't re-appear (list isn't empty), but the swipe indicator handles that.
Moved getRoomsStateFlow declaration before isShimmerVisible to fix initialization order.
ConversationsListActivity.kt
Removed isRefreshingState.value = false from the GetRoomsSuccessState branch (which fired too early, after the first DB emission).
Added a isLoadingRooms collector: isRefreshingState is set to false only when isLoadingRooms becomes false, i.e. after the full load cycle completes.

AI-assistant: Github Copilot 1.8.2-243 (Claude Sonnet 4.6)

Signed-off-by: Marcel Hibbe <dev@mhibbe.de>

Author: mahibi

app/src/main/java/com/nextcloud/talk/conversationlist/ConversationsListActivity.kt

@@ -384,10 +384,6 @@ class ConversationsListActivity : BaseActivity() {
     private fun initObservers() {
         conversationsListViewModel.getRoomsViewState.observe(this) { state ->
             when (state) {
-                is ConversationsListViewModel.GetRoomsSuccessState -> {
-                    isRefreshingState.value = false
-                }
-
                 is ConversationsListViewModel.GetRoomsErrorState -> {
                     isRefreshingState.value = false
                     handleHttpExceptions(state.throwable)
@@ -397,6 +393,14 @@ class ConversationsListActivity : BaseActivity() {
             }
         }
 
+        lifecycleScope.launch {
+            conversationsListViewModel.isLoadingRooms.collect { isLoading ->
+                if (!isLoading) {
+                    isRefreshingState.value = false
+                }
+            }
+        }
+
         lifecycleScope.launch {
             conversationsListViewModel.getRoomsFlow
                 .onEach { list ->

app/src/main/java/com/nextcloud/talk/conversationlist/viewmodels/ConversationsListViewModel.kt

@@ -139,20 +139,30 @@ class ConversationsListViewModel @Inject constructor(
             _getRoomsViewState.value = GetRoomsErrorState(it)
         }
 
-    private val _isShimmerVisible = MutableStateFlow(true)
+    private val _isLoadingRooms = MutableStateFlow(true)
 
     /**
-     * Drives the shimmer skeleton visibility. Set to false as soon as the first room-list
-     * emission arrives (same subscription as [getRoomsStateFlow] so it hides in the same
-     * coroutine step that populates [conversationListEntriesFlow].
+     * True while a [getRooms] call is in progress (from start until the fetch-Job completes,
+     * i.e. after both the local-DB and the network-sync emissions have been produced).
      */
-    val isShimmerVisible: StateFlow<Boolean> = _isShimmerVisible.asStateFlow()
+    val isLoadingRooms: StateFlow<Boolean> = _isLoadingRooms.asStateFlow()
 
     val getRoomsStateFlow = repository
         .roomListFlow
-        .onEach { _isShimmerVisible.value = false }
         .stateIn(viewModelScope, SharingStarted.Eagerly, listOf())
 
+    /**
+     * Drives the shimmer skeleton visibility. Visible while rooms are still being loaded for
+     * the first time (no data in the list yet). Hidden as soon as either the room list is
+     * non-empty or the loading job has fully completed.
+     */
+    val isShimmerVisible: StateFlow<Boolean> = combine(
+        _isLoadingRooms,
+        getRoomsStateFlow
+    ) { isLoading, rooms ->
+        isLoading && rooms.isEmpty()
+    }.stateIn(viewModelScope, SharingStarted.Eagerly, true)
+
     private val _federationInvitationHintVisible = MutableStateFlow(false)
     val federationInvitationHintVisible: StateFlow<Boolean> = _federationInvitationHintVisible.asStateFlow()
 
@@ -406,7 +416,12 @@ class ConversationsListViewModel @Inject constructor(
     fun getRooms(user: User) {
         val startNanoTime = System.nanoTime()
         Log.d(TAG, "fetchData - getRooms - calling: $startNanoTime")
-        repository.getRooms(user)
+        _isLoadingRooms.value = true
+        val job = repository.getRooms(user)
+        viewModelScope.launch {
+            job.join()
+            _isLoadingRooms.value = false
+        }
     }
 
     fun checkIfThreadsExist() {