Merge pull request #948 from m-voloshin/space-for-ptt

Spacebar is used to activate Push-To-Talk
pull/983/head 1336
yanas 9 years ago committed by GitHub
commit 924bb3c7f7
  1. 8
      modules/keyboardshortcut/keyboardshortcut.js

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

Loading…
Cancel
Save