Suchen am Fernseher — und die Leiste, die es dafür braucht
Das Backend konnte die Suche längst; es fehlte die Oberfläche. **`.searchable`, keine eigene Tastatur.** tvOS macht daraus einen eigenen Bildschirm mit Rastertastatur oben und Ergebnissen darunter — die Bedienung, die jeder von Apples Apps kennt. Etwas Eigenes zu bauen hiesse, sie nachzuahmen und dabei die Diktierfunktion zu verlieren. **Herausgelöst statt kopiert**, wie schon bei Start und Mediathek: `SearchFeed` trägt das Entprellen, die Mindestlänge und die zuletzt gesuchten Begriffe. Drei Stellen, an denen es nicht offensichtlich ist — und die dritte ist die wichtigste: geprüft wird auf `Task.isCancelled` und nicht auf `CancellationError`, weil eine abgebrochene Netzanfrage als Übertragungsfehler zurückkommt. Die naheliegende Prüfung lief ins Leere, und jeder Tastendruck leerte kurz die Liste. **Und erst jetzt eine Reiterleiste.** Bis hierher gab es genau einen Bildschirm, und eine Leiste mit einem Eintrag wäre Zierrat gewesen. Der Suchreiter trägt `role: .search`: tvOS stellt ihn dadurch ans Ende und gibt ihm die Lupe, dieselbe Stelle wie überall sonst. Keinen dritten Reiter für die Mediatheken — die stehen als Reihe auf der Startseite, und ein Reiter verspräche einen eigenen Bereich, hinter dem dasselbe Raster läge. Der offene Reiter wird ausdrücklich gehalten, nicht dem System überlassen: ein Link aus dem Regal über der App führt zur Startseite, und stünde gerade die Suche offen, passierte sichtbar nichts. Nebenbei bekommt die iOS-Suche einen Knopf zum Vergessen des Verlaufs. Wischen zum Löschen gab es dort, am Fernseher gibt es das nicht — und eine Liste, die sich nur auf einer Plattform leeren lässt, ist keine. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
co-authored by
Claude Opus 5
parent
5ee7bb4e6c
commit
6373566d1c
@@ -3,22 +3,21 @@ import FlixyDesign
|
||||
import MediaKit
|
||||
|
||||
/// Sucht über alle Mediatheken des Servers.
|
||||
///
|
||||
/// **Nur noch Darstellung.** Entprellen, fragen und merken liegen in
|
||||
/// `SearchFeed`, weil der Fernseher dieselben Fragen an denselben Server
|
||||
/// stellt. Siehe dort, warum das kein Beiwerk ist.
|
||||
struct SearchView: View {
|
||||
@Environment(FlixySession.self) private var session
|
||||
|
||||
@State private var text = ""
|
||||
@State private var hubs: [MediaRowGroup] = []
|
||||
@State private var isSearching = false
|
||||
@State private var recent: [String] = UserDefaults.standard.stringArray(forKey: recentKey) ?? []
|
||||
|
||||
private static let recentKey = "flixy.search.recent"
|
||||
private static let recentLimit = 8
|
||||
@State private var feed = SearchFeed()
|
||||
|
||||
var body: some View {
|
||||
Group {
|
||||
if text.isEmpty {
|
||||
recentSearches
|
||||
} else if hubs.isEmpty && !isSearching {
|
||||
} else if feed.hubs.isEmpty && !feed.isSearching {
|
||||
ContentUnavailableView.search(text: text)
|
||||
} else {
|
||||
results
|
||||
@@ -31,13 +30,13 @@ struct SearchView: View {
|
||||
// Jeder Tastendruck löst eine Anfrage aus — aber erst nach einer
|
||||
// kurzen Pause. Ohne sie schickt „Blade Runner" zwölf Anfragen los,
|
||||
// von denen elf verworfen werden.
|
||||
.task(id: text) { await search() }
|
||||
.task(id: text) { await feed.search(text, from: session.backend) }
|
||||
}
|
||||
|
||||
private var results: some View {
|
||||
ScrollView {
|
||||
LazyVStack(alignment: .leading, spacing: FlixySpacing.section) {
|
||||
ForEach(hubs) { hub in
|
||||
ForEach(feed.hubs) { hub in
|
||||
MediaRow(row: hub)
|
||||
}
|
||||
}
|
||||
@@ -48,7 +47,7 @@ struct SearchView: View {
|
||||
|
||||
@ViewBuilder
|
||||
private var recentSearches: some View {
|
||||
if recent.isEmpty {
|
||||
if feed.recent.isEmpty {
|
||||
ContentUnavailableView {
|
||||
Label("search.title", systemImage: "magnifyingglass")
|
||||
} description: {
|
||||
@@ -57,7 +56,7 @@ struct SearchView: View {
|
||||
} else {
|
||||
List {
|
||||
Section {
|
||||
ForEach(recent, id: \.self) { entry in
|
||||
ForEach(feed.recent, id: \.self) { entry in
|
||||
Button {
|
||||
text = entry
|
||||
} label: {
|
||||
@@ -65,12 +64,13 @@ struct SearchView: View {
|
||||
.foregroundStyle(Color.flixyTextSecondary)
|
||||
}
|
||||
}
|
||||
.onDelete { offsets in
|
||||
recent.remove(atOffsets: offsets)
|
||||
persistRecent()
|
||||
}
|
||||
} header: {
|
||||
Text("search.recent")
|
||||
} footer: {
|
||||
Button("search.forgetRecent", role: .destructive) {
|
||||
feed.forgetRecent()
|
||||
}
|
||||
.font(.flixyCaption)
|
||||
}
|
||||
.listRowBackground(Color.flixySurface)
|
||||
}
|
||||
@@ -78,47 +78,4 @@ struct SearchView: View {
|
||||
.flixyRevealsTopChrome()
|
||||
}
|
||||
}
|
||||
|
||||
private func search() async {
|
||||
let trimmed = text.trimmingCharacters(in: .whitespacesAndNewlines)
|
||||
guard let backend = session.backend, trimmed.count >= 2 else {
|
||||
hubs = []
|
||||
return
|
||||
}
|
||||
|
||||
do {
|
||||
try await Task.sleep(for: .milliseconds(300))
|
||||
} catch {
|
||||
// Ein neuer Tastendruck hat diese Aufgabe abgebrochen — genau der
|
||||
// Zweck der Pause.
|
||||
return
|
||||
}
|
||||
|
||||
isSearching = true
|
||||
defer { isSearching = false }
|
||||
|
||||
do {
|
||||
hubs = try await backend.search(trimmed, limit: 30)
|
||||
if !hubs.isEmpty { remember(trimmed) }
|
||||
} catch {
|
||||
// Abgebrochen, weil weitergetippt oder der Bildschirm verlassen
|
||||
// wurde: `Task.isCancelled` statt `catch is CancellationError` —
|
||||
// eine abgebrochene Netzanfrage kommt als Übertragungsfehler
|
||||
// zurück, nicht als Abbruch, und die alte Prüfung lief deshalb
|
||||
// ins Leere.
|
||||
if Task.isCancelled { return }
|
||||
hubs = []
|
||||
}
|
||||
}
|
||||
|
||||
private func remember(_ entry: String) {
|
||||
recent.removeAll { $0.caseInsensitiveCompare(entry) == .orderedSame }
|
||||
recent.insert(entry, at: 0)
|
||||
recent = Array(recent.prefix(Self.recentLimit))
|
||||
persistRecent()
|
||||
}
|
||||
|
||||
private func persistRecent() {
|
||||
UserDefaults.standard.set(recent, forKey: Self.recentKey)
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user