From 0f20cd4bd97d4a6f9b42a5fabd8795218d6fec91 Mon Sep 17 00:00:00 2001
From: Stefan Zermatten <stefanzermatten@gmail.com>
Date: Sun, 26 Jul 2020 19:39:50 +0200
Subject: [PATCH] Implemented drag and drop on spells page

---
 .../characterSheetTabs/SpellsTab.vue          |   5 +-
 app/imports/ui/layouts/Sidebar.vue            |   2 +-
 .../components/spells/SpellList.vue           | 119 ++++++++++++++----
 .../components/spells/SpellListCard.vue       |   5 +-
 .../components/spells/SpellListTile.vue       |   8 ++
 5 files changed, 113 insertions(+), 26 deletions(-)

diff --git a/app/imports/ui/creature/character/characterSheetTabs/SpellsTab.vue b/app/imports/ui/creature/character/characterSheetTabs/SpellsTab.vue
index 3851e4e0..19104bf0 100644
--- a/app/imports/ui/creature/character/characterSheetTabs/SpellsTab.vue
+++ b/app/imports/ui/creature/character/characterSheetTabs/SpellsTab.vue
@@ -3,7 +3,10 @@
     <column-layout wide-columns>
       <div v-if="spellsWithoutList.length">
         <v-card>
-          <spell-list :spells="spellsWithoutList" />
+          <spell-list
+            :spells="spellsWithoutList"
+            :parent-ref="{id: creatureId, collection: 'creatures'}"
+          />
         </v-card>
       </div>
       <div
diff --git a/app/imports/ui/layouts/Sidebar.vue b/app/imports/ui/layouts/Sidebar.vue
index 515a136d..1ccee871 100644
--- a/app/imports/ui/layouts/Sidebar.vue
+++ b/app/imports/ui/layouts/Sidebar.vue
@@ -135,7 +135,7 @@
           {title: 'Home', icon: 'home', to: '/'},
           {title: 'Characters', icon: 'portrait', to: '/characterList', requireLogin: true},
           {title: 'Library', icon: 'book', to: '/library', requireLogin: true},
-          {title: 'Tabletops', icon: 'api', to: '/tabletops', requireLogin: true},
+          //{title: 'Tabletops', icon: 'api', to: '/tabletops', requireLogin: true},
           //{title: 'Friends', icon: 'people', to: '/friends', requireLogin: true},
           {title: 'Feedback', icon: 'bug_report', to: '/feedback'},
           {title: 'About', icon: 'subject', to: '/about'},
diff --git a/app/imports/ui/properties/components/spells/SpellList.vue b/app/imports/ui/properties/components/spells/SpellList.vue
index 4d6d67af..f14372a4 100644
--- a/app/imports/ui/properties/components/spells/SpellList.vue
+++ b/app/imports/ui/properties/components/spells/SpellList.vue
@@ -4,26 +4,62 @@
     dense
     class="spell-list"
   >
-    <template v-for="level in levels">
-      <v-subheader :key="`${level}-header`">
-        {{ level === 0 ? 'Cantrips' : `Level ${level}` }}
-      </v-subheader>
-      <spell-list-tile
-        v-for="spell in spellsByLevel[level]"
-        :key="spell._id"
-        :data-id="`spell-list-tile-${spell._id}`"
-        :model="spell"
-        @click="clickProperty(spell._id)"
-      />
-    </template>
+    <draggable
+      v-model="computedSpells"
+      :group="`spell-list`"
+      ghost-class="ghost"
+      draggable=".item"
+      handle=".handle"
+      :animation="200"
+      @change="change"
+    >
+      <template v-for="spell in computedSpells">
+        <v-subheader
+          v-if="spell.isSubheader"
+          :key="`${spell.level}-header`"
+          class="item"
+        >
+          {{ spell.level === 0 ? 'Cantrips' : `Level ${spell.level}` }}
+        </v-subheader>
+        <spell-list-tile
+          v-else
+          :key="spell._id"
+          class="item"
+          :data-id="`spell-list-tile-${spell._id}`"
+          :model="spell"
+          @click="clickProperty(spell._id)"
+        />
+      </template>
+    </draggable>
   </v-list>
 </template>
 
 <script>
