[gnome-shell/gbsneto/icon-grid-dnd: 30/41] appDisplay: Add moveItem()

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

commit 6eb30438507907bc3945699ae689b1c456bfebc8
Author: Georges Basile Stavracas Neto <georges stavracas gmail com>
Date:   Tue Jul 2 17:41:15 2019 -0300

    appDisplay: Add moveItem()
    
    This is a handy function to implement changing an icon's position
    in the grid.
    
    WIP: better commit message
    
    https://gitlab.gnome.org/GNOME/gnome-shell/merge_requests/603

 js/ui/appDisplay.js | 18 ++++++++++++++++++
 js/ui/iconGrid.js   | 14 ++++++++++++++
 2 files changed, 32 insertions(+)
---
diff --git a/js/ui/appDisplay.js b/js/ui/appDisplay.js
index 17ed6c9b0..2a9a3bd18 100644
--- a/js/ui/appDisplay.js
+++ b/js/ui/appDisplay.js
@@ -160,6 +160,24 @@ class BaseAppView {
         this.emit('view-loaded');
     }
 
+    moveItem(item, newPosition) {
+        let itemIndex = this._allItems.indexOf(item);
+
+        if (itemIndex == -1) {
+            log('Trying to move item %s that is not in this app view'.format(item.id));
+            return;
+        }
+
+        if (newPosition > itemIndex)
+            newPosition -= 1;
+
+        // Remove from the old position
+        this._allItems.splice(itemIndex, 1);
+        this._allItems.splice(newPosition, 0, item);
+
+        this._grid.moveItem(item, newPosition);
+    }
+
     _selectAppInternal(id) {
         if (this._items[id])
             this._items[id].actor.navigate_focus(null, St.DirectionType.TAB_FORWARD, false);
diff --git a/js/ui/iconGrid.js b/js/ui/iconGrid.js
index 0f93338f9..60ef8b573 100644
--- a/js/ui/iconGrid.js
+++ b/js/ui/iconGrid.js
@@ -697,6 +697,20 @@ var IconGrid = GObject.registerClass({
             this.add_actor(item.actor);
     }
 
+    moveItem(item, newPosition) {
+        if (!this.contains(item.actor)) {
+            log('Cannot move item not contained by the IconGrid');
+            return;
+        }
+
+        let children = this.get_children();
+        let visibleChildren = children.filter(c => c.is_visible());
+        let visibleChildAtPosition = visibleChildren[newPosition];
+        let realPosition = children.indexOf(visibleChildAtPosition);
+
+        this.set_child_at_index(item.actor, realPosition);
+    }
+
     removeItem(item) {
         this.remove_child(item.actor);
     }