brl/gnome-shell
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

appDisplay: Separate app loading from filling the grid

Browse Source 
At the moment when loading the applications, each app is inserted
at its correct (alphabetical) position. Avoid this overhead by
loading all apps first, then sort them once and fill the grid with
the sorted actors.

https://bugzilla.gnome.org/show_bug.cgi?id=694192
This commit is contained in:
Florian Müllner
2013-02-19 21:31:05 +01:00
parent 6e3e2d9f29
commit 65f96494f8
1 changed files with 14 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
js/ui/appDisplay.js
View File

@ -89,12 +89,21 @@ const AlphabeticalView = new Lang.Class({
return null;
let itemIcon = this._createItemIcon(item);
let pos = Util.insertSorted(this._allItems, item, this._compareItems);
this._grid.addItem(itemIcon.actor, pos);
this._allItems.push(item);
this._items[id] = itemIcon;
return itemIcon;
},
loadGrid: function() {
this._allItems.sort(this._compareItems);
for (let i = 0; i < this._allItems.length; i++) {
let id = this._getItemId(this._allItems[i]);
if (!id)
continue;
this._grid.addItem(this._items[id].actor);
}
}
});
@ -276,6 +285,7 @@ const AppDisplay = new Lang.Class({
_loadCategory(dir, this._view);
}
}
this._view.loadGrid();
if (this._focusDummy) {
let focused = this._focusDummy.has_key_focus();
@ -367,6 +377,7 @@ const FolderIcon = new Lang.Class({
this.view = new FolderView();
this.view.actor.reactive = false;
_loadCategory(dir, this.view);
this.view.loadGrid();
this.actor.connect('clicked', Lang.bind(this,
function() {