Change the Sign Out button to be ListRowButton. Sets a better heigh…

…t value using `maxHeight` to ensure that it doesn't exceed the `ListRow` sizing.
jellyfin · Jan 29, 2025 · 0a40e30 · 0a40e30
1 parent 3f20320
commit 0a40e30
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 14 deletions.
diff --git a/Swiftfin tvOS/Components/ListRowButton.swift b/Swiftfin tvOS/Components/ListRowButton.swift
@@ -56,7 +56,7 @@ struct ListRowButton: View {
             }
         }
         .buttonStyle(.card)
-        .frame(height: 75)
+        .frame(maxHeight: 75)
         .focused($isFocused)
     }
 

diff --git a/Swiftfin tvOS/Views/SettingsView/SettingsView.swift b/Swiftfin tvOS/Views/SettingsView/SettingsView.swift
@@ -44,22 +44,14 @@ struct SettingsView: View {
                     .onSelect {
                         router.route(to: \.serverDetail, viewModel.userSession.server)
                     }
+                }
 
-                    Button {
+                Section {
+                    ListRowButton(L10n.switchUser) {
                         viewModel.signOut()
-                    } label: {
-                        HStack {
-
-                            Text(L10n.switchUser)
-                                .foregroundColor(.jellyfinPurple)
-
-                            Spacer()
-
-                            Image(systemName: "chevron.right")
-                                .font(.body.weight(.regular))
-                                .foregroundColor(.secondary)
-                        }
                     }
+                    .foregroundStyle(Color.jellyfinPurple.overlayColor, Color.jellyfinPurple)
+                    .listRowInsets(.zero)
                 }
 
                 Section(L10n.videoPlayer) {