Merge pull request #300 from Beldaa/fix/webcams-null-currenttime

fix/replay-crash-mobile-shortcuts-webcams

Author: antobinary

src/components/webcams/index.js

@@ -110,8 +110,10 @@ const Webcams = () => {
         player.webcams.on('play', () => {
           const frequency = getFrequency();
           interval.current = setInterval(() => {
-            const currentTime = player.webcams.currentTime();
-            dispatchTimeUpdate(currentTime);
+            if (player.webcams) {
+              const currentTime = player.webcams.currentTime();
+              dispatchTimeUpdate(currentTime);
+            }
           }, 1000 / (frequency ? frequency : config.rps));
         });
 

src/utils/shortcuts.js

@@ -43,18 +43,19 @@ export default class Shortcuts {
       return null;
     }
 
-    const listener = document.addEventListener('keydown', (e) => {
-      if (e.altKey && e.shiftKey) {
-        if (e.key === key) action();
+    const handler = (e) => {
+      if (e.altKey && e.shiftKey && e.key === key) {
+        action();
       }
-    });
+    };
 
-    this.listeners.push(listener);
+    document.addEventListener('keydown', handler);
+    this.listeners.push(handler);
   }
 
   destroy() {
     this.listeners.forEach(listener => {
-      document.removeEventListener(listener);
+      document.removeEventListener('keydown', listener);
     });
   }
 }