Add settings to auto expand roll message sections

lang/en.json

@@ -2107,7 +2107,11 @@
                     "extendCharacterDescriptions": { "label": "Characters" },
                     "extendAdversaryDescriptions": { "label": "Adversaries" },
                     "extendEnvironmentDescriptions": { "label": "Environments" },
-                    "extendItemDescriptions": { "label": "Items" }
+                    "extendItemDescriptions": { "label": "Items" },
+                    "expandRollMessage": "Auto-expand Roll Message",
+                    "expandRollMessageRoll": { "label": "Roll" },
+                    "expandRollMessageDamage": { "label": "Damage/Healing" },
+                    "expandRollMessageTarget": { "label": "Targets" }
                 },
                 "fearDisplay": {
                     "token": "Tokens",

module/data/settings/Appearance.mjs

@@ -71,6 +71,20 @@ export default class DhAppearance extends foundry.abstract.DataModel {
             extendItemDescriptions: new fields.BooleanField({
                 initial: false,
                 label: 'DAGGERHEART.SETTINGS.Appearance.FIELDS.extendItemDescriptions.label'
+            }),
+            expandRollMessage: new fields.SchemaField({
+                roll: new fields.BooleanField({
+                    initial: false,
+                    label: 'DAGGERHEART.SETTINGS.Appearance.FIELDS.expandRollMessageRoll.label'
+                }),
+                damage: new fields.BooleanField({
+                    initial: false,
+                    label: 'DAGGERHEART.SETTINGS.Appearance.FIELDS.expandRollMessageDamage.label'
+                }),
+                target: new fields.BooleanField({
+                    initial: false,
+                    label: 'DAGGERHEART.SETTINGS.Appearance.FIELDS.expandRollMessageTarget.label'
+                })
             })
         };
     }

module/documents/chatMessage.mjs

@@ -67,6 +67,19 @@ export default class DhpChatMessage extends foundry.documents.ChatMessage {
                     html.classList.add('critical');
                     break;
             }
+
+            const autoExpandRoll = game.settings.get(CONFIG.DH.id, CONFIG.DH.SETTINGS.gameSettings.appearance).expandRollMessage,
+                rollSections = html.querySelectorAll(".roll-part");
+            rollSections.forEach(s => {
+                if(s.classList.contains("roll-section")) {
+                    const toExpand = s.querySelector('[data-action="expandRoll"]');
+                    toExpand.classList.toggle("expanded", autoExpandRoll.roll);
+                } else if(s.classList.contains("damage-section"))
+                    s.classList.toggle("expanded", autoExpandRoll.damage);
+                else if(s.classList.contains("target-section"))
+                    s.classList.toggle("expanded", autoExpandRoll.target);
+            })
+
         }
         
         if(!game.user.isGM) {

templates/settings/appearance-settings.hbs

@@ -17,6 +17,13 @@
         {{formGroup settingFields.schema.fields.extendItemDescriptions value=settingFields._source.extendItemDescriptions localize=true}}
     </fieldset>
 
+    <fieldset>
+        <legend>{{localize 'DAGGERHEART.SETTINGS.Appearance.FIELDS.expandRollMessage'}}</legend>
+        {{formGroup settingFields.schema.fields.expandRollMessage.fields.roll value=settingFields.expandRollMessage.roll localize=true}}
+        {{formGroup settingFields.schema.fields.expandRollMessage.fields.damage value=settingFields.expandRollMessage.damage localize=true}}
+        {{formGroup settingFields.schema.fields.expandRollMessage.fields.target value=settingFields.expandRollMessage.target localize=true}}
+    </fieldset>
+
     {{#if showDiceSoNice}}
         <fieldset>
             <legend>{{localize "DAGGERHEART.SETTINGS.Menu.appearance.diceSoNice.title"}}</legend>

templates/ui/chat/parts/damage-part.hbs

@@ -1,4 +1,4 @@
-<div class="roll-part damage-section dice-roll" data-action="expandRoll">
+<div class="roll-part damage-section dice-roll" data-action="expandRoll"{{#if (empty damage)}} hidden{{/if}}>
     <div class="roll-part-header"><div><span>{{localize (ifThen hasHealing "DAGGERHEART.ACTIONS.TYPES.healing.name" "DAGGERHEART.ACTIONS.TYPES.damage.name")}}</span></div></div>
     <div class="roll-part-extra on-reduced">
         <div class="wrapper">
@@ -12,7 +12,7 @@
             <div class="wrapper">
                 {{#each damage as | roll index | }}
                     <fieldset>
-                        <legend>{{log @root}}
+                        <legend>
                             {{localize (concat 'DAGGERHEART.CONFIG.HealingType.' index '.inChatRoll')}} <div class="roll-formula">{{localize "DAGGERHEART.GENERAL.total"}}: {{roll.total}}</div>{{#if (and (eq index "hitPoints")../isDirect)}} <div class="roll-formula">{{localize "DAGGERHEART.CONFIG.DamageType.direct.short"}}</div>{{/if}}
                         </legend>
                         {{#each roll.parts}}

templates/ui/chat/roll.hbs

@@ -1,7 +1,7 @@
 <div class="chat-roll">
     <div class="roll-part-header"><span>{{title}}</span></div>
     {{#if hasRoll}}{{> 'systems/daggerheart/templates/ui/chat/parts/roll-part.hbs'}}{{/if}}
-    {{#if (and (not (empty damage)) (or hasDamage hasHealing))}}{{> 'systems/daggerheart/templates/ui/chat/parts/damage-part.hbs'}}{{/if}}
+    {{#if (or hasDamage hasHealing)}}{{> 'systems/daggerheart/templates/ui/chat/parts/damage-part.hbs'}}{{/if}}
     {{#if hasTarget}}{{> 'systems/daggerheart/templates/ui/chat/parts/target-part.hbs'}}{{/if}}
     <div class="roll-part-header"><div></div></div>
 </div>