From 034a723677161a845f368c57300991df94af5a54 Mon Sep 17 00:00:00 2001 From: Georges Basile Stavracas Neto Date: Tue, 21 Aug 2018 06:21:26 -0300 Subject: [PATCH] iconGrid: Rename ::key-focus-in signal As part of our quest to obsolete Shell.GenericContainer, IconGrid will become a Clutter.Actor subclass. As the ::key-focus-in signal would clash with Clutter.Actor::key-focus-in, rename it to ::child-focused. https://gitlab.gnome.org/GNOME/gnome-shell/merge_requests/153 --- js/ui/appDisplay.js | 10 +++++----- js/ui/iconGrid.js | 2 +- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/js/ui/appDisplay.js b/js/ui/appDisplay.js index 740f0c240..9d648c92e 100644 --- a/js/ui/appDisplay.js +++ b/js/ui/appDisplay.js @@ -126,8 +126,8 @@ var BaseAppView = new Lang.Class({ else this._grid = new IconGrid.IconGrid(gridParams); - this._grid.connect('key-focus-in', (grid, actor) => { - this._keyFocusIn(actor); + this._grid.connect('child-focused', (grid, actor) => { + this._childFocused(actor); }); // Standard hack for ClutterBinLayout this._grid.actor.x_expand = true; @@ -136,7 +136,7 @@ var BaseAppView = new Lang.Class({ this._allItems = []; }, - _keyFocusIn(actor) { + _childFocused(actor) { // Nothing by default }, @@ -717,7 +717,7 @@ var AllView = new Lang.Class({ }); }, - _keyFocusIn(icon) { + _childFocused(icon) { let itemPage = this._grid.getItemPage(icon); this.goToPage(itemPage); }, @@ -1154,7 +1154,7 @@ var FolderView = new Lang.Class({ this.actor.add_action(action); }, - _keyFocusIn(actor) { + _childFocused(actor) { Util.ensureActorVisibleInScrollView(this.actor, actor); }, diff --git a/js/ui/iconGrid.js b/js/ui/iconGrid.js index bf8e081e5..a0d008386 100644 --- a/js/ui/iconGrid.js +++ b/js/ui/iconGrid.js @@ -238,7 +238,7 @@ var IconGrid = new Lang.Class({ }, _keyFocusIn(actor) { - this.emit('key-focus-in', actor); + this.emit('child-focused', actor); }, _childAdded(grid, child) {