From cd4076a566bbdf9cfc020909f7454d833d8d070d Mon Sep 17 00:00:00 2001 From: Grzegorz Blaszczyk Date: Tue, 24 Dec 2019 17:24:50 +0100 Subject: [PATCH] fix: updates seekbar position after mouse up event is triggered. (#6372) When we are paused, call update directly, otherwise, call the throttled update. Fixes #6232, fixes #6234, fixes #6370. --- src/js/control-bar/progress-control/seek-bar.js | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/src/js/control-bar/progress-control/seek-bar.js b/src/js/control-bar/progress-control/seek-bar.js index 28e52771d5..73e4b91fd9 100644 --- a/src/js/control-bar/progress-control/seek-bar.js +++ b/src/js/control-bar/progress-control/seek-bar.js @@ -52,7 +52,8 @@ class SeekBar extends Slider { * @private */ setEventHandlers_() { - this.update = Fn.throttle(Fn.bind(this, this.update), UPDATE_REFRESH_INTERVAL); + this.update_ = Fn.bind(this, this.update); + this.update = Fn.throttle(this.update_, UPDATE_REFRESH_INTERVAL); this.on(this.player_, ['ended', 'durationchange', 'timeupdate'], this.update); if (this.player_.liveTracker) { @@ -335,6 +336,10 @@ class SeekBar extends Slider { this.player_.trigger({ type: 'timeupdate', target: this, manuallyTriggered: true }); if (this.videoWasPlaying) { silencePromise(this.player_.play()); + } else { + // We're done seeking and the time has changed. + // If the player is paused, make sure we display the correct time on the seek bar. + this.update_(); } }