[Fix] Random Fixes (#405)

* Fixed so minimum damage taken is 1, 0 was in playtest rules. Fixed so useDowntime isn't triggered on clicking enter

* .

* .

module/applications/sheets/actors/character.mjs

@@ -517,9 +517,10 @@ export default class CharacterSheet extends DHBaseActorSheet {
         const item = getDocFromElement(event.currentTarget);
         if (!item) return;
 
-        const max = item.system.resource.max ? itemAbleRollParse(item.system.resource.max, this.document, item) : null;
+        const max = event.currentTarget.max ? Number(event.currentTarget.max) : null;
         const value = max ? Math.min(Number(event.currentTarget.value), max) : event.currentTarget.value;
         await item.update({ 'system.resource.value': value });
+        this.render();
     }
 
     async updateItemQuantity(event) {
@@ -527,6 +528,7 @@ export default class CharacterSheet extends DHBaseActorSheet {
         if (!item) return;
 
         await item.update({ 'system.quantity': event.currentTarget.value });
+        this.render();
     }
 
     async updateArmorMarks(event) {

module/data/fields/actorField.mjs

@@ -8,7 +8,7 @@ const attributeField = label =>
 
 const resourceField = (max = 0, label, reverse = false) =>
     new fields.SchemaField({
-        value: new fields.NumberField({ initial: 0, integer: true, label }),
+        value: new fields.NumberField({ initial: 0, min: 0, integer: true, label }),
         max: new fields.NumberField({ initial: max, integer: true }),
         isReversed: new fields.BooleanField({ initial: reverse })
     });

module/documents/actor.mjs

@@ -568,13 +568,7 @@ export default class DhpActor extends Actor {
     }
 
     convertDamageToThreshold(damage) {
-        return damage >= this.system.damageThresholds.severe
+        return damage >= this.system.damageThresholds.severe ? 3 : damage >= this.system.damageThresholds.major ? 2 : 1;
-            ? 3
-            : damage >= this.system.damageThresholds.major
-              ? 2
-              : damage >= this.system.damageThresholds.minor
-                ? 1
-                : 0;
     }
 
     convertStressDamageToHP(resources) {

module/helpers/handlebarsHelper.mjs

@@ -48,8 +48,8 @@ export default class RegisterHandlebarsHelpers {
 
     static rollParsed(value, actor, item, numerical) {
         const isNumerical = typeof numerical === 'boolean' ? numerical : false;
-        const result = itemAbleRollParse(value, actor, item);
+        const result = itemAbleRollParse(value, actor.getRollData(), item);
-        return isNumerical && !result ? 0 : result;
+        return isNumerical ? (!result ? 0 : Number(result)) : result;
     }
 
     static setVar(name, value, context) {

styles/less/dialog/damage-reduction/damage-reduction-container.less

@@ -46,10 +46,6 @@
                 display: flex;
                 gap: 2px;
 
-                &:not(:last-child) {
-                    margin-right: 8px;
-                }
-
                 .mark-container {
                     cursor: pointer;
                     border: 1px solid light-dark(@dark-blue, @golden);
@@ -62,6 +58,10 @@
                     justify-content: center;
                     opacity: 0.4;
 
+                    &:not(:last-child) {
+                        margin-right: 8px;
+                    }
+
                     &.selected {
                         opacity: 1;
                     }

templates/sheets/actors/character/header.hbs

@@ -118,10 +118,10 @@
 
     {{#> 'systems/daggerheart/templates/sheets/global/tabs/tab-navigation.hbs'}}
         <div class="character-downtime-container">
-            <button data-action="useDowntime" data-type="shortRest" data-tooltip="{{localize "DAGGERHEART.APPLICATIONS.Downtime.shortRest.title"}}">
+            <button type="button" data-action="useDowntime" data-type="shortRest" data-tooltip="{{localize "DAGGERHEART.APPLICATIONS.Downtime.shortRest.title"}}">
                 <i class="fa-solid fa-chair"></i>
             </button>
-            <button data-action="useDowntime" data-type="longRest" data-tooltip="{{localize "DAGGERHEART.APPLICATIONS.Downtime.longRest.title"}}">
+            <button type="button" data-action="useDowntime" data-type="longRest" data-tooltip="{{localize "DAGGERHEART.APPLICATIONS.Downtime.longRest.title"}}">
                 <i class="fa-solid fa-bed"></i>
             </button>
         </div>

templates/sheets/actors/character/sidebar.hbs

@@ -4,20 +4,19 @@
         <a class="death-roll-btn" data-tooltip="DAGGERHEART.UI.Tooltip.makeDeathMove" {{#if
             isDeath}}data-action="makeDeathMove" {{/if}}><i class="fas fa-skull death-save"></i></a>
     </div>
-
     <div class="info-section">
         <div class="resources-section">
             <div class="status-bar">
                 <div class='status-value'>
-                    <p><input class="bar-input" name="system.resources.hitPoints.value"
+                    <p><input class="bar-input" name="system.resources.hitPoints.value" min="0" max='{{document.system.resources.hitPoints.max}}'
                             value="{{document.system.resources.hitPoints.value}}" type="number"></p>
                     <p>/</p>
                     <p class="bar-label">{{document.system.resources.hitPoints.max}}</p>
                 </div>
                 <progress
                     class='progress-bar'
-                    value='{{document.system.resources.hitPoints.value}}'
                     max='{{document.system.resources.hitPoints.max}}'
+                    value='{{document.system.resources.hitPoints.value}}'
                 ></progress>
                 <div class="status-label">
                     <h4>{{localize "DAGGERHEART.GENERAL.HitPoints.short"}}</h4>
@@ -26,7 +25,7 @@
             </div>
             <div class="status-bar">
                 <div class='status-value'>
-                    <p><input class="bar-input" name="system.resources.stress.value"
+                    <p><input class="bar-input" name="system.resources.stress.value" min="0" max='{{document.system.resources.stress.max}}'
                             value="{{document.system.resources.stress.value}}" type="number"></p>
                     <p>/</p>
                     <p class="bar-label">{{document.system.resources.stress.max}}</p>
@@ -34,6 +33,7 @@
                 <progress
                     class='progress-bar stress-color'
                     value='{{document.system.resources.stress.value}}'
+                    min="0"
                     max='{{document.system.resources.stress.max}}'
                 ></progress>
                 <div class="status-label">

templates/sheets/global/partials/inventory-item-V2.hbs

@@ -186,7 +186,7 @@ Parameters:
     {{/if}}
     {{#if (and (not hideResources) item.system.quantity)}}
     <div class="item-resource">
-      <input type="number" class="inventory-item-quantity" value="{{item.system.quantity}}" step="1" />
+      <input type="number" class="inventory-item-quantity" value="{{item.system.quantity}}" min="0" />
     </div>
     {{/if}}
 

templates/sheets/global/partials/item-resource.hbs

@@ -1,7 +1,7 @@
 {{#if (eq item.system.resource.type 'simple')}}
     <div class="item-resource">
         <i class="{{#if item.system.resource.icon}}{{item.system.resource.icon}}{{else}}fa-solid fa-hashtag{{/if}}"></i>
-        <input type="number" class="inventory-item-resource" value="{{item.system.resource.value}}" step="1" />
+        <input type="number" class="inventory-item-resource" value="{{item.system.resource.value}}" min="0" max="{{rollParsed item.system.resource.max item.actor item true}}" />
     </div>
 {{else}}
     <div class="item-resources">