From 555b7838643a2d8563a408eda0af7c9c6ecdbc53 Mon Sep 17 00:00:00 2001
From: Sam <sam.saffron@gmail.com>
Date: Mon, 8 Dec 2014 13:32:26 +1100
Subject: [PATCH] Add shortcuts support for emoji toolbar

(CTRL E to open, ESC to close)
---
 .../discourse/lib/Markdown.Editor.js            | 17 +++++++++++++++++
 .../emoji/assets/javascripts/emoji-toolbar.js   | 12 +++++++++++-
 plugins/emoji/config/locales/client.en.yml      |  2 +-
 3 files changed, 29 insertions(+), 2 deletions(-)

diff --git a/app/assets/javascripts/discourse/lib/Markdown.Editor.js b/app/assets/javascripts/discourse/lib/Markdown.Editor.js
index c72465724c6..41679bb8d0f 100644
--- a/app/assets/javascripts/discourse/lib/Markdown.Editor.js
+++ b/app/assets/javascripts/discourse/lib/Markdown.Editor.js
@@ -1214,6 +1214,17 @@
         makeSpritedButtonRow();
 
         var keyEvent = "keydown";
+
+        var extendedEvents = [];
+
+        if(window.PagedownCustom){
+          window.PagedownCustom.appendButtons.concat(window.PagedownCustom.insertButtons).forEach(function(button){
+            if(button.shortcut){
+              extendedEvents.push([button.shortcut, button.execute]);
+            }
+          });
+        }
+
         util.addEvent(inputBox, keyEvent, function (key) {
 
             // Check to see if we have a button key and, if so execute the callback.
@@ -1222,6 +1233,12 @@
                 var keyCode = key.charCode || key.keyCode;
                 var keyCodeStr = String.fromCharCode(keyCode).toLowerCase();
 
+                for(var i=0; i<extendedEvents.length; i++){
+                  if(keyCodeStr===extendedEvents[i][0]){
+                    extendedEvents[i][1]();
+                  }
+                }
+
                 switch (keyCodeStr) {
                     case "b":
                         doClick(buttons.bold);
diff --git a/plugins/emoji/assets/javascripts/emoji-toolbar.js b/plugins/emoji/assets/javascripts/emoji-toolbar.js
index e3b38407168..7d72e9f6b52 100644
--- a/plugins/emoji/assets/javascripts/emoji-toolbar.js
+++ b/plugins/emoji/assets/javascripts/emoji-toolbar.js
@@ -32,6 +32,8 @@ var renderPage = Handlebars.compile(
 
 var closeSelector = function(){
   $('.emoji-modal, .emoji-modal-wrapper').remove();
+  $('body, textarea').off('keydown.emoji');
+  $('#wmd-input').focus();
 };
 
 var showSelector = function(){
@@ -62,10 +64,18 @@ var showSelector = function(){
      composerController.appendText(":" + $(this).attr('title') + ":", {space: true});
      closeSelector();
   });
+
+  $('body, textarea').on('keydown.emoji', function(e){
+    if(e.which === 27){
+      closeSelector();
+      return false;
+    }
+  });
 };
 
 window.PagedownCustom.appendButtons.push({
   id: 'wmd-emoji-button',
   description: I18n.t("composer.emoji"),
-  execute: showSelector
+  execute: showSelector,
+  shortcut: 'e'
 });
diff --git a/plugins/emoji/config/locales/client.en.yml b/plugins/emoji/config/locales/client.en.yml
index a84b68bce30..d7aaa920edf 100644
--- a/plugins/emoji/config/locales/client.en.yml
+++ b/plugins/emoji/config/locales/client.en.yml
@@ -1,7 +1,7 @@
 en:
   js:
     composer:
-      emoji: "Emoji :smile:"
+      emoji: "Emoji :smile: CTRL+ E"
   admin_js:
     admin:
       site_settings: