Cleanup user client info sheet. Hide button if you don't have persmission to do so.

2 files changed, 35 insertions, 40 deletions

diff --git a/Hotline/macOS/MessageView.swift b/Hotline/macOS/MessageView.swift
index 861a14a..6690a88 100644
--- a/Hotline/macOS/MessageView.swift
+++ b/Hotline/macOS/MessageView.swift
@@ -1,37 +1,5 @@
 import SwiftUI
 
-struct UserInfo: Identifiable {
-  let username: String
-  let data: String
-  
-  var id: String {
-    self.username
-  }
-}
-
-struct UserInfoView: View {
-  @Environment(\.dismiss) private var dismiss
-  
-  let info: UserInfo
-  
-  var body: some View {
-    ScrollView(.vertical) {
-      Text(self.info.data)
-        .fontDesign(.monospaced)
-        .textSelection(.enabled)
-        .padding()
-    }
-    .frame(width: 300, height: 400)
-    .toolbar {
-      if #available(macOS 26.0, *) {
-        Button(role: .close) {
-          self.dismiss()
-        }
-      }
-    }
-  }
-}
-
 struct MessageView: View {
   @Environment(HotlineState.self) private var model: HotlineState
   @Environment(\.colorScheme) private var colorScheme
@@ -39,7 +7,7 @@ struct MessageView: View {
   @State private var input: String = ""
   @State private var scrollPos: Int?
   @State private var contentHeight: CGFloat = 0
-  @State private var userInfo: UserInfo?
+  @State private var userInfo: HotlineUserClientInfo?
   
   @Namespace private var bottomID
   @FocusState private var focusedField: FocusedField?
@@ -60,11 +28,13 @@ struct MessageView: View {
     }
     .background(Color(nsColor: .textBackgroundColor))
     .toolbar {
-      ToolbarItem {
-        Button {
-          self.getUserInfo()
-        } label: {
-          Image(systemName: "info.circle")
+      if self.model.access?.contains(.canGetClientInfo) == true {
+        ToolbarItem {
+          Button {
+            self.getUserInfo()
+          } label: {
+            Image(systemName: "info.circle")
+          }
         }
       }
     }
@@ -75,8 +45,8 @@ struct MessageView: View {
   
   private func getUserInfo() {
     Task {
-      if let (username, info) = try await self.model.getClientInfoText(id: self.userID) {
-        self.userInfo = UserInfo(username: username, data: info)
+      if let info = try await self.model.getClientInfoText(id: self.userID) {
+        self.userInfo = info
       }
     }
   }
diff --git a/Hotline/macOS/UserInfo.swift b/Hotline/macOS/UserInfo.swift
new file mode 100644
index 0000000..328fc02
--- /dev/null
+++ b/Hotline/macOS/UserInfo.swift
@@ -0,0 +1,25 @@
+import SwiftUI
+
+struct UserInfoView: View {
+  @Environment(\.dismiss) private var dismiss
+  
+  let info: HotlineUserClientInfo
+  
+  var body: some View {
+    ScrollView(.vertical) {
+      Text(self.info.details)
+        .fontDesign(.monospaced)
+        .textSelection(.enabled)
+        .frame(maxWidth: .infinity, alignment: .topLeading)
+        .padding()
+    }
+    .frame(width: 350, height: 400)
+    .toolbar {
+      ToolbarItem(placement: .confirmationAction) {
+        Button("OK") {
+          self.dismiss()
+        }
+      }
+    }
+  }
+}