From 2749a29c637aeef1723393bfafa8d2ad7cf197d6 Mon Sep 17 00:00:00 2001
From: Jonas Karlsson <jk@xdy.se>
Date: Sun, 16 Aug 2020 20:37:43 +0200
Subject: [PATCH] fix: Hardcode dex for skill rolls (really, start of support
 for choice of characteristic for skill rolls)

---
 src/module/handlebars.ts                 |  2 +-
 static/templates/actors/actor-sheet.html | 18 ++++++++++--------
 2 files changed, 11 insertions(+), 9 deletions(-)

diff --git a/src/module/handlebars.ts b/src/module/handlebars.ts
index 1264e0937..0fb456982 100644
--- a/src/module/handlebars.ts
+++ b/src/module/handlebars.ts
@@ -27,7 +27,7 @@ export default function registerHandlebarsHelpers():void {
   });
 
   Handlebars.registerHelper('shouldShowSkill', function(skill, hideUntrainedSkills) {
-    console.log('shouldShowSkill', skill, hideUntrainedSkills)
+    // console.log('shouldShowSkill', skill, hideUntrainedSkills)
     return skill.trained || hideUntrainedSkills
   });
 
diff --git a/static/templates/actors/actor-sheet.html b/static/templates/actors/actor-sheet.html
index 24d2cfa03..1f42c1193 100644
--- a/static/templates/actors/actor-sheet.html
+++ b/static/templates/actors/actor-sheet.html
@@ -108,13 +108,14 @@
 
               <label for="data.skills.{{key}}.value" class="flex3 resource-label skill-{{skill.trained}}">{{skill.label}}</label>
 
-              <span><select class="skill-stat" name="characteristic">
-                <option value="data.characteristics.strength.mod">{{data.strength.shortlabel}}</option>
-                <option value="data.characteristics.dexterity.mod">{{data.dexterity.shortlabel}}</option>
-                <option value="data.characteristics.endurance.mod">{{data.endurance.shortlabel}}</option>
-                <option value="data.characteristics.intellect.mod">{{data.intellect.shortlabel}}</option>
-                <option value="data.characteristics.education.mod">{{data.education.shortlabel}}</option>
-                <option value="data.characteristics.social.mod">{{data.social.shortlabel}}</option>
+              <span><select disabled class="skill-stat" name="characteristic">
+                <option value="data.characteristics.dexterity.mod">{{../data.characteristics.dexterity.shortlabel}}</option>
+                <option value="data.characteristics.strength.mod">{{../data.characteristics.strength.shortlabel}}</option>
+                <option value="data.characteristics.endurance.mod">{{../data.characteristics.endurance.shortlabel}}</option>
+                <option value="data.characteristics.intellect.mod">{{../data.characteristics.intellect.shortlabel}}</option>
+                <option value="data.characteristics.education.mod">{{../data.characteristics.education.shortlabel}}</option>
+                <option value="data.characteristics.social.mod">{{../data.characteristics.social.shortlabel}}</option>
+                <option value="data.characteristics.psionics.mod">{{../data.characteristics.psionics.shortlabel}}</option>
               </select></span>
 
               <span class="skill-arrows flex1">
@@ -127,9 +128,10 @@
               </span>
 
 
+              <!-- TODO Hardcoded dexterity for now, but it should get the value from the dropdown. Also moved dexterity to the top and disabled the select to make it seem less strange. :) -->
               <span class="skill-output" trainable="true" trained="{{skill.trained}}" class="flex1 skill-mod rollable"
                 data-label="{{skill.label}}">{{numberFormat (getSkillValueWithJoat skill @root.data.jackofalltrades) decimals=0 sign=true}}
-                <img class="rollable" data-roll="2d6+@skills.{{key}}.value+@characteristics.{{key}}.mod" src="./systems/twodsix/assets/d6-icon.svg"/>
+                <img class="rollable" data-roll="2d6+@skills.{{key}}.value+@characteristics.dexterity.mod" src="./systems/twodsix/assets/d6-icon.svg"/>
               </span>