From 11761689600cedd60bfc1aa8582f5e11602eccd1 Mon Sep 17 00:00:00 2001
From: Matthew Hodgson <matthew@matrix.org>
Date: Fri, 14 Aug 2015 15:50:37 +0100
Subject: [PATCH] escape key to cancel imageview popup

---
 skins/base/views/atoms/ImageView.js | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)

diff --git a/skins/base/views/atoms/ImageView.js b/skins/base/views/atoms/ImageView.js
index 898187b324..196e92fe9c 100644
--- a/skins/base/views/atoms/ImageView.js
+++ b/skins/base/views/atoms/ImageView.js
@@ -24,6 +24,23 @@ module.exports = React.createClass({
     displayName: 'ImageView',
     mixins: [ImageViewController],
 
+    // XXX: keyboard shortcuts for managing dialogs should be done by the modal dialog base class omehow, surely...
+    componentDidMount: function() {
+        document.addEventListener("keydown", this.onKeyDown);
+    },
+
+    componentWillUnmount: function() {
+        document.removeEventListener("keydown", this.onKeyDown);
+    },
+
+    onKeyDown: function(ev) {
+        if (ev.keyCode == 27) { // escape
+            ev.stopPropagation();
+            ev.preventDefault();
+            this.props.onFinished();
+        }
+    },
+
     render: function() {
 
         // XXX: can't we just do max-width: 80%, max-height: 80% on the CSS?