eb-player 2.4.0 → 2.4.2

package/dist/build/eb-player.css

@@ -3342,12 +3342,18 @@
    here — it cancels the native mirroring. Only elements positioned with a
    physical inline style (chapter markers use right: N% under RTL) need an
    explicit override.
+
+   Both overrides below MUST stay anchored to .eb-player[dir="rtl"]. As bare
+   [dir="rtl"] descendant selectors they also matched through an ancestor —
+   inside an RTL document (<html dir="rtl">) they fired even when the player
+   itself was LTR, mirroring the markers and volume thumb while Seekbar and
+   VolumeControl used their LTR pointer maths. Never widen these back.
    ============================================================ */
-[dir="rtl"] .eb-chapter-marker {
+.eb-player[dir="rtl"] .eb-chapter-marker {
   --eb-chapter-marker-x: 50%;
 }
 
-[dir="rtl"] .eb-volume-thumb {
+.eb-player[dir="rtl"] .eb-volume-thumb {
   --eb-volume-thumb-x: 50%;
 }
 

package/dist/build/ebplayer.bundle.js

@@ -4,7 +4,7 @@
     (global = typeof globalThis !== 'undefined' ? globalThis : global || self, factory(global.EBPlayer = {}));
 })(this, (function (exports) { 'use strict';
 
-    var __EB_PLAYER_VERSION__ = "2.4.0";
+    var __EB_PLAYER_VERSION__ = "2.4.2";
 
     /**
      * Finite State Machine for player playback state transitions.
@@ -3122,7 +3122,7 @@
             return b `
       <div
         class="eb-player${isIOS ? ' eb-ios-native' : ''}"
-        dir="${isRtl ? 'rtl' : ''}"
+        dir="${isRtl ? 'rtl' : 'ltr'}"
       >
         <video
           class="eb-video"
@@ -3434,20 +3434,20 @@
             inner.classList.toggle('eb-ad-playing', this.state.adPlaying);
         }
         /**
-         * Applies or removes the dir=rtl attribute without full re-render.
+         * Writes a concrete dir attribute without full re-render.
          * More efficient than re-rendering the entire template for a simple attribute change.
+         *
+         * Always asserts a direction — never removes the attribute. `dir` is an enumerated
+         * attribute: absent and `""` both mean "inherit from the ancestor", so removing it
+         * inside an RTL document (`<html dir="rtl">`) left the player mirrored by inheritance
+         * while Seekbar/VolumeControl had already switched to their LTR pointer maths. The
+         * player element is the authoritative direction root in BOTH states.
          */
         applyRtl() {
             const inner = this.el?.firstElementChild;
             if (inner == null)
                 return;
-            const isRtl = this.state.isRtl;
-            if (isRtl) {
-                inner.setAttribute('dir', 'rtl');
-            }
-            else {
-                inner.removeAttribute('dir');
-            }
+            inner.setAttribute('dir', this.state.isRtl ? 'rtl' : 'ltr');
         }
         invokeExtensions() {
             const config = this.config;
@@ -6651,7 +6651,7 @@
                 return;
             this.eventState = state;
             const { Events } = Hls;
-            driver.on(Events.MANIFEST_LOADED, this._onManifestLoaded.bind(this));
+            driver.on(Events.MANIFEST_PARSED, this._onManifestParsed.bind(this));
             driver.on(Events.LEVEL_LOADED, this._onLevelLoaded.bind(this));
             driver.on(Events.LEVEL_SWITCHED, this._onLevelSwitched.bind(this));
             driver.on(Events.AUDIO_TRACKS_UPDATED, this._onAudioTracksUpdated.bind(this));
@@ -6696,7 +6696,19 @@
             }
             this.bus?.emit('load-never-started', { src });
         }
-        _onManifestLoaded(_event, data) {
+        /**
+         * Populates the quality menu from MANIFEST_PARSED — never MANIFEST_LOADED.
+         *
+         * MANIFEST_LOADED carries the levels in raw manifest order. hls.js then
+         * builds a separate array (filtering audio-only and redundant variants) and
+         * sorts it lowest-to-highest; that sorted array becomes `hls.levels`, and it
+         * is the index space of `driver.currentLevel` (see setQuality) and of
+         * LEVEL_SWITCHED.level. Labelling from manifest order while selecting in
+         * sorted order silently inverts the menu on any manifest that lists its
+         * variants highest-first. MANIFEST_PARSED carries that same sorted array, so
+         * label and index always agree.
+         */
+        _onManifestParsed(_event, data) {
             const state = this.eventState;
             const driver = this.driver;
             if (!state)