fix: account for some dynamic changes to media-time-range (#859)

(mostly relevant for DVR).
muxinc · Mar 18, 2024 · bea8fca · bea8fca
1 parent b625e22
commit bea8fca
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/src/js/media-time-range.js b/src/js/media-time-range.js
@@ -426,7 +426,10 @@ class MediaTimeRange extends MediaChromeRange {
       this.updateBufferedBar();
     }
 
-    if (attrName === MediaUIAttributes.MEDIA_DURATION) {
+    if (
+      attrName === MediaUIAttributes.MEDIA_DURATION ||
+      attrName === MediaUIAttributes.MEDIA_SEEKABLE
+    ) {
       this.mediaChaptersCues = this.#mediaChaptersCues;
       this.updateBar();
     }
@@ -445,7 +448,7 @@ class MediaTimeRange extends MediaChromeRange {
       && !this.mediaPaused
       && !this.mediaLoading
       && !this.mediaEnded
-      && this.mediaDuration > 0
+      && this.mediaSeekableEnd > 0
       && isElementVisible(this);
   }
 
@@ -766,7 +769,7 @@ class MediaTimeRange extends MediaChromeRange {
       return;
     }
 
-    const duration = this.mediaDuration;
+    const duration = this.mediaSeekableEnd;
     // If no duration we can't calculate which time to show
     if (!duration) return;
 

diff --git a/src/js/utils/range-animation.js b/src/js/utils/range-animation.js
@@ -38,7 +38,8 @@ export class RangeAnimation {
     // 1. Always allow increases.
     // 2. Allow a relatively large decrease (user action or Safari jumping back :s).
     const increase = start - this.#range.valueAsNumber;
-    if (increase > 0 || increase < -.03) {
+    const durationDelta = Math.abs(duration - this.duration);
+    if (increase > 0 || increase < -.03 || durationDelta >= 0.5) {
       this.callback(start);
     }