From af79479716dc387b5f138757f7015e5caecb2145 Mon Sep 17 00:00:00 2001 From: litetex <40789489+litetex@users.noreply.github.com> Date: Tue, 25 Jan 2022 20:44:49 +0100 Subject: [PATCH] Fixed "Changing the seeks duration does not update the displayed seconds" --- app/src/main/java/org/schabi/newpipe/player/Player.java | 3 ++- .../schabi/newpipe/views/player/PlayerFastSeekOverlay.kt | 9 ++++----- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/app/src/main/java/org/schabi/newpipe/player/Player.java b/app/src/main/java/org/schabi/newpipe/player/Player.java index 029b33b7f..b16bd8b0e 100644 --- a/app/src/main/java/org/schabi/newpipe/player/Player.java +++ b/app/src/main/java/org/schabi/newpipe/player/Player.java @@ -586,7 +586,8 @@ public final class Player implements */ private void setupPlayerSeekOverlay() { binding.fastSeekOverlay - .seekSeconds((int) (retrieveSeekDurationFromPreferences(this) / 1000.0f)) + .seekSecondsSupplier( + () -> (int) (retrieveSeekDurationFromPreferences(this) / 1000.0f)) .performListener(new PlayerFastSeekOverlay.PerformListener() { @Override diff --git a/app/src/main/java/org/schabi/newpipe/views/player/PlayerFastSeekOverlay.kt b/app/src/main/java/org/schabi/newpipe/views/player/PlayerFastSeekOverlay.kt index 522977317..649b60494 100644 --- a/app/src/main/java/org/schabi/newpipe/views/player/PlayerFastSeekOverlay.kt +++ b/app/src/main/java/org/schabi/newpipe/views/player/PlayerFastSeekOverlay.kt @@ -42,11 +42,10 @@ class PlayerFastSeekOverlay(context: Context, attrs: AttributeSet?) : performListener = listener } - var seekSeconds: Int = 0 - private set + private var seekSecondsSupplier: () -> Int = { 0 } - fun seekSeconds(seconds: Int) = apply { - seekSeconds = seconds + fun seekSecondsSupplier(supplier: () -> Int) = apply { + seekSecondsSupplier = supplier } // Indicates whether this (double) tap is the first of a series @@ -94,7 +93,7 @@ class PlayerFastSeekOverlay(context: Context, attrs: AttributeSet?) : performListener?.onDoubleTap() - secondsView.seconds += seekSeconds + secondsView.seconds += seekSecondsSupplier.invoke() performListener?.seek(forward = shouldForward) }