Merge pull request #105 from Cussa/master

Small fixes (weapon icon, vagabond start equipement, talent selection on roll dialog, player sheet with a cut, chargen issue when re-roll, remove rolls from limited player sheet)

Author: DrOgres

Diff for: README.md

@@ -31,6 +31,14 @@ This system provides character sheets and items for  your play, if you would lik
 ## Release Notes
 =======
 
+v4.3.1
+- 🐛 Fix: get weapon icon when dragging to macro bar
+- 🐛 Fix: Vagabond start equipment should be "Quarterstaff" (fix #104)
+- 🐛 Fix: removes "None" from Talent selection
+- 💄🐛 UI/Fix: corrects the player character sheet cut on the bottom
+- 🐛 Fix: Chargen issue when re-roll character
+- 🐛 Fix: remove click/roll functionality from the limited character sheet
+
 v4.3.0
 - Improvement: any roll using the `/r` chat command now shows the result of the roll directly
   ![D66 Roll](asset/d66roll.png?raw=true)

Diff for: script/actor/vaesen-actor-sheet.js

@@ -121,6 +121,9 @@ export class VaesenActorSheet extends ActorSheet {
 
   activateListeners(html) {
     super.activateListeners(html);
+    if (this.actor.limited)
+      return;
+
     html.find(".item-create").click((ev) => {
       this.onItemCreate(ev);
     });

Diff for: script/sheet/player.js

@@ -117,6 +117,9 @@ export class PlayerCharacterSheet extends VaesenActorSheet {
   activateListeners(html) {
     super.activateListeners(html);
 
+    if (this.actor.limited)
+      return;
+
     html.find(".fav-togle").click((ev) => {
       this.onFavTogle(ev);
     });
@@ -227,7 +230,7 @@ export class PlayerCharacterSheet extends VaesenActorSheet {
       const div = $(item).parents(".weapon");
       const itemId = div.data("itemId");
       const itemName = $(item).text();
-      const img = $(div).children(".icon").attr("src");
+      const img = $(div).find(".icon").attr("src");
       const testName = `${itemName} ${game.i18n.localize("ROLL.ROLL")}`;
 
       const data = {

Diff for: script/util/generator.js

@@ -290,7 +290,7 @@ export class generator {
           label: game.i18n.localize("GENERATOR.REROLL"),
           callback: async () => {
             reroll = true;
-            await generator.generateCharacter();
+            await generator.generateCharacter(actor);
           }
         }
       },

Diff for: script/util/generator_data.js

@@ -781,7 +781,7 @@ export class generator_data {
         "Well-traveled"
       ],
       equipment: [
-        "Walking stick",
+        "Quarterstaff",
         "Knife or Dagger",
         "Crowbar",
         "Liquor",

Diff for: script/util/roll.js

@@ -319,7 +319,6 @@ function buildSelectMultipleHtmlDialog(options, name, id) {
   );
   html.push(`<div class="flex row" style="width: 100%;">`);
   html.push(`<select id="${id}" style="width: 100%;" multiple>`);
-  html.push(`<option value="0">None (0)</option>`);
   options.forEach((element) => {
     let descriptionWithoutTags = $("<p>").html(element.description).text();
     let requiresBonus =

Diff for: style/player.css

@@ -1,5 +1,5 @@
 .vaesen .player {
-    height: 100%;
+    /* height: 100%; */
     width: 100%;
 }
 

Diff for: system.json

@@ -1,7 +1,7 @@
 {
   "title": "Vaesen",
   "description": "Nordic horror role-playing",
-  "version": "4.3.0",
+  "version": "4.3.1",
   "esmodules": [
     "script/hooks.js"
   ],