+import draggable from 'vuedraggable';
 import SpellListTile from '/imports/ui/properties/components/spells/SpellListTile.vue';
+import { organizeDoc } from '/imports/api/parenting/organizeMethods.js';
+
+function spellsWithSubheaders(spells = []){
+  let result = [];
+  let lastSpell = undefined;
+  let sortedSpells = [...spells].sort((a, b) => a.level - b.level)
+  sortedSpells.forEach(spell => {
+    if (spell.isSubheader) return;
+    if (!lastSpell || spell.level > lastSpell.level){
+      result.push({
+        isSubheader: true,
+        level: spell.level,
+      });
+    }
+    result.push(spell);
+    lastSpell = spell;
+  });
+  return result;
+}
 
 export default {
   components: {
+    draggable,
     SpellListTile,
   },
   props: {
@@ -31,24 +67,36 @@ export default {
       type: Array,
       default: () => [],
     },
+    parentRef: {
+      type: Object,
+      required: true,
+    },
   },
+  data(){ return {
+    dataSpells: [],
+  }},
   computed: {
     levels(){
       let levels = new Set();
       this.spells.forEach(spell => levels.add(spell.level));
       return levels;
     },
-    spellsByLevel(){
-      let spellsByLevel = {};
-      this.spells.forEach(spell => {
-        if (!spellsByLevel[spell.level]){
-          spellsByLevel[spell.level] = [spell];
-        } else {
-          spellsByLevel[spell.level].push(spell);
-        }
-      });
-      return spellsByLevel;
-    },
+    computedSpells: {
+      get(){
+        return spellsWithSubheaders(this.dataSpells);
+      },
+      set(value){
+        this.dataSpells = value;
+      },
+    }
+  },
+  watch: {
+    spells(value){
+      this.dataSpells = spellsWithSubheaders(value);
+    }
+  },
+  mounted(){
+    this.dataSpells = spellsWithSubheaders(this.spells);
   },
   methods: {
 		clickProperty(_id){
@@ -58,6 +106,31 @@ export default {
 				data: {_id},
 			});
 		},
+    change({added, moved}){
+      let event = added || moved;
+      if (event){
+        // If this spell is now adjacent to another, set the order accordingly
+        let order;
+        let before = this.dataSpells[event.newIndex - 1];
+        let after = this.dataSpells[event.newIndex + 1];
+        if (before && before._id){
+          order = before.order + 0.5;
+        } else if (after && after._id) {
+          order = after.order - 0.5;
+        } else {
+          order = -0.5;
+        }
+        let doc = event.element;
+        organizeDoc.call({
+          docRef: {
+            id: doc._id,
+            collection: 'creatureProperties',
+          },
+          parentRef: this.parentRef,
+          order,
+        });
+      }
+    },
 	}
 }
 </script>
diff --git a/app/imports/ui/properties/components/spells/SpellListCard.vue b/app/imports/ui/properties/components/spells/SpellListCard.vue
index 6f22f0d8..8c69990b 100644
--- a/app/imports/ui/properties/components/spells/SpellListCard.vue
+++ b/app/imports/ui/properties/components/spells/SpellListCard.vue
@@ -10,7 +10,10 @@
       </v-toolbar-title>
       <v-spacer />
     </template>
-    <spell-list :spells="spells" />
+    <spell-list
+      :spells="spells"
+      :parent-ref="{id: model._id, collection: 'creatureProperties'}"
+    />
   </toolbar-card>
 </template>
 
diff --git a/app/imports/ui/properties/components/spells/SpellListTile.vue b/app/imports/ui/properties/components/spells/SpellListTile.vue
index b512ea5a..6806e057 100644
--- a/app/imports/ui/properties/components/spells/SpellListTile.vue
+++ b/app/imports/ui/properties/components/spells/SpellListTile.vue
@@ -17,6 +17,14 @@
         {{ components }}
       </v-list-tile-sub-title>
     </v-list-tile-content>
+    <v-list-tile-action>
+      <v-icon
+        style="height: 100%; width: 40px; cursor: move;"
+        class="handle"
+      >
+        drag_indicator
+      </v-icon>
+    </v-list-tile-action>
   </v-list-tile>
 </template>