From ef26153bb20a3c91082d68a7cba5de629ebb9444 Mon Sep 17 00:00:00 2001
From: Stefan Zermatten <stefanzermatten@gmail.com>
Date: Thu, 1 Dec 2022 13:28:33 +0200
Subject: [PATCH] Improved [redacted], added routes and navigation

---
 .../character/CharacterSheetToolbar.vue       |  56 ++++++---
 .../CharacterSheetPrinted.vue                 |  42 ++++++-
 .../CharacterSheetPrintedToolbar.vue          |  97 +++++++++++++++
 .../PrintedInventory.vue                      |  23 +---
 .../printedCharacterSheet/PrintedStats.vue    | 112 +++++++++---------
 .../components/PrintedContainer.vue           |   2 +-
 app/imports/client/ui/router.js               |   2 +
 7 files changed, 236 insertions(+), 98 deletions(-)
 create mode 100644 app/imports/client/ui/creature/character/printedCharacterSheet/CharacterSheetPrintedToolbar.vue

diff --git a/app/imports/client/ui/creature/character/CharacterSheetToolbar.vue b/app/imports/client/ui/creature/character/CharacterSheetToolbar.vue
index be85d916..6878df30 100644
--- a/app/imports/client/ui/creature/character/CharacterSheetToolbar.vue
+++ b/app/imports/client/ui/creature/character/CharacterSheetToolbar.vue
@@ -39,27 +39,45 @@
                 <v-icon>mdi-dots-vertical</v-icon>
               </v-btn>
             </template>
-            <v-list v-if="editPermission">
-              <v-list-item @click="deleteCharacter">
+            <v-list>
+              <v-list-item :to="printUrl">
                 <v-list-item-title>
-                  <v-icon>mdi-delete</v-icon> Delete
+                  <v-icon left>
+                    mdi-printer
+                  </v-icon> Print
                 </v-list-item-title>
               </v-list-item>
-              <v-list-item @click="showCharacterForm">
+              <template v-if="editPermission">
+                <v-list-item @click="deleteCharacter">
+                  <v-list-item-title>
+                    <v-icon left>
+                      mdi-delete
+                    </v-icon> Delete
+                  </v-list-item-title>
+                </v-list-item>
+                <v-list-item @click="showCharacterForm">
+                  <v-list-item-title>
+                    <v-icon left>
+                      mdi-pencil
+                    </v-icon> Edit details
+                  </v-list-item-title>
+                </v-list-item>
+                <v-list-item @click="showShareDialog">
+                  <v-list-item-title>
+                    <v-icon left>
+                      mdi-share-variant
+                    </v-icon> Sharing
+                  </v-list-item-title>
+                </v-list-item>
+              </template>
+              <v-list-item
+                v-else
+                @click="unshareWithMe"
+              >
                 <v-list-item-title>
-                  <v-icon>mdi-pencil</v-icon> Edit details
-                </v-list-item-title>
-              </v-list-item>
-              <v-list-item @click="showShareDialog">
-                <v-list-item-title>
-                  <v-icon>mdi-share-variant</v-icon> Sharing
-                </v-list-item-title>
-              </v-list-item>
-            </v-list>
-            <v-list v-else>
-              <v-list-item @click="unshareWithMe">
-                <v-list-item-title>
-                  <v-icon>mdi-delete</v-icon> Unshare with me
+                  <v-icon left>
+                    mdi-delete
+                  </v-icon> Unshare with me
                 </v-list-item-title>
               </v-list-item>
             </v-list>
@@ -144,6 +162,7 @@ import isDarkColor from '/imports/client/ui/utility/isDarkColor.js';
 import CharacterSheetFab from '/imports/client/ui/creature/character/CharacterSheetFab.vue';
 import getThemeColor from '/imports/client/ui/utility/getThemeColor.js';
 import SharedIcon from '/imports/client/ui/components/SharedIcon.vue';
