Skip to content

Commit

Permalink
live tv coordinator fix
Browse files Browse the repository at this point in the history
  • Loading branch information
jhays committed Aug 16, 2023
1 parent e2d9f05 commit c03b66f
Show file tree
Hide file tree
Showing 3 changed files with 15 additions and 39 deletions.
16 changes: 2 additions & 14 deletions Shared/Coordinators/LiveTVChannelsCoordinator.swift
Original file line number Diff line number Diff line change
Expand Up @@ -25,20 +25,8 @@ final class LiveTVChannelsCoordinator: NavigationCoordinatable {
#endif

#if os(tvOS)
func makeVideoPlayer(manager: VideoPlayerManager) -> NavigationViewCoordinator<BasicNavigationViewCoordinator> {
BasicNavigationViewCoordinator {
Group {
if Defaults[.VideoPlayer.videoPlayerType] == .swiftfin {
VideoPlayer(manager: manager)
.overlay {
VideoPlayer.Overlay()
}
} else {
NativeVideoPlayer(manager: manager)
}
}
}
.inNavigationViewCoordinator()
func makeVideoPlayer(manager: VideoPlayerManager) -> NavigationViewCoordinator<VideoPlayerCoordinator> {
NavigationViewCoordinator(VideoPlayerCoordinator(manager: manager))
}
#endif

Expand Down
16 changes: 2 additions & 14 deletions Shared/Coordinators/LiveTVProgramsCoordinator.swift
Original file line number Diff line number Diff line change
Expand Up @@ -26,20 +26,8 @@ final class LiveTVProgramsCoordinator: NavigationCoordinatable {
#endif

#if os(tvOS)
func makeVideoPlayer(manager: VideoPlayerManager) -> NavigationViewCoordinator<BasicNavigationViewCoordinator> {
BasicNavigationViewCoordinator {
Group {
if Defaults[.VideoPlayer.videoPlayerType] == .swiftfin {
VideoPlayer(manager: manager)
.overlay {
VideoPlayer.Overlay()
}
} else {
NativeVideoPlayer(manager: manager)
}
}
}
.inNavigationViewCoordinator()
func makeVideoPlayer(manager: VideoPlayerManager) -> NavigationViewCoordinator<VideoPlayerCoordinator> {
NavigationViewCoordinator(VideoPlayerCoordinator(manager: manager))
}
#endif

Expand Down
22 changes: 11 additions & 11 deletions Swiftfin tvOS/Views/VideoPlayer/Overlays/Overlay.swift
Original file line number Diff line number Diff line change
Expand Up @@ -33,17 +33,17 @@ extension VideoPlayer {
var body: some View {
ZStack {

// MainOverlay()
// .visible(currentOverlayType == .main)
//
// ConfirmCloseOverlay()
// .visible(currentOverlayType == .confirmClose)
//
// SmallMenuOverlay()
// .visible(currentOverlayType == .smallMenu)
//
// ChapterOverlay()
// .visible(currentOverlayType == .chapters)
MainOverlay()
.visible(currentOverlayType == .main)

ConfirmCloseOverlay()
.visible(currentOverlayType == .confirmClose)

SmallMenuOverlay()
.visible(currentOverlayType == .smallMenu)

ChapterOverlay()
.visible(currentOverlayType == .chapters)
}
.visible(isPresentingOverlay)
.animation(.linear(duration: 0.1), value: currentOverlayType)
Expand Down

0 comments on commit c03b66f

Please sign in to comment.