Fixes #2253 - Fix timeline thumbnail frames for media missing sizing …

…info
element-hq · Dec 18, 2023 · fda042b · fda042b
1 parent 123164f
commit fda042b
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 4 deletions.
diff --git a/ElementX/Sources/Other/SwiftUI/Layout/TimelineMediaFrame.swift b/ElementX/Sources/Other/SwiftUI/Layout/TimelineMediaFrame.swift
@@ -27,10 +27,14 @@ extension View {
             aspectRatio(contentAspectRatio, contentMode: .fit)
                 .frame(minHeight: minMediaHeight, maxHeight: minMediaHeight)
         } else {
-            aspectRatio(contentAspectRatio, contentMode: .fit)
-                .frame(maxHeight: min(maxMediaHeight, max(minMediaHeight, contentHeight ?? .infinity)))
-                // Required to prevent the reply details to get higher priority in rendering the width of the view.
-                .aspectRatio(contentAspectRatio, contentMode: .fit)
+            if let contentAspectRatio {
+                aspectRatio(contentAspectRatio, contentMode: .fit)
+                    .frame(maxHeight: min(maxMediaHeight, max(minMediaHeight, contentHeight ?? .infinity)))
+                    // Required to prevent the reply details to get higher priority in rendering the width of the view.
+                    .aspectRatio(contentAspectRatio, contentMode: .fit)
+            } else { // Otherwise let the image load and use its native aspect ratio with a max height
+                frame(maxHeight: maxMediaHeight)
+            }
         }
     }
 }
diff --git a/changelog.d/2253.bugfix b/changelog.d/2253.bugfix
@@ -0,0 +1 @@
+Fix timeline thumbnail frames for media missing sizing info
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		Fix timeline thumbnail frames for media missing sizing info