From 0dd5b5251a7c1cd2f13c7baf18da9b08755d3ec6 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Florian=20M=C3=BCllner?= Date: Sat, 27 Sep 2014 11:56:00 +0200 Subject: [PATCH] alternateTab: Fix Escape not dismissing popup Since shell commit dd85670f8b25, the handler got a return value to determine whether Escape should be handled automatically; we do want this for the window switcher, so add an appropriate return value to restore the expected behavior. https://bugzilla.gnome.org/show_bug.cgi?id=737457 --- extensions/alternate-tab/extension.js | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/extensions/alternate-tab/extension.js b/extensions/alternate-tab/extension.js index cecfc486..0a119700 100644 --- a/extensions/alternate-tab/extension.js +++ b/extensions/alternate-tab/extension.js @@ -44,7 +44,11 @@ function enable() { this._select(this._previous()); else if (keysym == Clutter.Right) this._select(this._next()); + else + return Clutter.EVENT_PROPAGATE; } + + return Clutter.EVENT_STOP; }; setKeybinding('switch-applications', Lang.bind(Main.wm, Main.wm._startWindowSwitcher));