[gnome-shell/gbsneto/icon-grid-dnd: 21/41] baseAppView: Remove BaseAppView.addItem()



commit b0e4c50cbe20e8cc4e0e2447de18ce68815aa7dd
Author: Georges Basile Stavracas Neto <georges stavracas gmail com>
Date:   Tue Jul 2 10:12:08 2019 -0300

    baseAppView: Remove BaseAppView.addItem()
    
    Now that BaseAppView does not allow for subclasses to
    add and remove items directly, the addItem() method
    can be removed.
    
    Remove BaseAppView.addItem().
    
    https://gitlab.gnome.org/GNOME/gnome-shell/merge_requests/603

 js/ui/appDisplay.js | 15 ++++-----------
 1 file changed, 4 insertions(+), 11 deletions(-)
---
diff --git a/js/ui/appDisplay.js b/js/ui/appDisplay.js
index 2b7e21c24..431cb20e8 100644
--- a/js/ui/appDisplay.js
+++ b/js/ui/appDisplay.js
@@ -134,8 +134,10 @@ class BaseAppView {
         }
 
         // Add new app icons
-        for (let app of addedApps)
-            this.addItem(app);
+        for (let appIcon of addedApps) {
+            this._allItems.push(appIcon);
+            this._items[appIcon.id] = appIcon;
+        }
 
         this._loadGrid();
     }
@@ -144,15 +146,6 @@ class BaseAppView {
         return this._allItems;
     }
 
-    addItem(icon) {
-        let id = icon.id;
-        if (this._items[id] !== undefined)
-            return;
-
-        this._allItems.push(icon);
-        this._items[id] = icon;
-    }
-
     _compareItems(a, b) {
         return a.name.localeCompare(b.name);
     }


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