From 9d7f93303ff44523a7354c18ac6ec460ac5cc143 Mon Sep 17 00:00:00 2001
From: Thomas Ricouard <ricouard77@gmail.com>
Date: Tue, 20 Dec 2022 08:14:57 +0100
Subject: [PATCH] Better Media viewer

---
 .../Sources/Status/Row/StatusCardView.swift   |  2 +
 .../Status/Row/StatusMediaPreviewView.swift   | 60 +++++++++++--------
 2 files changed, 36 insertions(+), 26 deletions(-)

diff --git a/Packages/Status/Sources/Status/Row/StatusCardView.swift b/Packages/Status/Sources/Status/Row/StatusCardView.swift
index b732ea62..cdd71587 100644
--- a/Packages/Status/Sources/Status/Row/StatusCardView.swift
+++ b/Packages/Status/Sources/Status/Row/StatusCardView.swift
@@ -14,6 +14,8 @@ struct StatusCardView: View {
             content: { image in
               image.resizable()
                 .aspectRatio(contentMode: .fill)
+                .frame(maxHeight: 200)
+                .clipped()
             },
             placeholder: {
               ProgressView()
diff --git a/Packages/Status/Sources/Status/Row/StatusMediaPreviewView.swift b/Packages/Status/Sources/Status/Row/StatusMediaPreviewView.swift
index 3790b227..2dd5f2bb 100644
--- a/Packages/Status/Sources/Status/Row/StatusMediaPreviewView.swift
+++ b/Packages/Status/Sources/Status/Row/StatusMediaPreviewView.swift
@@ -39,10 +39,10 @@ public struct StatusMediaPreviewView: View {
   @ViewBuilder
   private func makePreview(attachement: MediaAttachement) -> some View {
     if let type = attachement.supportedType {
-      switch type {
-      case .image:
-        Group {
-          GeometryReader { proxy in
+      Group {
+        GeometryReader { proxy in
+          switch type {
+          case .image:
             AsyncImage(
               url: attachement.url,
               content: { image in
@@ -57,16 +57,17 @@ public struct StatusMediaPreviewView: View {
                   .frame(maxWidth: 80, maxHeight: 80)
               }
             )
+          case .gifv:
+            VideoPlayer(player: AVPlayer(url: attachement.url))
+              .frame(width: proxy.frame(in: .local).width)
+              .frame(height: attachements.count > 2 ? 100 : 200)
           }
-          .frame(height: attachements.count > 2 ? 100 : 200)
         }
-        .cornerRadius(4)
-        .onTapGesture {
-          selectedMediaSheetManager.selectedAttachement = attachement
-        }
-      case .gifv:
-        VideoPlayer(player: AVPlayer(url: attachement.url))
-          .frame(maxHeight: attachements.count > 2 ? 100 : 200)
+        .frame(height: attachements.count > 2 ? 100 : 200)
+      }
+      .cornerRadius(4)
+      .onTapGesture {
+        selectedMediaSheetManager.selectedAttachement = attachement
       }
     }
   }
@@ -78,21 +79,28 @@ public struct StatusMediaPreviewView: View {
     attachements.insert(selectedAttachement, at: 0)
     return TabView {
       ForEach(attachements) { attachement in
-        VStack {
-          Spacer()
-          AsyncImage(
-            url: attachement.url,
-            content: { image in
-              image
-                .resizable()
-                .aspectRatio(contentMode: .fit)
-            },
-            placeholder: {
-              ProgressView()
-                .frame(maxWidth: 80, maxHeight: 80)
+        if let type = attachement.supportedType {
+          VStack {
+            Spacer()
+            switch type {
+            case .image:
+              AsyncImage(
+                url: attachement.url,
+                content: { image in
+                  image
+                    .resizable()
+                    .aspectRatio(contentMode: .fit)
+                },
+                placeholder: {
+                  ProgressView()
+                    .frame(maxWidth: 80, maxHeight: 80)
+                }
+              )
+            case .gifv:
+              VideoPlayer(player: AVPlayer(url: attachement.url))
             }
-          )
-          Spacer()
+            Spacer()
+          }
         }
       }
     }