[gnome-shell/gbsneto/icon-grid-part1: 6/8] baseAppView: Remove unused BaseAppView methods



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

    baseAppView: Remove unused BaseAppView methods
    
    Now that BaseAppView does not allow for subclasses to add
    and remove items directly, the addItem() method can be
    removed.
    
    Equally, BaseAppView.hasItem() is now unused since it's
    functionality is implicitly performed by the diff algorithm.
    
    Remove BaseAppView.addItem() and BaseAppView.hasItem().
    
    https://gitlab.gnome.org/GNOME/gnome-shell/merge_requests/645

 js/ui/appDisplay.js | 13 -------------
 1 file changed, 13 deletions(-)
---
diff --git a/js/ui/appDisplay.js b/js/ui/appDisplay.js
index bcd1c8f68..036762fb5 100644
--- a/js/ui/appDisplay.js
+++ b/js/ui/appDisplay.js
@@ -148,19 +148,6 @@ class BaseAppView {
         return this._allItems;
     }
 
-    hasItem(id) {
-        return this._items[id] !== undefined;
-    }
-
-    addItem(icon) {
-        let id = icon.id;
-        if (this.hasItem(id))
-            throw new Error(`icon with id ${id} already added to view`);
-
-        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]