Merge pull request #1894 from ErisDS/issue-1558

Notifications can be dismissed with ESC
TryGhost · Jan 9, 2014 · c833946 · c833946
2 parents 117f3c9 + eae2147
commit c833946
Showing 1 changed file with 8 additions and 0 deletions.
diff --git a/core/client/views/base.js b/core/client/views/base.js
@@ -161,6 +161,14 @@
             Ghost.on('urlchange', function () {
                 self.clearEverything();
             });
+            shortcut.add("ESC", function () {
+                // Make sure there isn't currently an open modal, as the escape key should close that first.
+                // This is a temporary solution to enable closing extra-long notifications, and should be refactored
+                // into something more robust in future
+                if ($('.js-modal').length < 1) {
+                    self.clearEverything();
+                }
+            });
         },
         events: {
             'animationend .js-notification': 'removeItem',