From 56ac6d8848b8e37e6688be9fed445216ed9c5eb8 Mon Sep 17 00:00:00 2001 From: Arkadiusz Fal Date: Thu, 12 Feb 2026 08:51:54 +0100 Subject: [PATCH] Update media browser view options sheet layout --- Yattee/Localizable.xcstrings | 47 ++++++++++--------- .../MediaBrowserViewOptionsSheet.swift | 15 +----- 2 files changed, 26 insertions(+), 36 deletions(-) diff --git a/Yattee/Localizable.xcstrings b/Yattee/Localizable.xcstrings index b842b63e..7cb6730b 100644 --- a/Yattee/Localizable.xcstrings +++ b/Yattee/Localizable.xcstrings @@ -4908,6 +4908,7 @@ } }, "mediaBrowser.viewOptions.filters" : { + "extractionState" : "stale", "localizations" : { "en" : { "stringUnit" : { @@ -11299,6 +11300,17 @@ } } }, + "settings.notifications.disableAll" : { + "comment" : "Button to disable notifications for all channels", + "localizations" : { + "en" : { + "stringUnit" : { + "state" : "translated", + "value" : "Disable All" + } + } + } + }, "settings.notifications.enable" : { "comment" : "Toggle to enable background notifications", "localizations" : { @@ -11310,6 +11322,17 @@ } } }, + "settings.notifications.enableAll" : { + "comment" : "Button to enable notifications for all channels", + "localizations" : { + "en" : { + "stringUnit" : { + "state" : "translated", + "value" : "Enable All" + } + } + } + }, "settings.notifications.footer" : { "comment" : "Description for notifications toggle", "localizations" : { @@ -11354,28 +11377,6 @@ } } }, - "settings.notifications.disableAll" : { - "comment" : "Button to disable notifications for all channels", - "localizations" : { - "en" : { - "stringUnit" : { - "state" : "translated", - "value" : "Disable All" - } - } - } - }, - "settings.notifications.enableAll" : { - "comment" : "Button to enable notifications for all channels", - "localizations" : { - "en" : { - "stringUnit" : { - "state" : "translated", - "value" : "Enable All" - } - } - } - }, "settings.notifications.noSubscriptions.description" : { "comment" : "Description when no subscriptions exist", "localizations" : { @@ -17268,4 +17269,4 @@ } }, "version" : "1.0" -} +} \ No newline at end of file diff --git a/Yattee/Views/MediaBrowser/MediaBrowserViewOptionsSheet.swift b/Yattee/Views/MediaBrowser/MediaBrowserViewOptionsSheet.swift index 7d4d6457..236ea5c6 100644 --- a/Yattee/Views/MediaBrowser/MediaBrowserViewOptionsSheet.swift +++ b/Yattee/Views/MediaBrowser/MediaBrowserViewOptionsSheet.swift @@ -24,17 +24,14 @@ struct MediaBrowserViewOptionsSheet: View { NavigationStack { Form { Section { + Toggle("mediaBrowser.viewOptions.showOnlyPlayable", isOn: $showOnlyPlayable) Picker("mediaBrowser.viewOptions.sortBy", selection: $sortOrder) { ForEach(availableSortOptions) { order in Label(order.displayName, systemImage: order.systemImage) .tag(order) } } - } header: { - Text("mediaBrowser.viewOptions.sortBy") - } - Section { Picker("mediaBrowser.viewOptions.order", selection: $sortAscending) { Label(String(localized: "mediaBrowser.viewOptions.ascending"), systemImage: "arrow.up") .tag(true) @@ -44,14 +41,6 @@ struct MediaBrowserViewOptionsSheet: View { .pickerStyle(.segmented) .listRowBackground(Color.clear) .listRowInsets(EdgeInsets(top: 8, leading: 0, bottom: 8, trailing: 0)) - } header: { - Text("mediaBrowser.viewOptions.order") - } - - Section { - Toggle("mediaBrowser.viewOptions.showOnlyPlayable", isOn: $showOnlyPlayable) - } header: { - Text("mediaBrowser.viewOptions.filters") } } .navigationTitle("mediaBrowser.viewOptions.title") @@ -70,7 +59,7 @@ struct MediaBrowserViewOptionsSheet: View { } } #if os(iOS) - .presentationDetents([.medium]) + .presentationDetents([.height(280)]) .presentationDragIndicator(.visible) #endif .onAppear {