From 9dfa2ad84ebe7aa09c400fc55e9bb6295d029c73 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Florian=20M=C3=BCllner?= Date: Thu, 15 Sep 2011 21:49:13 +0200 Subject: [PATCH] app-display: Remove AppIconMenu::popup The signal is just duplicating the parent class' ::open-state-changed signal, so remove it. --- js/ui/appDisplay.js | 11 +---------- 1 file changed, 1 insertion(+), 10 deletions(-) diff --git a/js/ui/appDisplay.js b/js/ui/appDisplay.js index da371e5f2..e348555fd 100644 --- a/js/ui/appDisplay.js +++ b/js/ui/appDisplay.js @@ -559,7 +559,7 @@ AppWellIcon.prototype = { this._menu.connect('activate-window', Lang.bind(this, function (menu, window) { this.activateWindow(window); })); - this._menu.connect('popup', Lang.bind(this, function (menu, isPoppedUp) { + this._menu.connect('open-state-changed', Lang.bind(this, function (menu, isPoppedUp) { if (!isPoppedUp) this._onMenuPoppedDown(); })); @@ -643,7 +643,6 @@ AppIconMenu.prototype = { this._source = source; this.connect('activate', Lang.bind(this, this._onActivate)); - this.connect('open-state-changed', Lang.bind(this, this._onOpenStateChanged)); this.actor.add_style_class_name('app-well-menu'); @@ -707,14 +706,6 @@ AppIconMenu.prototype = { this.open(); }, - _onOpenStateChanged: function (menu, open) { - if (open) { - this.emit('popup', true); - } else { - this.emit('popup', false); - } - }, - _onActivate: function (actor, child) { if (child._window) { let metaWindow = child._window;