From 7c893cd9bc71a4ea127e1871a951f9d94fa0ba09 Mon Sep 17 00:00:00 2001 From: Manish Kumar Verma Date: Tue, 2 Apr 2024 13:54:08 +0530 Subject: [PATCH] simplified code --- .../github/libretube/ui/fragments/PlayerFragment.kt | 11 ++++------- 1 file changed, 4 insertions(+), 7 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 19018bdb88..b73b52e03c 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 @@ -1300,7 +1300,7 @@ class PlayerFragment : Fragment(), OnlinePlayerOptions { } } - private fun setPlayerResolution(resolution: Int, isSelectedByUser: Boolean) { + private fun setPlayerResolution(resolution: Int, isSelectedByUser: Boolean = false) { val transformedResolution = if (!isSelectedByUser && isShort) { ceil(resolution * 16.0 / 9.0).toInt() } else { @@ -1319,9 +1319,9 @@ class PlayerFragment : Fragment(), OnlinePlayerOptions { if (!this::trackSelector.isInitialized) return if (!isFullscreen && noFullscreenResolution != null) { - setPlayerResolution(noFullscreenResolution!!, false) + setPlayerResolution(noFullscreenResolution!!) } else if (fullscreenResolution != null) { - setPlayerResolution(fullscreenResolution ?: Int.MAX_VALUE, false) + setPlayerResolution(fullscreenResolution ?: Int.MAX_VALUE) } } @@ -1495,13 +1495,10 @@ class PlayerFragment : Fragment(), OnlinePlayerOptions { if (currentQuality == it.resolution) "${it.name} ✓" else it.name } ) { which -> - var newResolution = resolutions[which].resolution + val newResolution = resolutions[which].resolution setPlayerResolution(newResolution, true) // save the selected resolution to update on fullscreen change - if (isShort) { - newResolution = ceil(newResolution * 9.0 / 16.0).toInt() - } if (noFullscreenResolution != null && viewModel.isFullscreen.value != true) { noFullscreenResolution = newResolution } else {