From d37ec23fa9483de112f101bb79462acbf0b70215 Mon Sep 17 00:00:00 2001
From: Benoit Marty <benoit@matrix.org>
Date: Fri, 29 Nov 2024 11:49:21 +0100
Subject: [PATCH] Fix preview rendering

---
 .../libraries/mediaviewer/impl/local/file/MediaFileView.kt     | 2 ++
 .../libraries/mediaviewer/impl/local/image/MediaImageView.kt   | 2 ++
 .../libraries/mediaviewer/impl/local/video/MediaVideoView.kt   | 3 +--
 3 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/libraries/mediaviewer/impl/src/main/kotlin/io/element/android/libraries/mediaviewer/impl/local/file/MediaFileView.kt b/libraries/mediaviewer/impl/src/main/kotlin/io/element/android/libraries/mediaviewer/impl/local/file/MediaFileView.kt
index 0e345138a0..ec50ef9f4c 100644
--- a/libraries/mediaviewer/impl/src/main/kotlin/io/element/android/libraries/mediaviewer/impl/local/file/MediaFileView.kt
+++ b/libraries/mediaviewer/impl/src/main/kotlin/io/element/android/libraries/mediaviewer/impl/local/file/MediaFileView.kt
@@ -14,6 +14,7 @@ import androidx.compose.foundation.interaction.MutableInteractionSource
 import androidx.compose.foundation.layout.Box
 import androidx.compose.foundation.layout.Column
 import androidx.compose.foundation.layout.Spacer
+import androidx.compose.foundation.layout.fillMaxSize
 import androidx.compose.foundation.layout.height
 import androidx.compose.foundation.layout.padding
 import androidx.compose.foundation.layout.size
@@ -112,6 +113,7 @@ internal fun MediaFileViewPreview(
     @PreviewParameter(MediaInfoFileProvider::class) info: MediaInfo
 ) = ElementPreview {
     MediaFileView(
+        modifier = Modifier.fillMaxSize(),
         localMediaViewState = rememberLocalMediaViewState(),
         uri = null,
         info = info,
diff --git a/libraries/mediaviewer/impl/src/main/kotlin/io/element/android/libraries/mediaviewer/impl/local/image/MediaImageView.kt b/libraries/mediaviewer/impl/src/main/kotlin/io/element/android/libraries/mediaviewer/impl/local/image/MediaImageView.kt
index 7057bd8b74..a286b7dc2d 100644
--- a/libraries/mediaviewer/impl/src/main/kotlin/io/element/android/libraries/mediaviewer/impl/local/image/MediaImageView.kt
+++ b/libraries/mediaviewer/impl/src/main/kotlin/io/element/android/libraries/mediaviewer/impl/local/image/MediaImageView.kt
@@ -8,6 +8,7 @@
 package io.element.android.libraries.mediaviewer.impl.local.image
 
 import androidx.compose.foundation.Image
+import androidx.compose.foundation.layout.fillMaxSize
 import androidx.compose.runtime.Composable
 import androidx.compose.ui.Modifier
 import androidx.compose.ui.layout.ContentScale
@@ -55,6 +56,7 @@ fun MediaImageView(
 @Composable
 internal fun MediaImageViewPreview() = ElementPreview {
     MediaImageView(
+        modifier = Modifier.fillMaxSize(),
         localMediaViewState = rememberLocalMediaViewState(),
         localMedia = null,
         onClick = {},
diff --git a/libraries/mediaviewer/impl/src/main/kotlin/io/element/android/libraries/mediaviewer/impl/local/video/MediaVideoView.kt b/libraries/mediaviewer/impl/src/main/kotlin/io/element/android/libraries/mediaviewer/impl/local/video/MediaVideoView.kt
index 2aef883752..d978549192 100644
--- a/libraries/mediaviewer/impl/src/main/kotlin/io/element/android/libraries/mediaviewer/impl/local/video/MediaVideoView.kt
+++ b/libraries/mediaviewer/impl/src/main/kotlin/io/element/android/libraries/mediaviewer/impl/local/video/MediaVideoView.kt
@@ -14,7 +14,6 @@ import androidx.compose.foundation.background
 import androidx.compose.foundation.layout.Box
 import androidx.compose.foundation.layout.fillMaxSize
 import androidx.compose.foundation.layout.fillMaxWidth
-import androidx.compose.foundation.layout.wrapContentSize
 import androidx.compose.runtime.Composable
 import androidx.compose.runtime.LaunchedEffect
 import androidx.compose.runtime.derivedStateOf
@@ -179,7 +178,7 @@ private fun ExoPlayerMediaVideoView(
             Text(
                 modifier = Modifier
                     .background(ElementTheme.colors.bgSubtlePrimary)
-                    .wrapContentSize(),
+                    .align(Alignment.Center),
                 text = "A Video Player will render here",
             )
         } else {