+import getCreatureUrlName from '/imports/api/creature/creatures/getCreatureUrlName.js';
 
 export default {
   components: {
@@ -167,6 +186,9 @@ export default {
     isDark() {
       return isDarkColor(this.toolbarColor);
     },
+    printUrl() {
+      return `/print-character/${this.creature._id}/${getCreatureUrlName(this.creature)}`;
+    },
   },
   methods: {
     ...mapMutations([
diff --git a/app/imports/client/ui/creature/character/printedCharacterSheet/CharacterSheetPrinted.vue b/app/imports/client/ui/creature/character/printedCharacterSheet/CharacterSheetPrinted.vue
index 9fe2bec8..f6d2d522 100644
--- a/app/imports/client/ui/creature/character/printedCharacterSheet/CharacterSheetPrinted.vue
+++ b/app/imports/client/ui/creature/character/printedCharacterSheet/CharacterSheetPrinted.vue
@@ -42,7 +42,7 @@
                 {{ creature.alignment }} {{ background }}
               </div>
               <dir v-if="race || creature.gender">
-                {{ race }} {{ creature.gender }}
+                {{ creature.gender }} {{ race }}
               </dir>
               <div v-if="level && classes && classes.length === 1">
                 Level {{ level }} {{ classes[0].name }}
@@ -236,9 +236,41 @@ export default {
   color: black;
   font-size: 11pt;
 }
+
+.character-sheet-printed * {
+  print-color-adjust: exact;
+  -webkit-print-color-adjust: exact;
+  cursor: unset !important;
+}
+
 .page {
   padding: 4px;
 }
+
+.character-sheet-printed .column-layout, .character-sheet-printed .column-layout.wide-columns {
+  position:relative;
+  width: 100%;
+  widows: 0;
+  orphans: 0;
+  -webkit-column-fill: balance-all;
+  column-fill: balance-all;
+}
+
+.character-sheet-printed .column-layout>div {
+  position:relative;
+  display: block;
+  width: 100%;
+  backface-visibility: hidden;
+  -webkit-backface-visibility: hidden;
+  -webkit-column-break-inside: avoid;
+  break-inside: avoid;
+  page-break-inside: avoid;
+}
+.character-sheet-printed .column-layout>div>* {
+  -webkit-column-break-inside: avoid;
+  break-inside: avoid;
+  page-break-inside: avoid;
+}
 .character-sheet-printed .inactive {
   opacity: 1 !important;
 }
@@ -253,8 +285,6 @@ export default {
   background-image: url(/crown-dice-logo-cropped-transparent.png);
   background-size: contain;
   background-position: 0 center;
-  print-color-adjust: exact;
-  -webkit-print-color-adjust: exact;
 }
 
 .character-sheet-printed .v-divider {
@@ -265,7 +295,6 @@ export default {
 .character-sheet-printed .double-border {
   position: relative;
   padding: 11px 10px;
-  page-break-inside: avoid;
 }
 
 .character-sheet-printed .double-border::before {
@@ -286,8 +315,8 @@ export default {
 .character-sheet-printed .octagon-border {
   position: relative;
   padding: 4px 20px;
-  page-break-inside: avoid;
 }
+
 .character-sheet-printed .octagon-border::before {
   content: "";
   position: absolute;
@@ -326,6 +355,9 @@ export default {
   }
 }
 @media print {
+  .v-main, .v-application, .v-application--wrap, .character-sheet-printed {
+    display: block;
+  }
   header {
     display: none !important;
   }
diff --git a/app/imports/client/ui/creature/character/printedCharacterSheet/CharacterSheetPrintedToolbar.vue b/app/imports/client/ui/creature/character/printedCharacterSheet/CharacterSheetPrintedToolbar.vue
new file mode 100644
index 00000000..838080e5
--- /dev/null
+++ b/app/imports/client/ui/creature/character/printedCharacterSheet/CharacterSheetPrintedToolbar.vue
@@ -0,0 +1,97 @@
+<template>
+  <v-app-bar
+    app
+    class="character-sheet-printed-toolbar"
+    :color="toolbarColor"
+    :dark="isDark"
+    :light="!isDark"
+    clipped-right
+    :extended="$vuetify.breakpoint.smAndUp"
+    :tabs="$vuetify.breakpoint.smAndUp"
+    dense
+  >
+    <v-app-bar-nav-icon @click="toggleDrawer" />
+    <v-btn
+      icon
+      :to="characterUrl"
+    >
+      <v-icon>mdi-arrow-left</v-icon>
+    </v-btn>
+    <v-toolbar-title>
+      <v-fade-transition mode="out-in">
+        <div :key="$store.state.pageTitle">
+          {{ $store.state.pageTitle }}
+        </div>
+      </v-fade-transition>
+    </v-toolbar-title>
+    <v-spacer />
+    <div
+      slot="extension"
+      style="width: 100%"
+    >
+      <v-btn
+        class="print-fab"
+        color="accent"
+        elevation="4"
+        fab
+        @click="print"
+      >
+        <v-icon>mdi-printer</v-icon>
+      </v-btn>
+    </div>
+  </v-app-bar>
+</template>
+
+<script lang="js">
+import Creatures from '/imports/api/creature/creatures/Creatures.js';
+import { mapMutations } from 'vuex';
+import isDarkColor from '/imports/client/ui/utility/isDarkColor.js';
+import getThemeColor from '/imports/client/ui/utility/getThemeColor.js';
+import getCreatureUrlName from '/imports/api/creature/creatures/getCreatureUrlName.js';
+
+export default {
+  inject: {
+    context: { default: {} }
+  },
+  computed: {
+    creatureId() {
+      return this.$route.params.id;
+    },
+    toolbarColor() {
+      if (this.creature && this.creature.color) {
+        return this.creature.color;
+      } else {
+        return getThemeColor('secondary');
+      }
+    },
+    isDark() {
+      return isDarkColor(this.toolbarColor);
+    },
+    characterUrl() {
+      if (!this.creature) return;
+      return `/character/${this.creature._id}/${getCreatureUrlName(this.creature)}`;
+    },
+  },
+  methods: {
+    ...mapMutations([
+      'toggleDrawer',
+    ]),
+    print() {
+      print();
+    },
+  },
+  meteor: {
+    creature() {
+      return Creatures.findOne(this.creatureId);
+    },
+  },
+}
+</script>
+
+<style scoped>
+  .print-fab {
+    position: absolute;
+    bottom: -24px;
+    right: 24px;
+  }
+</style>
\ No newline at end of file
diff --git a/app/imports/client/ui/creature/character/printedCharacterSheet/PrintedInventory.vue b/app/imports/client/ui/creature/character/printedCharacterSheet/PrintedInventory.vue
index 1e87f3f4..21d8d65a 100644
--- a/app/imports/client/ui/creature/character/printedCharacterSheet/PrintedInventory.vue
+++ b/app/imports/client/ui/creature/character/printedCharacterSheet/PrintedInventory.vue
@@ -62,7 +62,7 @@
       >
         <div 
           :key="container._id"
-          class="span-all container-header"
+          class="span-all"
         >
           <printed-container
             class="octagon-border"
@@ -232,22 +232,6 @@ export default {
 </script>
 
 <style lang="css" scoped>
-.octagon-border {
-  position: relative;
-  padding: 4px 20px;
-  page-break-inside: avoid;
-}
-.octagon-border::before {
-  content: "";
-  position: absolute;
-  top: 0;
-  bottom: 0;
-  left: 0;
-  right: 0;
-  border-image: url(/images/print/octagonBorder.png) 124 118 fill;
-  border-image-width: 22px;
-  z-index: -1;
-}
 
 .label {
   font-size: 14pt;
@@ -262,9 +246,4 @@ export default {
 .inventory-stat > .v-icon {
   margin-right: 8px;
 }
-
-.container-header {
-  page-break-after: avoid;
-  page-break-inside: avoid;
-}
 </style>
diff --git a/app/imports/client/ui/creature/character/printedCharacterSheet/PrintedStats.vue b/app/imports/client/ui/creature/character/printedCharacterSheet/PrintedStats.vue
index c307f7ea..b8325777 100644
--- a/app/imports/client/ui/creature/character/printedCharacterSheet/PrintedStats.vue
+++ b/app/imports/client/ui/creature/character/printedCharacterSheet/PrintedStats.vue
@@ -2,97 +2,104 @@
   <div class="stats">
     <column-layout>
       <div
-        v-if="abilities.length"
-        class="ability-scores"
+        v-for="ability in abilities"
+        :key="ability._id"
       >
-        <div class="layout flex column">
-          <div
-            v-for="ability in abilities"
-            :key="ability._id"
-            class="ability"
-          >
-            <div class="score">
-              <div class="double-border top big-number">
-                <template v-if="creature.settings.swapScoresAndMods">
-                  {{ ability.total }}
-                </template>
-                <template v-else>
-                  {{ numberToSignedString(ability.modifier) }}
-                </template>
-              </div>
-              <div class="bottom">
-                <template v-if="creature.settings.swapScoresAndMods">
-                  {{ numberToSignedString(ability.modifier) }}
-                </template>
-                <template v-else>
-                  {{ ability.total }}
-                </template>
-              </div>
+        <div
+          class="ability"
+        >
+          <div class="score">
+            <div class="double-border top big-number">
+              <template v-if="creature.settings.swapScoresAndMods">
+                {{ ability.total }}
+              </template>
+              <template v-else>
+                {{ numberToSignedString(ability.modifier) }}
+              </template>
             </div>
-            <div class="double-border name label">
-              {{ ability.name }}
+            <div class="bottom">
+              <template v-if="creature.settings.swapScoresAndMods">
+                {{ numberToSignedString(ability.modifier) }}
+              </template>
+              <template v-else>
+                {{ ability.total }}
+              </template>
             </div>
           </div>
+          <div class="double-border name label">
+            {{ ability.name }}
+          </div>
         </div>
       </div>
 
       <div
         v-for="toggle in toggles"
         :key="toggle._id"
-        class="number-label"
       >
-        <div class="box double-border" />
-        <div class="label double-border">
-          {{ toggle.name }}
+        <div
+          class="number-label"
+        >
+          <div class="box double-border" />
+          <div class="label double-border">
+            {{ toggle.name }}
+          </div>
         </div>
       </div>
 
       <div
         v-for="stat in stats"
         :key="stat._id"
-        class="number-label"
-        :class="stat.variableName == 'armor' && 'shield-number-label'"
       >
         <div
-          :class="stat.variableName == 'armor' ? 'shield-border' : 'octagon-border'"
-          class="number big-number"
+          class="number-label"
+          :class="stat.variableName == 'armor' && 'shield-number-label'"
         >
-          {{ stat.value }}
-        </div>
-        <div class="label double-border">
-          {{ stat.name }}
+          <div
+            :class="stat.variableName == 'armor' ? 'shield-border' : 'octagon-border'"
+            class="number big-number"
+          >
+            {{ stat.value }}
+          </div>
+          <div class="label double-border">
+            {{ stat.name }}
+          </div>
         </div>
       </div>
 
       <div
         v-for="modifier in modifiers"
         :key="modifier._id"
-        class="number-label"
       >
-        <div class="number octagon-border big-number">
-          {{ numberToSignedString(modifier.value) }}
-        </div>
-        <div class="label double-border">
-          {{ modifier.name }}
+        <div
+          class="number-label"
+        >
+          <div class="number octagon-border big-number">
+            {{ numberToSignedString(modifier.value) }}
+          </div>
+          <div class="label double-border">
+            {{ modifier.name }}
+          </div>
         </div>
       </div>
 
       <div
         v-for="check in checks"
         :key="check._id"
-        class="number-label"
       >
-        <div class="number octagon-border big-number">
-          {{ numberToSignedString(check.value) }}
-        </div>
-        <div class="label double-border">
-          {{ check.name }}
+        <div
+          class="number-label"
+        >
+          <div class="number octagon-border big-number">
+            {{ numberToSignedString(check.value) }}
+          </div>
+          <div class="label double-border">
+            {{ check.name }}
+          </div>
         </div>
       </div>
       <div
         v-for="healthBar in healthBars"
         :key="healthBar._id"
-        class="m-2"
       >
         <div class="double-border">
           <div class="label">
@@ -115,7 +122,6 @@
       </div>
       <div
         v-if="hitDice.length"
-        class="hit-dice m-2"
       >
         <div class="double-border">
           <div>
diff --git a/app/imports/client/ui/creature/character/printedCharacterSheet/components/PrintedContainer.vue b/app/imports/client/ui/creature/character/printedCharacterSheet/components/PrintedContainer.vue
index 2bb4adb9..40a0ce97 100644
--- a/app/imports/client/ui/creature/character/printedCharacterSheet/components/PrintedContainer.vue
+++ b/app/imports/client/ui/creature/character/printedCharacterSheet/components/PrintedContainer.vue
@@ -1,5 +1,5 @@
 <template>
-  <div class="container">
+  <div class="inventory-container">
     <div class="d-flex justify-center">
       <property-icon
         class="ml-2"
diff --git a/app/imports/client/ui/router.js b/app/imports/client/ui/router.js
index 749ab231..b7e473be 100644
--- a/app/imports/client/ui/router.js
+++ b/app/imports/client/ui/router.js
@@ -13,6 +13,7 @@ const CharacterSheetPage = () => import('/imports/client/ui/pages/CharacterSheet
 const CharacterSheetToolbar = () => import('/imports/client/ui/creature/character/CharacterSheetToolbar.vue');
 const CharacterSheetRightDrawer = () => import('/imports/client/ui/creature/character/CharacterSheetRightDrawer.vue');
 const CharacterSheetPrinted = () => import('/imports/client/ui/creature/character/printedCharacterSheet/CharacterSheetPrinted.vue');
+const CharacterSheetPrintedToolbar = () => import('/imports/client/ui/creature/character/printedCharacterSheet/CharacterSheetPrintedToolbar.vue');
 const SignIn = () => import('/imports/client/ui/pages/SignIn.vue');
 const Register = () => import('/imports/client/ui/pages/Register.vue');
 const IconAdmin = () => import('/imports/client/ui/icons/IconAdmin.vue');
@@ -185,6 +186,7 @@ RouterFactory.configure(router => {
     alias: '/print-character/:id/:urlName',
     components: {
       default: CharacterSheetPrinted,
+      toolbar: CharacterSheetPrintedToolbar,
     },
     meta: {
       title: 'Print Character Sheet',