module/applications/dialogs/groupRollDialog.mjs

@@ -37,7 +37,7 @@ export default class GroupRollDialog extends HandlebarsApplicationMixin(Applicat
         tag: 'form',
         id: 'GroupRollDialog',
         classes: ['daggerheart', 'views', 'dh-style', 'dialog', 'group-roll-dialog'],
-        position: { width: 390, height: 'auto' },
+        position: { width: 380, height: 'auto' },
         window: {
             icon: 'fa-solid fa-users'
         },
@@ -47,7 +47,7 @@ export default class GroupRollDialog extends HandlebarsApplicationMixin(Applicat
             makeRoll: this.#makeRoll,
             removeRoll: this.#removeRoll,
             rerollDice: this.#rerollDice,
-            markSuccessful: this.#markSuccessful,
+            markSuccessfull: this.#markSuccessfull,
             cancelRoll: this.#onCancelRoll,
             finishRoll: this.#finishRoll
         },
@@ -204,8 +204,7 @@ export default class GroupRollDialog extends HandlebarsApplicationMixin(Applicat
             isEditable: actor?.testUserPermission(game.user, CONST.DOCUMENT_OWNERSHIP_LEVELS.OWNER),
             key: partId,
             readyToRoll: Boolean(data.rollChoice),
-            hasRolled: Boolean(data.rollData),
+            hasRolled: Boolean(data.rollData)
-            modifier: data.successfull ? 1 : data.successfull === false ? -1 : 0
         };
     }
 
@@ -407,13 +406,12 @@ export default class GroupRollDialog extends HandlebarsApplicationMixin(Applicat
         );
     }
 
-    static #markSuccessful(_event, button) {
+    static #markSuccessfull(_event, button) {
-        const memberKey = button.closest('[data-member-key]').dataset.memberKey;
+        const previousValue = this.party.system.groupRoll.aidingCharacters[button.dataset.member].successfull;
-        const previousValue = this.party.system.groupRoll.aidingCharacters[memberKey].successfull;
+        const newValue = Boolean(button.dataset.successfull === 'true');
-        const newValue = Boolean(button.dataset.success === 'true');
         this.updatePartyData(
             {
-                [`system.groupRoll.aidingCharacters.${memberKey}.successfull`]:
+                [`system.groupRoll.aidingCharacters.${button.dataset.member}.successfull`]:
                     previousValue === newValue ? null : newValue
             },
             this.getUpdatingParts(button)

styles/less/dialog/group-roll-dialog/main.less

@@ -19,31 +19,59 @@
         }
 
         .item-tags {
-            gap: 6px;
-            .tag.failure,
             .tag.success {
-                font-weight: 600;
-                justify-content: center;
-                min-width: 3ch;
-            }
-
-            .tag.success {
-                border-color: @green;
                 background: @green-10;
                 color: @green;
             }
 
             .tag.failure {
-                border-color: @red;
                 background: @red-10;
                 color: @red;
             }
-
-            .tag {
-                padding-top: 0;
-                padding-bottom: 0;
-                line-height: 1.1875rem;
         }
+
+        .member-roll-container {
+            display: flex;
+            align-items: center;
+            justify-content: space-between;
+            gap: 8px;
+            height: 3.375rem;
+
+            .name-area {
+                display: flex;
+                flex-direction: column;
+                flex: 1;
+                justify-content: center;
+                .name {
+                    font-weight: 500;
+                }
+                .trait {
+                    display: flex;
+                    align-items: center;
+                    gap: var(--spacer-8);
+                    select {
+                        --input-height: 2em;
+                        width: auto;
+                    }
+                }
+            }
+
+            &.inactive {
+                opacity: 0.3;
+                pointer-events: none;
+            }
+
+            a.roll-button.initial-roll {
+                width: 1.875rem;
+                height: 1.875rem;
+                margin-right: 2px; /** makes hover look a bit nicer */
+            }
+        }
+    }
+
+    .roll-container {
+        display: flex;
+        flex-direction: column;
     }
 
     .with-result {
@@ -73,61 +101,6 @@
         }
     }
 
