Remember more things

THEOplayer · Feb 16, 2024 · ed22886 · ed22886
1 parent 714eff4
commit ed22886
Showing 1 changed file with 14 additions and 9 deletions.
diff --git a/ui/src/main/java/com/theoplayer/android/ui/SeekBar.kt b/ui/src/main/java/com/theoplayer/android/ui/SeekBar.kt
@@ -27,25 +27,30 @@ fun SeekBar(
     val player = Player.current
     val currentTime = player?.currentTime?.toFloat() ?: 0.0f
     val seekable = player?.seekable ?: TimeRanges(listOf())
-    val valueRange = seekable.bounds ?: 0.0..0.0
 
+    val valueRange = remember(seekable) {
+        val bounds = seekable.bounds ?: 0.0..0.0
+        bounds.start.toFloat()..bounds.endInclusive.toFloat()
+    }
     var seekTime by remember { mutableStateOf<Float?>(null) }
     var wasPlayingBeforeSeek by remember { mutableStateOf(false) }
 
     Slider(
         modifier = modifier,
         colors = colors,
         value = seekTime ?: currentTime,
-        valueRange = valueRange.start.toFloat()..valueRange.endInclusive.toFloat(),
+        valueRange = valueRange,
         enabled = seekable.isNotEmpty(),
-        onValueChange = { time ->
-            seekTime = time
-            player?.player?.let {
-                if (!it.isPaused) {
-                    wasPlayingBeforeSeek = true
-                    it.pause()
+        onValueChange = remember {
+            { time ->
+                seekTime = time
+                player?.player?.let {
+                    if (!it.isPaused) {
+                        wasPlayingBeforeSeek = true
+                        it.pause()
+                    }
+                    it.currentTime = time.toDouble()
                 }
-                it.currentTime = time.toDouble()
             }
         },
         // This needs to always be the *same* callback,