Merge pull request #6757 from Bnyro/master

fix: video aspect ratio ignored when playing videos offline
libre-tube · Nov 18, 2024 · 9acb486 · 9acb486
2 parents 996586c + 889c253
commit 9acb486
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/app/src/main/java/com/github/libretube/ui/activities/OfflinePlayerActivity.kt b/app/src/main/java/com/github/libretube/ui/activities/OfflinePlayerActivity.kt
@@ -71,18 +71,16 @@ class OfflinePlayerActivity : BaseActivity() {
             playerBinding.duration.text = DateUtils.formatElapsedTime(
                 player.duration / 1000
             )
-
-            if (events.contains(Player.EVENT_TRACKS_CHANGED)) {
-                requestedOrientation = PlayerHelper.getOrientation(
-                    playerController.videoSize.width,
-                    playerController.videoSize.height
-                )
-            }
         }
 
         override fun onIsPlayingChanged(isPlaying: Boolean) {
             super.onIsPlayingChanged(isPlaying)
 
+            requestedOrientation = PlayerHelper.getOrientation(
+                playerController.videoSize.width,
+                playerController.videoSize.height
+            )
+
             if (PlayerHelper.pipEnabled) {
                 PictureInPictureCompat.setPictureInPictureParams(
                     this@OfflinePlayerActivity,

diff --git a/app/src/main/java/com/github/libretube/ui/views/CustomExoPlayerView.kt b/app/src/main/java/com/github/libretube/ui/views/CustomExoPlayerView.kt
@@ -260,6 +260,8 @@ abstract class CustomExoPlayerView(
         binding.queueToggle.setOnClickListener {
             PlayingQueueSheet().show(supportFragmentManager, null)
         }
+
+        updateMarginsByFullscreenMode()
     }
 
     /**