diff --git a/app/src/main/java/com/github/libretube/fragments/PlayerFragment.kt b/app/src/main/java/com/github/libretube/fragments/PlayerFragment.kt index addc10160..9b77d620e 100644 --- a/app/src/main/java/com/github/libretube/fragments/PlayerFragment.kt +++ b/app/src/main/java/com/github/libretube/fragments/PlayerFragment.kt @@ -1222,9 +1222,8 @@ class PlayerFragment : Fragment() { val captionLanguage = subtitlesNamesList[index] val captionLanguageCode = subtitleCodesList[index] - // update the icon + // update the icon of the captions button playerBinding.captions.setImageResource(R.drawable.ic_caption) - playerBinding.captions.setColorFilter(Color.WHITE) // select the new caption preference trackSelector.buildUponParameters() @@ -1236,7 +1235,6 @@ class PlayerFragment : Fragment() { } else { // none selected playerBinding.captions.setImageResource(R.drawable.ic_caption_outlined) - playerBinding.captions.setColorFilter(Color.GRAY) // disable captions trackSelector.buildUponParameters() @@ -1416,8 +1414,7 @@ class PlayerFragment : Fragment() { // hide the close image button playerBinding.closeImageButton.visibility = if (isLocked && - !Globals.isFullScreen && - autoRotationEnabled + !(Globals.isFullScreen && !autoRotationEnabled) ) View.VISIBLE else View.GONE // disable double tap to seek when the player is locked @@ -1569,10 +1566,22 @@ class PlayerFragment : Fragment() { // set portrait mode unsetFullscreen() + with(binding.playerMotionLayout) { + getConstraintSet(R.id.start).constrainHeight(R.id.player, -1) + enableTransition(R.id.yt_transition, false) + } + binding.linLayout.visibility = View.GONE + Globals.isFullScreen = false } else { // enable exoPlayer controls again exoPlayerView.useController = true + + with(binding.playerMotionLayout) { + getConstraintSet(R.id.start).constrainHeight(R.id.player, 0) + enableTransition(R.id.yt_transition, true) + } + binding.linLayout.visibility = View.VISIBLE } } @@ -1581,7 +1590,7 @@ class PlayerFragment : Fragment() { binding.playerScrollView.getHitRect(bounds) if (SDK_INT >= Build.VERSION_CODES.O && - exoPlayer.isPlaying && (binding.playerScrollView.getLocalVisibleRect(bounds) || Globals.isFullScreen) + (binding.playerScrollView.getLocalVisibleRect(bounds) || Globals.isFullScreen) ) { activity?.enterPictureInPictureMode(updatePipParams()) } diff --git a/app/src/main/res/layout/exo_styled_player_control_view.xml b/app/src/main/res/layout/exo_styled_player_control_view.xml index 9e64eb5b0..43d300c00 100644 --- a/app/src/main/res/layout/exo_styled_player_control_view.xml +++ b/app/src/main/res/layout/exo_styled_player_control_view.xml @@ -32,15 +32,15 @@ @@ -99,7 +99,7 @@ android:id="@+id/captions" style="@style/PlayerControlTop" android:src="@drawable/ic_caption_outlined" - app:tint="@android:color/darker_gray" /> + app:tint="@android:color/white" /> + android:paddingStart="10dp" + android:paddingEnd="15dp" > @@ -226,7 +223,7 @@ android:background="@android:color/transparent" android:clipToPadding="false" android:gravity="center" - android:padding="@dimen/exo_styled_controls_padding"> + android:padding="20dp">