Fix #2652 - Room list looping and fighting scrolling after filtering

- also fixes filtering empty state vertical centering

ElementX/Sources/Screens/HomeScreen/HomeScreenModels.swift

@@ -121,6 +121,18 @@ struct HomeScreenViewState: BindableState {
     var shouldShowEmptyFilterState: Bool {
         !bindings.isSearchFieldFocused && bindings.filtersState.isFiltering && visibleRooms.isEmpty
     }
+    
+    var shouldShowFilters: Bool {
+        !bindings.isSearchFieldFocused
+    }
+    
+    var shouldShowRecoveryKeyConfirmationBanner: Bool {
+        securityBannerMode == .recoveryKeyConfirmation
+    }
+    
+    var shouldShowInvitesButton: Bool {
+        !ServiceLocator.shared.settings.roomListInvitesEnabled && hasPendingInvitations && !bindings.isSearchFieldFocused
+    }
 }
 
 struct HomeScreenViewStateBindings {

ElementX/Sources/Screens/HomeScreen/View/HomeScreenContent.swift

@@ -14,6 +14,7 @@
 // limitations under the License.
 //
 
+import Compound
 import SwiftUI
 
 struct HomeScreenContent: View {
@@ -54,10 +55,7 @@ struct HomeScreenContent: View {
                 case .rooms:
                     LazyVStack(spacing: 0) {
                         Section {
-                            if context.viewState.shouldShowEmptyFilterState {
+                            if !context.viewState.shouldShowEmptyFilterState {
-                                RoomListFiltersEmptyStateView(state: context.filtersState)
-                                    .frame(height: geometry.size.height)
-                            } else {
                                 HomeScreenRoomList(context: context)
                             }
                         } header: {
@@ -88,12 +86,19 @@ struct HomeScreenContent: View {
             .onChange(of: context.viewState.visibleRooms) { _ in
                 updateVisibleRange()
             }
-            .background(
+            .background {
-                Button("", action: {
+                Button("") {
                     context.send(viewAction: .globalSearch)
-                })
+                }
                 .keyboardShortcut(KeyEquivalent("k"), modifiers: [.command])
-            )
+            }
+            .overlay {
+                if context.viewState.shouldShowEmptyFilterState {
+                    RoomListFiltersEmptyStateView(state: context.filtersState)
+                        .background(.compound.bgCanvasDefault)
+                        .frame(maxWidth: .infinity, maxHeight: .infinity)
+                }
+            }
             .scrollDismissesKeyboard(.immediately)
             .scrollDisabled(context.viewState.roomListMode == .skeletons)
             .scrollBounceBehavior(context.viewState.roomListMode == .empty ? .basedOnSize : .automatic)
@@ -104,16 +109,20 @@ struct HomeScreenContent: View {
     
     @ViewBuilder
     private var topSection: some View {
+        // An empty VStack causes glitches within the room list
+        if context.viewState.shouldShowFilters ||
+            context.viewState.shouldShowRecoveryKeyConfirmationBanner ||
+            context.viewState.shouldShowInvitesButton {
             VStack(spacing: 0) {
-            if !context.isSearchFieldFocused {
+                if context.viewState.shouldShowFilters {
                     RoomListFiltersView(state: $context.filtersState)
                 }
                 
-            if context.viewState.securityBannerMode == .recoveryKeyConfirmation {
+                if context.viewState.shouldShowRecoveryKeyConfirmationBanner {
                     HomeScreenRecoveryKeyConfirmationBanner(context: context)
                 }
                 
-            if context.viewState.hasPendingInvitations, !context.isSearchFieldFocused, !ServiceLocator.shared.settings.roomListInvitesEnabled {
+                if context.viewState.shouldShowInvitesButton {
                     HomeScreenInvitesButton(title: L10n.actionInvitesList, hasBadge: context.viewState.hasUnreadPendingInvitations) {
                         context.send(viewAction: .selectInvites)
                     }
@@ -123,6 +132,7 @@ struct HomeScreenContent: View {
             }
             .background(Color.compound.bgCanvasDefault)
         }
+    }
     
     @ViewBuilder
     private var migrationView: some View {

changelog.d/2652.bugfix

@@ -0,0 +1 @@
+Fix room list looping and fighting scrolling after filtering