Fix a bug where the pinned items banner could overlay the composer. (#…

…3441)
element-hq · Oct 24, 2024 · 2b82959 · 2b82959
1 parent f26159a
commit 2b82959
Showing 1 changed file with 8 additions and 8 deletions.
diff --git a/ElementX/Sources/Screens/RoomScreen/View/RoomScreen.swift b/ElementX/Sources/Screens/RoomScreen/View/RoomScreen.swift
@@ -28,6 +28,14 @@ struct RoomScreen: View {
     var body: some View {
         timeline
             .background(Color.compound.bgCanvasDefault.ignoresSafeArea())
+            .overlay(alignment: .top) {
+                Group {
+                    if roomContext.viewState.shouldShowPinnedEventsBanner {
+                        pinnedItemsBanner
+                    }
+                }
+                .animation(.elementDefault, value: roomContext.viewState.shouldShowPinnedEventsBanner)
+            }
             .safeAreaInset(edge: .bottom, spacing: 0) {
                 VStack(spacing: 0) {
                     RoomScreenFooterView(details: roomContext.viewState.footerDetails,
@@ -52,14 +60,6 @@ struct RoomScreen: View {
                         .environment(\.shouldAutomaticallyLoadImages, !timelineContext.viewState.hideTimelineMedia)
                 }
             }
-            .overlay(alignment: .top) {
-                Group {
-                    if roomContext.viewState.shouldShowPinnedEventsBanner {
-                        pinnedItemsBanner
-                    }
-                }
-                .animation(.elementDefault, value: roomContext.viewState.shouldShowPinnedEventsBanner)
-            }
             .navigationTitle(L10n.screenRoomTitle) // Hidden but used for back button text.
             .navigationBarTitleDisplayMode(.inline)
             .navigationBarHidden(isNavigationBarHidden)