unified-video-framework 1.4.250 → 1.4.251

package/packages/web/src/WebPlayer.ts

@@ -1122,6 +1122,11 @@ export class WebPlayer extends BasePlayer {
       // Create YouTube iframe player with custom controls integration
       await this.createYouTubePlayer(videoId);
       
+      // Force another metadata update after player creation
+      setTimeout(() => {
+        this.updateMetadataUI();
+      }, 1000);
+      
       this.debugLog('✅ YouTube video loaded successfully');
       this.debugLog('YouTube video title:', metadata.title);
     } catch (error) {
@@ -1249,6 +1254,12 @@ export class WebPlayer extends BasePlayer {
       
       // Extract available YouTube qualities
       this.extractYouTubeAvailableQualities();
+      
+      // Update metadata UI and settings after player is ready
+      setTimeout(() => {
+        this.updateMetadataUI();
+        this.updateSettingsMenu();
+      }, 500);
     }
 
     // Start time tracking
@@ -1486,16 +1497,18 @@ export class WebPlayer extends BasePlayer {
     
     const percent = (currentTime / duration) * 100;
     
-    // Update progress filled
+    // Update progress filled (only if not dragging)
     const progressFilled = document.getElementById('uvf-progress-filled') as HTMLElement;
     if (progressFilled && !this.isDragging) {
       progressFilled.style.width = percent + '%';
     }
     
-    // Update progress handle
+    // Update progress handle (only if not dragging)
     const progressHandle = document.getElementById('uvf-progress-handle') as HTMLElement;
     if (progressHandle && !this.isDragging) {
       progressHandle.style.left = percent + '%';
+      // Remove dragging class if it was set
+      progressHandle.classList.remove('dragging');
     }
     
     // Update buffered progress
@@ -1504,8 +1517,13 @@ export class WebPlayer extends BasePlayer {
       progressBuffered.style.width = buffered + '%';
     }
     
-    // Update time display
-    this.updateTimeDisplay();
+    // Update time display with YouTube-specific times
+    const timeDisplay = document.getElementById('uvf-time-display');
+    if (timeDisplay) {
+      const currentTimeStr = this.formatTime(currentTime);
+      const durationStr = this.formatTime(duration);
+      timeDisplay.textContent = `${currentTimeStr} / ${durationStr}`;
+    }
   }
 
 
@@ -8128,12 +8146,27 @@ export class WebPlayer extends BasePlayer {
     const progressBar = document.querySelector('.uvf-progress-bar') as HTMLElement;
     const progressFilled = document.getElementById('uvf-progress-filled') as HTMLElement;
     const progressHandle = document.getElementById('uvf-progress-handle') as HTMLElement;
-    if (!progressBar || !this.video) return;
+    if (!progressBar) return;
+    
+    // Get duration from appropriate source
+    let duration = 0;
+    if (this.youtubePlayer && this.youtubePlayerReady) {
+      try {
+        duration = this.youtubePlayer.getDuration() || 0;
+      } catch (error) {
+        this.debugWarn('Error getting YouTube duration for seeking:', error);
+        return;
+      }
+    } else if (this.video) {
+      duration = this.video.duration;
+    } else {
+      this.debugWarn('No video source available for seeking');
+      return;
+    }
     
-    const duration = this.video.duration;
     // Validate duration before calculating seek time
     if (!isFinite(duration) || isNaN(duration) || duration <= 0) {
-      this.debugWarn('Invalid video duration, cannot seek via progress bar');
+      this.debugWarn('Invalid video duration, cannot seek via progress bar:', duration);
       return;
     }
     
@@ -8148,21 +8181,24 @@ export class WebPlayer extends BasePlayer {
       return;
     }
     
+    this.debugLog('Seeking to position:', time, 'seconds (', Math.round(percent), '%)');
+    
     // Update UI immediately for responsive feedback
-    if (progressFilled) {
+    if (progressFilled && !this.isDragging) {
       progressFilled.style.width = percent + '%';
     }
-    if (progressHandle) {
+    if (progressHandle && !this.isDragging) {
       progressHandle.style.left = percent + '%';
-      // Add dragging class for visual feedback
-      if (this.isDragging) {
-        progressHandle.classList.add('dragging');
-      } else {
-        progressHandle.classList.remove('dragging');
-      }
+      progressHandle.classList.add('dragging');
     }
     
+    // Perform the actual seek
     this.seek(time);
+    
+    // For YouTube, provide immediate visual feedback since API might be delayed
+    if (this.youtubePlayer && this.youtubePlayerReady) {
+      this.emit('onSeeking');
+    }
   }
 
   private formatTime(seconds: number): string {
@@ -9196,6 +9232,13 @@ export class WebPlayer extends BasePlayer {
           option.classList.add('active');
         }
       });
+      
+      // Update quality badge
+      const qualityBadge = document.getElementById('uvf-quality-badge');
+      if (qualityBadge) {
+        const qualityOption = this.youtubeAvailableQualities.find(q => q.value === quality);
+        qualityBadge.textContent = qualityOption ? qualityOption.label : 'AUTO';
+      }
       return;
     }
     
@@ -9592,9 +9635,9 @@ export class WebPlayer extends BasePlayer {
    * Detect available video qualities from different sources
    */
   private detectAvailableQualities(): void {
-    // Don't add default 'Auto' for YouTube - it's already included in youtubeAvailableQualities
+    // Check if we're using YouTube and have detected qualities
     const isYouTube = this.youtubePlayer && this.youtubePlayerReady && this.youtubeAvailableQualities.length > 0;
-    this.availableQualities = isYouTube ? [] : [{ value: 'auto', label: 'Auto' }];
+    this.availableQualities = [];
     let detectedQualities: Array<{ value: string; label: string; height?: number }> = [];
 
     // YouTube qualities
@@ -9605,6 +9648,8 @@ export class WebPlayer extends BasePlayer {
         height: q.height
       }));
       this.debugLog('Using YouTube qualities:', detectedQualities);
+      this.availableQualities = detectedQualities;
+      return; // Early return for YouTube
     } else if (this.hls && this.hls.levels) {
       // HLS qualities
       this.hls.levels.forEach((level: any, index: number) => {