From f84d9b4a178ae7d47a1f727c5b72172fe567326e Mon Sep 17 00:00:00 2001
From: Bnyro <bnyro@tutanota.com>
Date: Wed, 16 Aug 2023 18:46:08 +0200
Subject: [PATCH] feat(player): show if quality is auto or limited

---
 .../libretube/ui/views/OnlinePlayerView.kt    | 35 ++++++++++++++-----
 app/src/main/res/values/strings.xml           |  1 +
 2 files changed, 27 insertions(+), 9 deletions(-)

diff --git a/app/src/main/java/com/github/libretube/ui/views/OnlinePlayerView.kt b/app/src/main/java/com/github/libretube/ui/views/OnlinePlayerView.kt
index d12492baa..7f44b235a 100644
--- a/app/src/main/java/com/github/libretube/ui/views/OnlinePlayerView.kt
+++ b/app/src/main/java/com/github/libretube/ui/views/OnlinePlayerView.kt
@@ -32,27 +32,21 @@ class OnlinePlayerView(
                 BottomSheetItem(
                     context.getString(R.string.quality),
                     R.drawable.ic_hd,
-                    { "${player?.videoSize?.height}p" }
+                    this::getCurrentResolutionSummary
                 ) {
                     playerOptions?.onQualityClicked()
                 },
                 BottomSheetItem(
                     context.getString(R.string.audio_track),
                     R.drawable.ic_audio,
-                    { getCurrentAudioTrackTitle() }
+                    this::getCurrentAudioTrackTitle
                 ) {
                     playerOptions?.onAudioStreamClicked()
                 },
                 BottomSheetItem(
                     context.getString(R.string.captions),
                     R.drawable.ic_caption,
-                    {
-                        if (trackSelector != null && trackSelector!!.parameters.preferredTextLanguages.isNotEmpty()) {
-                            trackSelector!!.parameters.preferredTextLanguages[0]
-                        } else {
-                            context.getString(R.string.none)
-                        }
-                    }
+                    this::getCurrentCaptionLanguage
                 ) {
                     playerOptions?.onCaptionsClicked()
                 },
@@ -65,6 +59,29 @@ class OnlinePlayerView(
             )
     }
 
+    @androidx.annotation.OptIn(androidx.media3.common.util.UnstableApi::class)
+    private fun getCurrentResolutionSummary(): String {
+        val currentQuality = player?.videoSize?.height ?: 0
+        var summary = "${currentQuality}p"
+        val trackSelector = trackSelector ?: return summary
+        val selectedQuality = trackSelector.parameters.maxVideoHeight
+        if (selectedQuality == Int.MAX_VALUE) {
+            summary += " - ${context.getString(R.string.auto)}"
+        } else if (selectedQuality > currentQuality) {
+            summary += " - ${context.getString(R.string.resolution_limited)}"
+        }
+        return summary
+    }
+
+    @androidx.annotation.OptIn(androidx.media3.common.util.UnstableApi::class)
+    private fun getCurrentCaptionLanguage(): String {
+        return if (trackSelector != null && trackSelector!!.parameters.preferredTextLanguages.isNotEmpty()) {
+            trackSelector!!.parameters.preferredTextLanguages[0]
+        } else {
+            context.getString(R.string.none)
+        }
+    }
+
     private fun getCurrentAudioTrackTitle(): String {
         if (player == null) {
             return context.getString(R.string.unknown_or_no_audio)
diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
index 5fdb9592a..4bc7545f8 100644
--- a/app/src/main/res/values/strings.xml
+++ b/app/src/main/res/values/strings.xml
@@ -462,6 +462,7 @@
     <string name="no_chapter">No chapter</string>
     <string name="screen_orientation">Screen orientation</string>
     <string name="auto_generated">auto-generated</string>
+    <string name="resolution_limited">limited</string>
 
     <!-- Notification channel strings -->
     <string name="download_channel_name">Download Service</string>