Merge pull request #760 from samvera-labs/switching-manifests-bug

Check player is not null before player.trigger('resize') event
samvera-labs · Jan 13, 2025 · 09285aa · 09285aa
2 parents e0064dd + dac1543
commit 09285aa
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/src/services/ramp-hooks.js b/src/services/ramp-hooks.js
@@ -489,7 +489,9 @@ export const useVideoJSPlayer = ({
 
     // Listen for resize events on desktop browsers and trigger player.resize event
     window.addEventListener('resize', () => {
-      player.trigger('resize');
+      // Check if player is initialized before triggering resize event, especially helpful
+      // when switching the Manifest in the demo site without a page reload
+      if (player?.player_) player.trigger('resize');
     });
 
     /**
@@ -499,7 +501,9 @@ export const useVideoJSPlayer = ({
      */
     if (window.visualViewport) {
       window.visualViewport.addEventListener('resize', () => {
-        player.trigger('resize');
+        // Check if player is initialized before triggering resize event, especially helpful
+        // when switching the Manifest in the demo site without a page reload
+        if (player?.player_) player.trigger('resize');
       });
     }
   };