From f81b2b677f03b00ba0182dc443995ff0aa505901 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Daniel=20D=E2=80=99Aquino?= Date: Fri, 18 Oct 2024 19:04:32 -0700 Subject: [PATCH] Fix portrait video sizing on full screen carousel MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Changelog-Fixed: Fixed portrait video size on full screen carousel Signed-off-by: Daniel D’Aquino --- damus/Views/Images/FullScreenCarouselView.swift | 4 ---- 1 file changed, 4 deletions(-) diff --git a/damus/Views/Images/FullScreenCarouselView.swift b/damus/Views/Images/FullScreenCarouselView.swift index 12a1898b..cdfe8961 100644 --- a/damus/Views/Images/FullScreenCarouselView.swift +++ b/damus/Views/Images/FullScreenCarouselView.swift @@ -60,10 +60,6 @@ struct FullScreenCarouselView: View { VStack { if case .video = urls[safe: index] { ImageContainerView(video_coordinator: video_coordinator, url: urls[index], settings: settings, imageDict: $imageDict) - .clipped() // SwiftUI hack from https://stackoverflow.com/a/74401288 to make playback controls show up within the TabView - .aspectRatio(contentMode: .fit) - .padding(.top, Theme.safeAreaInsets?.top) - .padding(.bottom, Theme.safeAreaInsets?.bottom) .modifier(SwipeToDismissModifier(minDistance: 50, onDismiss: { presentationMode.wrappedValue.dismiss() }))