From 0711a70398432f1e5437b61a44a9e5b136aed1fc Mon Sep 17 00:00:00 2001 From: "Jasper St. Pierre" Date: Wed, 15 Aug 2012 12:22:01 -0400 Subject: [PATCH] js: Use global.focus_manager, rather than grabbing it ourselves --- js/ui/ctrlAltTab.js | 5 ++--- js/ui/messageTray.js | 9 +++------ js/ui/panelMenu.js | 3 +-- 3 files changed, 6 insertions(+), 11 deletions(-) diff --git a/js/ui/ctrlAltTab.js b/js/ui/ctrlAltTab.js index 193f14ed9..3462eac23 100644 --- a/js/ui/ctrlAltTab.js +++ b/js/ui/ctrlAltTab.js @@ -27,7 +27,6 @@ const CtrlAltTabManager = new Lang.Class({ _init: function() { this._items = []; - this._focusManager = St.FocusManager.get_for_stage(global.stage); }, addGroup: function(root, name, icon, params) { @@ -41,11 +40,11 @@ const CtrlAltTabManager = new Lang.Class({ this._items.push(item); root.connect('destroy', Lang.bind(this, function() { this.removeGroup(root); })); - this._focusManager.add_group(root); + global.focus_manager.add_group(root); }, removeGroup: function(root) { - this._focusManager.remove_group(root); + global.focus_manager.remove_group(root); for (let i = 0; i < this._items.length; i++) { if (this._items[i].root == root) { this._items.splice(i, 1); diff --git a/js/ui/messageTray.js b/js/ui/messageTray.js index 4fcb271e7..986214a4d 100644 --- a/js/ui/messageTray.js +++ b/js/ui/messageTray.js @@ -314,8 +314,6 @@ const Notification = new Lang.Class({ this._table.connect('style-changed', Lang.bind(this, this._styleChanged)); this.actor.set_child(this._table); - this._buttonFocusManager = St.FocusManager.get_for_stage(global.stage); - // The first line should have the title, followed by the // banner text, but ellipsized if they won't both fit. We can't // make St.Table or St.BoxLayout do this the way we want (don't @@ -638,10 +636,10 @@ const Notification = new Lang.Class({ } if (this._buttonBox.get_n_children() > 0) - this._buttonFocusManager.remove_group(this._buttonBox); + global.focus_manager.remove_group(this._buttonBox); this._buttonBox.add(button); - this._buttonFocusManager.add_group(this._buttonBox); + global.focus_manager.add_group(this._buttonBox); button.connect('clicked', Lang.bind(this, this._onActionInvoked, id)); this.updated(); @@ -1211,8 +1209,7 @@ const SummaryItem = new Lang.Class({ this.rightClickMenu.add(item.actor); } - let focusManager = St.FocusManager.get_for_stage(global.stage); - focusManager.add_group(this.rightClickMenu); + global.focus_manager.add_group(this.rightClickMenu); }, prepareNotificationStackForShowing: function() { diff --git a/js/ui/panelMenu.js b/js/ui/panelMenu.js index f1800694b..542bb72e6 100644 --- a/js/ui/panelMenu.js +++ b/js/ui/panelMenu.js @@ -182,8 +182,7 @@ const Button = new Lang.Class({ _onMenuKeyPress: function(actor, event) { let symbol = event.get_key_symbol(); if (symbol == Clutter.KEY_Left || symbol == Clutter.KEY_Right) { - let focusManager = St.FocusManager.get_for_stage(global.stage); - let group = focusManager.get_group(this.actor); + let group = global.focus_manager.get_group(this.actor); if (group) { let direction = (symbol == Clutter.KEY_Left) ? Gtk.DirectionType.LEFT : Gtk.DirectionType.RIGHT; group.navigate_focus(this.actor, direction, false);