[gnome-shell/gbsneto/custom-icon-positions: 8/17] appDisplay: Use custom function to retrieve item page and position

[Georges Basile Stavracas Neto <gbsneto src gnome org>]

commit 063d2ca26d1913fcd8170960107f945fdcaea153
Author: Georges Basile Stavracas Neto <georges stavracas gmail com>
Date:   Tue May 26 01:41:43 2020 -0300

    appDisplay: Use custom function to retrieve item page and position
    
    It is important that '_loadApps()' return a sorted list -- adding the
    same icons at the same positions but in different orders results in
    a wrong icon grid.
    
    Add support for using a custom positioning function, and implement it
    in AppDisplay. Because FolderView doesn't implement a custom sorting
    function, the items are still sorted alphabetically.
    
    https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/1284

 js/ui/appDisplay.js | 38 ++++++++++++++++++++++++++++++++------
 1 file changed, 32 insertions(+), 6 deletions(-)
---
diff --git a/js/ui/appDisplay.js b/js/ui/appDisplay.js
index a0bfbb5910..9c78103ec8 100644
--- a/js/ui/appDisplay.js
+++ b/js/ui/appDisplay.js
@@ -315,15 +315,20 @@ var BaseAppView = GObject.registerClass({
 
         // Add new app icons
         const { itemsPerPage } = this._grid;
-        addedApps.forEach(icon => {
-            let iconIndex = newApps.indexOf(icon);
-
-            this._orderedItems.splice(iconIndex, 0, icon);
-            this._items.set(icon.id, icon);
+        const getItemPosition = item => {
+            if (this._getItemPosition)
+                return this._getItemPosition(item);
 
+            const iconIndex = newApps.indexOf(item);
             const page = Math.floor(iconIndex / itemsPerPage);
             const position = iconIndex % itemsPerPage;
-            this._grid.addItem(icon, page, position);
+
+            return [page, position];
+        };
+
+        addedApps.forEach(icon => {
+            const [page, position] = getItemPosition(icon);
+            this._addItem(icon, page, position);
         });
 
         this._viewIsReady = true;
@@ -725,6 +730,27 @@ class AppDisplay extends BaseAppView {
         return this._appInfoList;
     }
 
+    _getItemPosition(item) {
+        return this._pageManager.getAppPosition(item.id);
+    }
+
+    _compareItems(a, b) {
+        const [aPage, aPosition] = this._pageManager.getAppPosition(a.id);
+        const [bPage, bPosition] = this._pageManager.getAppPosition(b.id);
+
+        if (aPage === -1 && bPage === -1)
+            return a.name.localeCompare(b.name);
+        else if (aPage === -1)
+            return 1;
+        else if (bPage === -1)
+            return -1;
+
+        if (aPage !== bPage)
+            return aPage - bPage;
+
+        return aPosition - bPosition;
+    }
+
     _loadApps() {
         let appIcons = [];
         this._appInfoList = Shell.AppSystem.get_default().get_installed().filter(appInfo => {