Timeline media: kind of align with other messenger apps

features/messages/impl/src/main/kotlin/io/element/android/features/messages/impl/timeline/components/event/TimelineItemAspectRatioBox.kt

@@ -25,22 +25,23 @@ import androidx.compose.ui.Alignment
 import androidx.compose.ui.Modifier
 import androidx.compose.ui.unit.dp
 
+private const val MAX_HEIGHT_IN_DP = 360f
+private const val MIN_ASPECT_RATIO = 0.6f
+private const val MAX_ASPECT_RATIO = 4f
+private const val DEFAULT_ASPECT_RATIO = 1.33f
+
 @Composable
 fun TimelineItemAspectRatioBox(
-    height: Int?,
     aspectRatio: Float?,
     modifier: Modifier = Modifier,
     contentAlignment: Alignment = Alignment.TopStart,
-    content: @Composable BoxScope.() -> Unit,
+    content: @Composable (BoxScope.() -> Unit),
 ) {
-    val maxHeight = minOf(300, maxOf(100, height ?: Int.MAX_VALUE))
+    val safeAspectRatio = (aspectRatio ?: DEFAULT_ASPECT_RATIO).coerceIn(MIN_ASPECT_RATIO, MAX_ASPECT_RATIO)
-    val aspectRatioModifier = aspectRatio?.let {
-        Modifier.aspectRatio(it)
-    } ?: Modifier
     Box(
         modifier = modifier
-            .heightIn(max = maxHeight.dp)
+            .heightIn(max = MAX_HEIGHT_IN_DP.dp)
-            .then(aspectRatioModifier),
+            .aspectRatio(safeAspectRatio, true),
         contentAlignment = contentAlignment,
         content = content
     )

features/messages/impl/src/main/kotlin/io/element/android/features/messages/impl/timeline/components/event/TimelineItemImageView.kt

@@ -34,14 +34,13 @@ fun TimelineItemImageView(
     modifier: Modifier = Modifier,
 ) {
     TimelineItemAspectRatioBox(
-        height = content.height,
         aspectRatio = content.aspectRatio,
         modifier = modifier
     ) {
         BlurHashAsyncImage(
             model = MediaRequestData(content.preferredMediaSource, MediaRequestData.Kind.File(content.body, content.mimeType)),
             blurHash = content.blurhash,
-            contentScale = ContentScale.Fit,
+            contentScale = ContentScale.Crop,
         )
     }
 }

features/messages/impl/src/main/kotlin/io/element/android/features/messages/impl/timeline/components/event/TimelineItemVideoView.kt

@@ -43,7 +43,6 @@ fun TimelineItemVideoView(
     modifier: Modifier = Modifier,
 ) {
     TimelineItemAspectRatioBox(
-        height = content.height,
         aspectRatio = content.aspectRatio,
         modifier = modifier,
         contentAlignment = Alignment.Center,
@@ -51,8 +50,7 @@ fun TimelineItemVideoView(
         BlurHashAsyncImage(
             model = MediaRequestData(content.thumbnailSource, MediaRequestData.Kind.File(content.body, content.mimeType)),
             blurHash = content.blurHash,
-            modifier = Modifier.fillMaxSize(),
+            contentScale = ContentScale.Crop,
-            contentScale = ContentScale.Fit,
         )
         Box(
             modifier = Modifier.roundedBackground(),