haskal
/
jiti-meet
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Removed separate check for SPACE

This commit is contained in:
Maxim Voloshin 2016-09-29 16:07:21 +03:00
parent 33078a868c
commit 6e90b767ef
1 changed files with 5 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
modules/keyboardshortcut/keyboardshortcut.js
View File

@ -17,10 +17,12 @@ function initGlobalShortcuts() {
APP.UI.toggleKeyboardShortcutsPanel();
}, "keyboardShortcuts.toggleShortcuts");
KeyboardShortcut.registerShortcut("SPACE", null, function() {
// register SPACE shortcut in two steps to insure visibility of help message
KeyboardShortcut.registerShortcut(" ", null, function() {
JitsiMeetJS.analytics.sendEvent("shortcut.talk.clicked");
APP.conference.muteAudio(true);
}, "keyboardShortcuts.pushToTalk");
});
KeyboardShortcut._addShortcutToHelp("SPACE","keyboardShortcuts.pushToTalk");
/**
* FIXME: Currently focus keys are directly implemented below in onkeyup.
@ -68,7 +70,7 @@ var KeyboardShortcut = {
$(":focus").is("input[type=password]") ||
$(":focus").is("textarea"))) {
var key = self._getKeyboardKey(e).toUpperCase();
if(key === "SPACE") {
if(key === " ") {
if(APP.conference.isLocalAudioMuted())
APP.conference.muteAudio(false);
}
@ -139,9 +141,6 @@ var KeyboardShortcut = {
* @returns {string} e.key or something close if not supported
*/
_getKeyboardKey: function (e) {
if (e.which === 32) {
return 'SPACE';
}
if (typeof e.key === "string") {
return e.key;
}