From b2477eb41294d358b2c39d13b05bd0e923d90676 Mon Sep 17 00:00:00 2001 From: Jorge Martin Espinosa Date: Tue, 22 Jul 2025 16:06:49 +0200 Subject: [PATCH] Remove the scaling added in `Player.Listener.onVideoSizeChanged` (#5055) --- .../impl/local/video/MediaVideoView.kt | 15 --------------- 1 file changed, 15 deletions(-) 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 8e619fbdc9..8e826383b6 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 @@ -25,7 +25,6 @@ import androidx.compose.runtime.remember import androidx.compose.runtime.setValue import androidx.compose.ui.Alignment import androidx.compose.ui.Modifier -import androidx.compose.ui.geometry.Size import androidx.compose.ui.platform.LocalContext import androidx.compose.ui.platform.LocalInspectionMode import androidx.compose.ui.viewinterop.AndroidView @@ -34,7 +33,6 @@ import androidx.media3.common.MediaItem import androidx.media3.common.Player import androidx.media3.common.Player.STATE_READY import androidx.media3.common.Timeline -import androidx.media3.common.VideoSize import androidx.media3.exoplayer.ExoPlayer import androidx.media3.ui.AspectRatioFrameLayout import androidx.media3.ui.PlayerView @@ -56,7 +54,6 @@ import io.element.android.libraries.mediaviewer.impl.local.player.seekToEnsurePl import io.element.android.libraries.mediaviewer.impl.local.player.togglePlay import io.element.android.libraries.mediaviewer.impl.local.rememberLocalMediaViewState import kotlinx.coroutines.delay -import me.saket.telephoto.zoomable.ZoomableContentLocation import me.saket.telephoto.zoomable.zoomable import kotlin.time.Duration.Companion.seconds @@ -154,18 +151,6 @@ private fun ExoPlayerMediaVideoView( isReady = playbackState == STATE_READY, ) } - - override fun onVideoSizeChanged(videoSize: VideoSize) { - // Ensure that the user cannot zoom/move outside of the video bounds - localMediaViewState.zoomableState.setContentLocation( - ZoomableContentLocation.scaledInsideAndCenterAligned( - Size( - videoSize.width.toFloat(), - videoSize.height.toFloat(), - ) - ) - ) - } } }