[gnome-shell/gbsneto/dont-create-so-many-icons-pretty-please] allView: Rename variable



commit 6cd35ceeb9156b7350648267c24c9418cc5e5d2d
Author: Georges Basile Stavracas Neto <georges stavracas gmail com>
Date:   Thu Nov 21 18:03:34 2019 -0300

    allView: Rename variable
    
    The variable that holds the list of application icons is
    called 'newApps', but that technically was never true,
    since we only create new app icons when necessary.
    
    Rename it to 'appIcons'.
    
    https://gitlab.gnome.org/GNOME/gnome-shell/merge_requests/841

 js/ui/appDisplay.js | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)
---
diff --git a/js/ui/appDisplay.js b/js/ui/appDisplay.js
index 6c5792f64d..cb1478f3be 100644
--- a/js/ui/appDisplay.js
+++ b/js/ui/appDisplay.js
@@ -453,7 +453,7 @@ var AllView = GObject.registerClass({
     }
 
     _loadApps() {
-        let newApps = [];
+        let appIcons = [];
         this._appInfoList = Shell.AppSystem.get_default().get_installed().filter(appInfo => {
             try {
                 appInfo.get_id(); // catch invalid file encodings
@@ -478,7 +478,7 @@ var AllView = GObject.registerClass({
                 icon.connect('name-changed', this._itemNameChanged.bind(this));
                 icon.connect('apps-changed', this._redisplay.bind(this));
             }
-            newApps.push(icon);
+            appIcons.push(icon);
             this.folderIcons.push(icon);
         });
 
@@ -499,10 +499,10 @@ var AllView = GObject.registerClass({
                                    { isDraggable: favoritesWritable });
             }
 
-            newApps.push(icon);
+            appIcons.push(icon);
         });
 
-        return newApps;
+        return appIcons;
     }
 
     // Overridden from BaseAppView


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]