-        .member-roll-container {
-            display: flex;
-            align-items: center;
-            justify-content: space-between;
-            gap: 8px;
-            height: 3.375rem;
-
-            &.inactive {
-                opacity: 0.3;
-                pointer-events: none;
-            }
-
-            .name-area {
-                display: flex;
-                flex-direction: column;
-                flex: 1;
-                justify-content: center;
-                .name {
-                    font-weight: 500;
-                }
-                .trait {
-                    display: flex;
-                    align-items: center;
-                    gap: 6px;
-                    select {
-                        --input-height: 2em;
-                        width: auto;
-                    }
-                }
-                .item-tags {
-                    align-items: stretch;
-                }
-            }
-
-            .buttons {
-                display: flex;
-                flex-direction: column;
-                button {
-                    --button-size: 1.5em;
-                    padding: 0 var(--spacer-4);
-                }
-            }
-
-            a.roll-button.initial-roll {
-                width: 1.875rem;
-                height: 1.875rem;
-                margin-right: 2px; /** makes hover look a bit nicer */
-            }
-
-            .with-result {
-                display: flex;
-                justify-content: end;
-                align-items: center;
-                gap: 6px;
-
     .roll-data {
         display: flex;
         flex-direction: column;
@@ -184,22 +157,6 @@
         }
     }
 
-                .buttons {
-                    gap: 2px;
-                    button {                    
-                        color: var(--medium-red);
-                        &[data-success=true] {
-                            color: var(--green);
-                        }
-                        &.inactive {
-                            opacity: 0.4;
-                        }
-                    }
-                }
-            }
-        }
-    }
-
     .group-roll-results {
         display: flex;
         flex-direction: column;
@@ -221,10 +178,6 @@
             margin-block: var(--spacer-4);
         }
 
-        .modifiers .item-tags {
-            min-height: calc(2px + 1.1875rem);
-        }
-
         .total {
             .label {
                 font-size: var(--font-size-14);

templates/dialogs/groupRollDialog/parts/member.hbs

@@ -1,17 +1,13 @@
 {{#with (ifThen (eq partId "leader") leader (lookup members partId))}}
     <section class="member-roll-container {{type}} {{#if @root.allHaveRolled}}select-padding{{/if}} {{#unless isEditable}}inactive{{/unless}}" data-member-key="{{@root.partId}}">
         {{log this}}
+        {{log @root}}
         <img class="portrait" src="{{img}}" />
         <div class="name-area">
             <span class="name">{{name}}</span>
             {{#if hasRolled}}
                 <div class="trait item-tags">
                     <div class="tag">{{rollChoiceLabel}}</div>
-                    {{#if modifier}}
-                        <span class="tag {{#if (gte modifier 0)}}success{{else}}failure{{/if}}">
-                            {{numberFormat modifier sign=true}}
-                        </span>
-                    {{/if}}
                 </div>
             {{else if readyToRoll}}
                 <div class="trait">
@@ -23,8 +19,7 @@
             {{/if}}
         </div>
         {{#if roll}}
-            <div class="with-result {{#if roll.withHope}}hope{{else if roll.withFear}}fear{{else}}critical{{/if}}">
+            <div class="roll-data with-result {{#if roll.withHope}}hope{{else if roll.withFear}}fear{{else}}critical{{/if}}">
-                <div class="roll-data">
                 <div class="duality-label">
                     {{roll.total}} 
                     <span class="with">{{localize "DAGGERHEART.GENERAL.withThing" thing=roll.totalLabel}}</span>
@@ -45,17 +40,10 @@
                             <img src="{{concat "systems/daggerheart/assets/icons/dice/" (ifThen (eq roll.advantage.type 1) "adv/" "disadv/") roll.advantage.dice ".svg"}}" />
                         </span>
                     {{/if}}
-                        <span class="roll-operator ">{{#if (gte roll.modifierTotal 0)}}+{{else}}-{{/if}}</span>
+                    <span class="roll-operator">{{#if (gte roll.modifierTotal 0)}}+{{else}}-{{/if}}</span>
                     <span class="roll-value">{{positive roll.modifierTotal}}</span>
                 </div>
             </div>
-                {{#if (and isEditable (ne ../partId "leader"))}}
-                    <div class="buttons">
-                        <button type="button" data-action="markSuccessful" data-success="true" class="plain icon fa-solid fa-check {{#if modifier}}{{#if successfull}}active{{else}}inactive{{/if}}{{/if}}"></button>
-                        <button type="button" data-action="markSuccessful" class="plain icon fa-solid fa-times {{#if modifier}}{{#if successfull}}inactive{{else}}active{{/if}}{{/if}}"></button>
-                    </div>
-                {{/if}}
-            </div>
         {{else if readyToRoll}}
             <a class="roll-button initial-roll" data-action="makeRoll" data-member="{{@root.partId}}">
                 <img class="roll-img duality" src="systems/daggerheart/assets/icons/dice/duality/DualityBW.svg" alt="2d12">

templates/dialogs/groupRollDialog/parts/result.hbs

@@ -12,9 +12,7 @@
             <span class="label">Modifiers</span>
             <div class="item-tags">
                 {{#each groupRoll.modifiers as |modifier|}}
-                    <span class="tag {{#if (gte modifier 0)}}success{{else}}failure{{/if}}">
+                    <span class="tag">{{#if (gte modifier 0)}}+{{else}}-{{/if}}{{positive modifier}}</span>
-                        {{numberFormat modifier sign=true}}
-                    </span>
                 {{/each}}
             </div>
         </div>