From 6e98d71c3cb6c32dd340003c7d54f9987a28291f Mon Sep 17 00:00:00 2001
From: Stefan Zermatten <stefanzermatten@gmail.com>
Date: Thu, 15 Oct 2020 13:00:29 +0200
Subject: [PATCH] Improved slot UI look and feel

---
 .../ui/creature/character/CharacterSheet.vue  |  2 +-
 .../characterSheetTabs/CharacterTab.vue       |  2 +-
 .../ui/creature/slots/SlotFillDialog.vue      | 47 ++++++++++++-------
 app/imports/ui/creature/slots/Slots.vue       |  4 ++
 4 files changed, 36 insertions(+), 19 deletions(-)

diff --git a/app/imports/ui/creature/character/CharacterSheet.vue b/app/imports/ui/creature/character/CharacterSheet.vue
index abe09eea..932794c2 100644
--- a/app/imports/ui/creature/character/CharacterSheet.vue
+++ b/app/imports/ui/creature/character/CharacterSheet.vue
@@ -50,7 +50,7 @@
             <spells-tab :creature-id="creatureId" />
           </v-tab-item>
           <v-tab-item>
-            <persona-tab :creature-id="creatureId" />
+            <character-tab :creature-id="creatureId" />
           </v-tab-item>
           <v-tab-item>
             <tree-tab :creature-id="creatureId" />
diff --git a/app/imports/ui/creature/character/characterSheetTabs/CharacterTab.vue b/app/imports/ui/creature/character/characterSheetTabs/CharacterTab.vue
index 9ac25308..1f4f8a62 100644
--- a/app/imports/ui/creature/character/characterSheetTabs/CharacterTab.vue
+++ b/app/imports/ui/creature/character/characterSheetTabs/CharacterTab.vue
@@ -22,7 +22,7 @@
       </div>
       <div>
         <v-card>
-          <v-card-text>
+          <v-card-text style="background-color: inherit;">
             <slots :creature-id="creatureId" />
           </v-card-text>
         </v-card>
diff --git a/app/imports/ui/creature/slots/SlotFillDialog.vue b/app/imports/ui/creature/slots/SlotFillDialog.vue
index 31ca317c..43f9ac60 100644
--- a/app/imports/ui/creature/slots/SlotFillDialog.vue
+++ b/app/imports/ui/creature/slots/SlotFillDialog.vue
@@ -6,25 +6,38 @@
       </v-toolbar-title>
     </template>
     <div class="library-nodes">
-      <v-list>
-        <v-list-tile
-          v-for="node in libraryNodes"
-          :key="node._id"
-          :class="node._id === (selectedNode && selectedNode._id) && 'primary--text'"
-          @click="selectedNode = node"
+      <v-fade-transition mode="out-in">
+        <v-list v-if="$subReady.slotFillers">
+          <v-list-tile
+            v-for="node in libraryNodes"
+            :key="node._id"
+            :class="node._id === (selectedNode && selectedNode._id) && 'primary--text'"
+            @click="selectedNode = node"
+          >
+            <tree-node-view
+              :model="node"
+              :selected="node._id === (selectedNode && selectedNode._id)"
+            />
+          </v-list-tile>
+        </v-list>
+        <h4
+          v-else-if="$subReady.slotFillers"
+          class="ma-4"
         >
-          <tree-node-view
-            :model="node"
-            :selected="node._id === (selectedNode && selectedNode._id)"
+          Nothing suitable was found in your libraries
+        </h4>
+        <div
+          v-else
+          key="character-loading"
+          class="fill-height layout justify-center align-center"
+        >
+          <v-progress-circular
+            indeterminate
+            color="primary"
+            size="64"
           />
-        </v-list-tile>
-      </v-list>
-      <h4
-        v-if="!libraryNodes.length"
-        class="ma-4"
-      >
-        Nothing suitable was found in your libraries
-      </h4>
+        </div>
+      </v-fade-transition>
     </div>
     <template slot="actions">
       <v-spacer />
diff --git a/app/imports/ui/creature/slots/Slots.vue b/app/imports/ui/creature/slots/Slots.vue
index ec97bcee..b908efd5 100644
--- a/app/imports/ui/creature/slots/Slots.vue
+++ b/app/imports/ui/creature/slots/Slots.vue
@@ -36,6 +36,7 @@
         v-if="slot.quantityExpected > slot.children.length"
         icon
         :data-id="`slot-add-button-${slot._id}`"
+        style="background-color: inherit;"
         @click="fillSlot(slot._id)"
       >
         <v-icon>add</v-icon>
@@ -112,4 +113,7 @@ export default {
 </script>
 
 <style lang="css" scoped>
+  div {
+    background-color: inherit;
+  }
 </style>