From 784dbd3edd768cffb4e87765208e1cccc34ba52d Mon Sep 17 00:00:00 2001
From: Bnyro <bnyro@tutanota.com>
Date: Mon, 24 Jul 2023 13:36:04 +0200
Subject: [PATCH] fix: fullscreen orientation after shorts video

---
 .../libretube/ui/fragments/PlayerFragment.kt  | 25 +++++++++++++------
 1 file changed, 17 insertions(+), 8 deletions(-)

diff --git a/app/src/main/java/com/github/libretube/ui/fragments/PlayerFragment.kt b/app/src/main/java/com/github/libretube/ui/fragments/PlayerFragment.kt
index 4adb3ab53..9d253fb6e 100644
--- a/app/src/main/java/com/github/libretube/ui/fragments/PlayerFragment.kt
+++ b/app/src/main/java/com/github/libretube/ui/fragments/PlayerFragment.kt
@@ -510,6 +510,21 @@ class PlayerFragment : Fragment(), OnlinePlayerOptions {
         NavigationHelper.startAudioPlayer(requireContext())
     }
 
+    /**
+     * If enabled, determine the orientation o use based on the video's aspect ratio
+     * Expected behavior: Portrait for shorts, Landscape for normal videos
+     */
+    private fun updateFullscreenOrientation() {
+        if (!PlayerHelper.autoRotationEnabled) {
+            val height = streams.videoStreams.firstOrNull()?.height ?: exoPlayer.videoSize.height
+            val width = streams.videoStreams.firstOrNull()?.width ?: exoPlayer.videoSize.width
+
+            // different orientations of the video are only available when autorotation is disabled
+            val orientation = PlayerHelper.getOrientation(width, height)
+            mainActivity.requestedOrientation = orientation
+        }
+    }
+
     private fun setFullscreen() {
         with(binding.playerMotionLayout) {
             getConstraintSet(R.id.start).constrainHeight(R.id.player, -1)
@@ -525,14 +540,7 @@ class PlayerFragment : Fragment(), OnlinePlayerOptions {
         playerBinding.fullscreen.setImageResource(R.drawable.ic_fullscreen_exit)
         playerBinding.exoTitle.visibility = View.VISIBLE
 
-        if (!PlayerHelper.autoRotationEnabled) {
-            val height = streams.videoStreams.firstOrNull()?.height ?: exoPlayer.videoSize.height
-            val width = streams.videoStreams.firstOrNull()?.width ?: exoPlayer.videoSize.width
-
-            // different orientations of the video are only available when autorotation is disabled
-            val orientation = PlayerHelper.getOrientation(width, height)
-            mainActivity.requestedOrientation = orientation
-        }
+        updateFullscreenOrientation()
 
         viewModel.isFullscreen.value = true
     }
@@ -746,6 +754,7 @@ class PlayerFragment : Fragment(), OnlinePlayerOptions {
                 prepareExoPlayerView()
                 initializePlayerView()
                 setupSeekbarPreview()
+                updateFullscreenOrientation()
 
                 exoPlayer.prepare()
                 if (PreferenceHelper.getBoolean(PreferenceKeys.PLAY_AUTOMATICALLY, true)) {