From 3262eb8365aa939a03d6c0e7a68d02242f33da94 Mon Sep 17 00:00:00 2001 From: Bnyro Date: Tue, 19 Nov 2024 09:40:49 +0100 Subject: [PATCH] fix: recent player UI regressions --- .../libretube/ui/fragments/PlayerFragment.kt | 29 +++++++++---------- 1 file changed, 13 insertions(+), 16 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 f42f1ed19..f22ba88c4 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 @@ -400,7 +400,6 @@ class PlayerFragment : Fragment(), OnlinePlayerOptions { initializeTransitionLayout() initializeOnClickActions() - initializePlayerView() if (PlayerHelper.autoFullscreenEnabled && resources.configuration.orientation == Configuration.ORIENTATION_LANDSCAPE) { setFullscreen() @@ -464,21 +463,6 @@ class PlayerFragment : Fragment(), OnlinePlayerOptions { } } - private fun initializePlayerView() { - // initialize the player view actions - binding.player.initialize( - doubleTapOverlayBinding, - playerGestureControlsViewBinding, - chaptersViewModel - ) - binding.player.initPlayerOptions( - viewModel, - commonPlayerViewModel, - viewLifecycleOwner, - this - ) - } - @SuppressLint("ClickableViewAccessibility") private fun initializeTransitionLayout() { mainActivity.binding.container.isVisible = true @@ -982,6 +966,19 @@ class PlayerFragment : Fragment(), OnlinePlayerOptions { player = playerController } + // initialize the player view actions + binding.player.initialize( + doubleTapOverlayBinding, + playerGestureControlsViewBinding, + chaptersViewModel + ) + binding.player.initPlayerOptions( + viewModel, + commonPlayerViewModel, + viewLifecycleOwner, + this + ) + updatePlayerView() if (binding.playerMotionLayout.progress != 1.0f) {