Minor tvOS UI improvements

This commit is contained in:
Arkadiusz Fal 2021-09-29 14:36:52 +02:00
parent 282170209b
commit 1b1be1aefe
7 changed files with 35 additions and 13 deletions

View File

@ -5,13 +5,22 @@ struct FixtureEnvironmentObjectsModifier: ViewModifier {
func body(content: Content) -> some View {
content
.environmentObject(InstancesModel())
.environmentObject(InvidiousAPI())
.environmentObject(api)
.environmentObject(NavigationModel())
.environmentObject(PlaybackModel())
.environmentObject(PlaylistsModel())
.environmentObject(RecentsModel())
.environmentObject(SearchModel())
.environmentObject(SubscriptionsModel(api: InvidiousAPI()))
.environmentObject(SubscriptionsModel(api: api))
}
private var api: InvidiousAPI {
let api = InvidiousAPI()
api.validInstance = true
api.signedIn = true
return api
}
}

View File

@ -154,6 +154,8 @@ struct AddToPlaylistView: View {
model.selectedPlaylistID = playlist.id
}
}
Button("Cancel", role: .cancel) {}
}
}

View File

@ -103,10 +103,10 @@ struct PlaylistFormView: View {
Spacer()
Button("Cancel") {
dismiss()
}
#if !os(tvOS)
Button("Cancel") {
dismiss()
}
.keyboardShortcut(.cancelAction)
#endif
}
@ -206,6 +206,10 @@ struct PlaylistFormView: View {
self.visibility = visibility
}
}
#if os(tvOS)
Button("Cancel", role: .cancel) {}
#endif
}
#endif
}

View File

@ -17,6 +17,8 @@ struct PlaylistsView: View {
@State private var showingAddToPlaylist = false
@State private var videoIDToAddToPlaylist = ""
@Namespace private var focusNamespace
var videos: [Video] {
model.currentPlaylist?.videos ?? []
}
@ -63,6 +65,7 @@ struct PlaylistsView: View {
#if !os(iOS)
if !model.isEmpty {
selectPlaylistButton
.prefersDefaultFocus(in: focusNamespace)
}
if model.currentPlaylist != nil {
@ -95,6 +98,7 @@ struct PlaylistsView: View {
}
#endif
}
.focusScope(focusNamespace)
.onAppear {
model.load()
}
@ -112,18 +116,14 @@ struct PlaylistsView: View {
selectPlaylistButton
}
#if os(iOS)
Spacer()
#endif
if model.currentPlaylist != nil {
editPlaylistButton
}
#if !os(iOS)
newPlaylistButton
.padding(.leading, 40)
#endif
Spacer()
newPlaylistButton
.padding(.leading, 40)
}
}
@ -182,6 +182,8 @@ struct PlaylistsView: View {
model.selectPlaylist(playlist.id)
}
}
Button("Cancel", role: .cancel) {}
}
#else
Menu(model.currentPlaylist?.title ?? "Select playlist") {

View File

@ -48,6 +48,7 @@ struct AccountsSettingsView: View {
.contextMenu {
Button("Toggle Default") { toggleDefault(account) }
Button("Remove", role: .destructive) { removeAccount(account) }
Button("Cancel", role: .cancel) {}
}
#endif
}

View File

@ -124,6 +124,8 @@ struct TrendingView: View {
ForEach(TrendingCategory.allCases) { category in
Button(category.name) { self.category = category }
}
Button("Cancel", role: .cancel) {}
}
#else

View File

@ -29,6 +29,8 @@ struct AccountSelectionView: View {
}
}
}
Button("Cancel", role: .cancel) {}
}
}
}