Don't conflict with alt + num web browser hotkey

diff --git a/client/src/assets/player/shared/hotkeys/peertube-hotkeys-plugin.ts b/client/src/assets/player/shared/hotkeys/peertube-hotkeys-plugin.ts
index 8255245b51d8746323b9235a9448509e785192a3..ec1e1038bfed90f7bb2bb342d9fbc45fa920da5b 100644 (file)
--- a/client/src/assets/player/shared/hotkeys/peertube-hotkeys-plugin.ts
+++ b/client/src/assets/player/shared/hotkeys/peertube-hotkeys-plugin.ts
@@ -160,7 +160,7 @@ class PeerTubeHotkeysPlugin extends Plugin {
     // 0-9 key handlers
     for (let i = 0; i < 10; i++) {
       handlers.push({
-        accept: e => e.key === i + '' && !e.ctrlKey, // If using ctrl key, it's a web browser hotkey
+        accept: e => this.isNakedOrShift(e, i + ''),
         cb: e => {
           e.preventDefault()
 
@@ -190,6 +190,10 @@ class PeerTubeHotkeysPlugin extends Plugin {
   private isNaked (event: KeyboardEvent, key: string) {
     return (!event.ctrlKey && !event.altKey && !event.metaKey && !event.shiftKey && event.key === key)
   }
+
+  private isNakedOrShift (event: KeyboardEvent, key: string) {
+    return (!event.ctrlKey && !event.altKey && !event.metaKey && event.key === key)
+  }
 }
 
 videojs.registerPlugin('peerTubeHotkeysPlugin', PeerTubeHotkeysPlugin)