From e723484498a631803961b68468611aede94c8709 Mon Sep 17 00:00:00 2001
From: Daniel Gasienica <daniel@signal.org>
Date: Tue, 8 May 2018 17:30:11 -0400
Subject: [PATCH] Simplify closing emoji panel on escape key down

---
 js/views/conversation_view.js | 16 ++++++++++------
 1 file changed, 10 insertions(+), 6 deletions(-)

diff --git a/js/views/conversation_view.js b/js/views/conversation_view.js
index 715cc9b79..c6d8d3f6e 100644
--- a/js/views/conversation_view.js
+++ b/js/views/conversation_view.js
@@ -223,7 +223,7 @@
     },
 
     events: {
-      'keydown': 'escapeEmojiPanel',
+      keydown: 'onKeyDown',
       'submit .send': 'checkUnverifiedSendMessage',
       'input .send-message': 'updateMessageFieldSize',
       'keydown .send-message': 'updateMessageFieldSize',
@@ -1202,12 +1202,12 @@
         this.closeEmojiPanel();
       }
     },
-    escapeEmojiPanel(e){
-      // Close emoji panel on escape key press
-      const keyCode = e.keyCode || e.which;
-      if (keyCode == 27 && this.emojiPanel != null) {
-        this.closeEmojiPanel();
+    onKeyDown(event) {
+      if (event.key !== 'Escape') {
+        return;
       }
+
+      this.closeEmojiPanel();
     },
     openEmojiPanel() {
       this.$emojiPanelContainer.outerHeight(200);
@@ -1217,6 +1217,10 @@
       this.updateMessageFieldSize({});
     },
     closeEmojiPanel() {
+      if (this.emojiPanel === null) {
+        return;
+      }
+
       this.$emojiPanelContainer.empty().outerHeight(0);
       this.emojiPanel = null;
       this.updateMessageFieldSize({});