code improvement

element-hq · Jan 22, 2025 · 8d9603b · 8d9603b
1 parent 3566d0b
commit 8d9603b
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 3 deletions.
diff --git a/ElementX/Sources/Other/SwiftUI/Views/AvatarHeaderView.swift b/ElementX/Sources/Other/SwiftUI/Views/AvatarHeaderView.swift
@@ -40,7 +40,7 @@ struct AvatarHeaderView<Footer: View>: View {
 
         if let roomAlias = room.canonicalAlias {
             subtitle = roomAlias
-        } else if case let .heroes(heroes) = room.avatar, heroes.count == 1 {
+        } else if room.isDirect, case let .heroes(heroes) = room.avatar, heroes.count == 1 {
             subtitle = heroes[0].userID
         } else {
             subtitle = nil
@@ -200,7 +200,8 @@ struct AvatarHeaderView_Previews: PreviewProvider, TestablePreview {
                                                        avatarURL: .mockMXCAvatar),
                                          canonicalAlias: "#test:matrix.org",
                                          isEncrypted: true,
-                                         isPublic: true),
+                                         isPublic: true,
+                                         isDirect: false),
                              avatarSize: .room(on: .details),
                              mediaProvider: MediaProviderMock(configuration: .init())) {
                 HStack(spacing: 32) {

diff --git a/ElementX/Sources/Services/Room/RoomDetails.swift b/ElementX/Sources/Services/Room/RoomDetails.swift
@@ -14,4 +14,5 @@ struct RoomDetails {
     let canonicalAlias: String?
     let isEncrypted: Bool
     let isPublic: Bool
+    let isDirect: Bool
 }
diff --git a/ElementX/Sources/Services/Room/RoomProxyProtocol.swift b/ElementX/Sources/Services/Room/RoomProxyProtocol.swift
@@ -182,7 +182,8 @@ extension JoinedRoomProxyProtocol {
                     avatar: infoPublisher.value.avatar,
                     canonicalAlias: infoPublisher.value.canonicalAlias,
                     isEncrypted: isEncrypted,
-                    isPublic: infoPublisher.value.isPublic)
+                    isPublic: infoPublisher.value.isPublic,
+                    isDirect: infoPublisher.value.isDirect)
     }
 
     var isDirectOneToOneRoom: Bool {