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

appDisplay: Don't emit 'apps-changed' when deleting folder

Browse Source 
To delete a folder, FolderView needs to reset all keys under that particular
folder's GSettings path. That generates 5 'changed' signals, all of which
end up calling AppDisplay._redisplay(), which is costly.

Don't emit 'apps-changed' when deleting a folder.

https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/1447
This commit is contained in:
Georges Basile Stavracas Neto 2020-09-24 13:36:40 -03:00 committed by Florian Müllner
parent 31591ff029
commit 7dafd25ef4
1 changed files with 12 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
js/ui/appDisplay.js
View File

@ -1639,6 +1639,7 @@ class FolderView extends BaseAppView {
action.connect('pan', this._onPan.bind(this));
this._scrollView.add_action(action);
this._deletingFolder = false;
this._appIds = [];
this._redisplay();
}
@ -1809,6 +1810,8 @@ class FolderView extends BaseAppView {
// Remove the folder if this is the last app icon; otherwise,
// just remove the icon
if (folderApps.length == 0) {
this._deletingFolder = true;
// Resetting all keys deletes the relocatable schema
let keys = this._folder.settings_schema.list_keys();
for (const key of keys)
@ -1818,6 +1821,8 @@ class FolderView extends BaseAppView {
let folders = settings.get_strv('folder-children');
folders.splice(folders.indexOf(this._id), 1);
settings.set_strv('folder-children', folders);
this._deletingFolder = false;
} else {
// If this is a categories-based folder, also add it to
// the list of excluded apps
@ -1831,6 +1836,10 @@ class FolderView extends BaseAppView {
this._folder.set_strv('apps', folderApps);
}
}
get deletingFolder() {
return this._deletingFolder;
}
});
var FolderIcon = GObject.registerClass({
@ -1972,6 +1981,9 @@ var FolderIcon = GObject.registerClass({
}
_sync() {
if (this.view.deletingFolder)
return;
this.emit('apps-changed');
this._updateName();
this.visible = this.view.getAllItems().length > 0;