[gnome-shell/wip/pressure: 11/14] popupMenu: Remove our own Escape handling
- From: Jasper St. Pierre <jstpierre src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-shell/wip/pressure: 11/14] popupMenu: Remove our own Escape handling
- Date: Mon, 30 Jul 2012 23:48:46 +0000 (UTC)
commit 278704d7ae2e823033603f776e7be4bdc34544d0
Author: Jasper St. Pierre <jstpierre mecheye net>
Date: Wed Feb 29 18:37:52 2012 -0500
popupMenu: Remove our own Escape handling
GrabHelper does this for us
js/ui/popupMenu.js | 20 --------------------
1 files changed, 0 insertions(+), 20 deletions(-)
---
diff --git a/js/ui/popupMenu.js b/js/ui/popupMenu.js
index 04888db..205448f 100644
--- a/js/ui/popupMenu.js
+++ b/js/ui/popupMenu.js
@@ -1186,7 +1186,6 @@ const PopupMenu = new Lang.Class({
this.actor = this._boxPointer.actor;
this.actor._delegate = this;
this.actor.style_class = 'popup-menu-boxpointer';
- this.actor.connect('key-press-event', Lang.bind(this, this._onKeyPressEvent));
this._boxWrapper = new Shell.GenericContainer();
this._boxWrapper.connect('get-preferred-width', Lang.bind(this, this._boxGetPreferredWidth));
@@ -1216,15 +1215,6 @@ const PopupMenu = new Lang.Class({
this.box.allocate(box, flags);
},
- _onKeyPressEvent: function(actor, event) {
- if (event.get_key_symbol() == Clutter.Escape) {
- this.close(BoxPointer.PopupAnimation.FULL);
- return true;
- }
-
- return false;
- },
-
setArrowOrigin: function(origin) {
this._boxPointer.setArrowOrigin(origin);
},
@@ -1513,7 +1503,6 @@ const PopupComboMenu = new Lang.Class({
this.actor = this.box;
this.actor._delegate = this;
- this.actor.connect('key-press-event', Lang.bind(this, this._onKeyPressEvent));
this.actor.connect('key-focus-in', Lang.bind(this, this._onKeyFocusIn));
sourceActor.connect('style-changed',
Lang.bind(this, this._onSourceActorStyleChanged));
@@ -1521,15 +1510,6 @@ const PopupComboMenu = new Lang.Class({
global.focus_manager.add_group(this.actor);
},
- _onKeyPressEvent: function(actor, event) {
- if (event.get_key_symbol() == Clutter.Escape) {
- this.close(BoxPointer.PopupAnimation.FULL);
- return true;
- }
-
- return false;
- },
-
_onKeyFocusIn: function(actor) {
let items = this._getMenuItems();
let activeItem = items[this._activeItemPos];
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]