From 2451f222e845c6ade8fba3d5d88aab89a722ac64 Mon Sep 17 00:00:00 2001
From: Pavel Djundik <xPaw@users.noreply.github.com>
Date: Fri, 24 Jan 2020 11:32:28 +0200
Subject: [PATCH 1/2] Bind esc key handler once

Fixes #3690
---
 client/components/App.vue         | 5 +++++
 client/components/ContextMenu.vue | 7 ++-----
 client/components/ImageViewer.vue | 6 ++----
 3 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/client/components/App.vue b/client/components/App.vue
index cd378127..3133101f 100644
--- a/client/components/App.vue
+++ b/client/components/App.vue
@@ -44,6 +44,7 @@ export default {
 		this.prepareOpenStates();
 	},
 	mounted() {
+		Mousetrap.bind("esc", this.escapeKey);
 		Mousetrap.bind("alt+u", this.toggleUserList);
 		Mousetrap.bind("alt+s", this.toggleSidebar);
 
@@ -64,6 +65,7 @@ export default {
 		this.dayChangeTimeout = setTimeout(emitDayChange, this.msUntilNextDay());
 	},
 	beforeDestroy() {
+		Mousetrap.unbind("esc", this.escapeKey);
 		Mousetrap.unbind("alt+u", this.toggleUserList);
 		Mousetrap.unbind("alt+s", this.toggleSidebar);
 
@@ -71,6 +73,9 @@ export default {
 		clearTimeout(this.dayChangeTimeout);
 	},
 	methods: {
+		escapeKey() {
+			this.$root.$emit("escapekey");
+		},
 		toggleSidebar(e) {
 			// Do not handle this keybind in the chat input because
 			// it can be used to type letters with umlauts
diff --git a/client/components/ContextMenu.vue b/client/components/ContextMenu.vue
index 75b484f2..f060bb0d 100644
--- a/client/components/ContextMenu.vue
+++ b/client/components/ContextMenu.vue
@@ -38,7 +38,6 @@
 </template>
 
 <script>
-import Mousetrap from "mousetrap";
 import {
 	generateUserContextMenu,
 	generateChannelContextMenu,
@@ -63,15 +62,13 @@ export default {
 		};
 	},
 	mounted() {
-		Mousetrap.bind("esc", this.close);
-
+		this.$root.$on("escapekey", this.close);
 		this.$root.$on("contextmenu:user", this.openUserContextMenu);
 		this.$root.$on("contextmenu:channel", this.openChannelContextMenu);
 		this.$root.$on("contextmenu:removenetwork", this.openRemoveNetworkContextMenu);
 	},
 	destroyed() {
-		Mousetrap.unbind("esc", this.close);
-
+		this.$root.$off("escapekey", this.close);
 		this.$root.$off("contextmenu:user", this.openUserContextMenu);
 		this.$root.$off("contextmenu:channel", this.openChannelContextMenu);
 		this.$root.$off("contextmenu:removenetwork", this.openRemoveNetworkContextMenu);
diff --git a/client/components/ImageViewer.vue b/client/components/ImageViewer.vue
index 00130581..53fc3f13 100644
--- a/client/components/ImageViewer.vue
+++ b/client/components/ImageViewer.vue
@@ -25,8 +25,6 @@
 </template>
 
 <script>
-import Mousetrap from "mousetrap";
-
 export default {
 	name: "ImageViewer",
 	data() {
@@ -68,10 +66,10 @@ export default {
 		},
 	},
 	mounted() {
-		Mousetrap.bind("esc", this.closeViewer);
+		this.$root.$on("escapekey", this.closeViewer);
 	},
 	destroyed() {
-		Mousetrap.unbind("esc", this.closeViewer);
+		this.$root.$off("escapekey", this.closeViewer);
 	},
 	methods: {
 		closeViewer() {

From bbe6b343713e7e3bfe7f9acb253271dbe6d51806 Mon Sep 17 00:00:00 2001
From: Pavel Djundik <xPaw@users.noreply.github.com>
Date: Fri, 24 Jan 2020 11:35:04 +0200
Subject: [PATCH 2/2] Unfocus chat input when pressing escape

---
 client/components/ChatInput.vue | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/client/components/ChatInput.vue b/client/components/ChatInput.vue
index 481575e1..63732206 100644
--- a/client/components/ChatInput.vue
+++ b/client/components/ChatInput.vue
@@ -101,6 +101,8 @@ export default {
 		},
 	},
 	mounted() {
+		this.$root.$on("escapekey", this.blurInput);
+
 		if (this.$store.state.settings.autocomplete) {
 			autocompletionRef = autocompletion(this.$refs.input);
 		}
@@ -161,6 +163,8 @@ export default {
 		}
 	},
 	destroyed() {
+		this.$root.$off("escapekey", this.blurInput);
+
 		if (autocompletionRef) {
 			autocompletionRef.destroy();
 			autocompletionRef = null;
@@ -255,6 +259,9 @@ export default {
 		openFileUpload() {
 			this.$refs.uploadInput.click();
 		},
+		blurInput() {
+			this.$refs.input.blur();
+		},
 	},
 };
 </script>