fix(FEC-7642): firefox speed selector keyboard shortcut doesnt work (#…

…368) different key code in firefox for semi colon
kaltura · Mar 19, 2019 · ce9d3b8 · ce9d3b8
1 parent 809b384
commit ce9d3b8
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 0 deletions.
diff --git a/src/ui-manager.js b/src/ui-manager.js
@@ -7,6 +7,7 @@ import {copyDeep} from './utils/copy-deep';
 import {mergeDeep} from './utils/merge-deep';
 import {LogLevel, getLogLevel, setLogLevel} from './utils/logger';
 import {EventType} from './event/event-type';
+import {setEnv} from './utils/key-map';
 
 import reducer from './store';
 import en_translations from './translations/en.json';
@@ -53,6 +54,7 @@ class UIManager {
     this._createStore(config);
     this.setConfig(config);
     this._setLocaleTranslations(config);
+    setEnv(this.player.env);
   }
 
   /**

diff --git a/src/utils/key-map.js b/src/utils/key-map.js
@@ -18,6 +18,15 @@ export const KeyMap: {[key: string]: number} = {
   PERIOD: 190
 };
 
+/**
+ * set env for keymap
+ * @param {Object} env - env object
+ * @returns {void}
+ */
+export function setEnv(env: Object): void {
+  KeyMap.SEMI_COLON = env.browser.name.toLowerCase() === 'firefox' ? 59 : 186;
+}
+
 /**
  * gets the key name for a certain key code
  * @param {number} keyCode - key code