Skip to content

Commit 1fcd5ae

Browse files
shahlabsgkatsev
authored andcommitted
fix: trigger timeupdate during seek (#4754)
If the player is paused, when we seek, the time displays don't update. So, we trigger a timeupdate manually to let components know that the time has updated.
1 parent bbea5cc commit 1fcd5ae

File tree

2 files changed

+55
-46
lines changed

2 files changed

+55
-46
lines changed

package-lock.json

Lines changed: 46 additions & 46 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/js/control-bar/progress-control/seek-bar.js

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -244,6 +244,15 @@ class SeekBar extends Slider {
244244
super.handleMouseUp(event);
245245

246246
this.player_.scrubbing(false);
247+
248+
/**
249+
* Trigger timeupdate because we're done seeking and the time has changed.
250+
* This is particularly useful for if the player is paused to time the time displays.
251+
*
252+
* @event Tech#timeupdate
253+
* @type {EventTarget~Event}
254+
*/
255+
this.player_.trigger({ type: 'timeupdate', target: this, manuallyTriggered: true });
247256
if (this.videoWasPlaying) {
248257
this.player_.play();
249258
}

0 commit comments

Comments
 (0)