Add Reroll and Swap visualisation in chatlog

system/scripts/dice/dietype/l5r-base-die.js

@@ -91,6 +91,9 @@ export class L5rBaseDie extends DiceTerm {
             ["success", "explosive", "opportunity", "strife"].forEach((props) => {
                 this.l5r5e[props] += parseInt(face[props]);
             });
+            if (face.explosive) {
+                term.exploded = true;
+            }
         });
     }
 

system/scripts/dice/roll-n-keep-dialog.js

@@ -507,12 +507,23 @@ export class RollnKeepDialog extends FormApplication {
      */
     async _rebuildRoll() {
         // Get all kept dices + new (choice null)
-        const diceList = this.object.dicesList.reduce((acc, step) => {
+        const diceList = this.object.dicesList.reduce((acc, step, stepIdx) => {
-            step.forEach((die) => {
+            step.forEach((die, idx) => {
                 if (!!die && [RollnKeepDialog.CHOICES.keep, RollnKeepDialog.CHOICES.nothing].includes(die.choice)) {
                     if (!acc[die.type]) {
                         acc[die.type] = [];
                     }
+                    // Check previous dice, to add html classes in chat
+                    if (stepIdx > 0 && this.object.dicesList[stepIdx - 1][idx]) {
+                        switch (this.object.dicesList[stepIdx - 1][idx].choice) {
+                            case RollnKeepDialog.CHOICES.reroll:
+                                die.class = "rerolled";
+                                break;
+                            case RollnKeepDialog.CHOICES.swap:
+                                die.class = "swapped";
+                                break;
+                        }
+                    }
                     acc[die.type].push(die);
                 }
             });
@@ -535,6 +546,11 @@ export class RollnKeepDialog extends FormApplication {
                 term.results.map((res) => {
                     const die = diceList[term.constructor.name].shift();
                     res.result = die.face;
+
+                    // add class to term result
+                    if (die.class) {
+                        res[die.class] = true;
+                    }
                     return res;
                 });
                 term.l5rSummary();

system/scripts/dice/roll.js

@@ -104,11 +104,14 @@ export class RollL5r5e extends Roll {
         ); // ignore math symbols
         this.l5r5e.dicesTypes.l5r = this.dice.some((term) => term instanceof game.l5r5e.L5rBaseDie);
         summary.totalBonus = Math.max(0, summary.totalSuccess - this.l5r5e.difficulty);
+
+        if (!this.l5r5e.keepLimit) {
             this.l5r5e.keepLimit = this.dice.reduce(
                 (acc, term) => (term instanceof game.l5r5e.RingDie ? acc + term.number : acc),
                 0
             );
         }
+    }
 
     /**
      * Summarise the total of success, strife... for L5R dices for the current term
@@ -183,8 +186,9 @@ export class RollL5r5e extends Roll {
                         classes: [
                             cls.name.toLowerCase(),
                             "d" + term.faces,
-                            !isL5rDie && r.rerolled ? "rerolled" : null,
+                            isL5rDie && r.swapped ? "swapped" : null,
-                            !isL5rDie && r.exploded ? "exploded" : null,
+                            r.rerolled ? "rerolled" : null,
+                            r.exploded ? "exploded" : null,
                             !isL5rDie && r.discarded ? "discarded" : null,
                             !isL5rDie && r.result === 1 ? "min" : null,
                             !isL5rDie && r.result === term.faces ? "max" : null,
@@ -240,12 +244,20 @@ export class RollL5r5e extends Roll {
                 ? {}
                 : {
                       ...this.l5r5e,
-                      dices: this.dice.map((d) => {
+                      dices: this.dice.map((term) => {
+                          const isL5rDie = term instanceof game.l5r5e.L5rBaseDie;
                           return {
-                              diceTypeL5r: d instanceof game.l5r5e.L5rBaseDie,
+                              diceTypeL5r: isL5rDie,
-                              rolls: d.results.map((r) => {
+                              rolls: term.results.map((r) => {
                                   return {
-                                      result: d.constructor.getResultLabel(r.result),
+                                      result: term.constructor.getResultLabel(r.result),
+                                      classes: [
+                                          isL5rDie && r.swapped ? "swapped" : null,
+                                          r.rerolled ? "rerolled" : null,
+                                          r.exploded ? "exploded" : null,
+                                      ]
+                                          .filter((c) => !!c)
+                                          .join(" "),
                                   };
                               }),
                           };

system/styles/scss/dices.scss

@@ -9,7 +9,16 @@
 }
 
 // Dice in chat
-.chat-dice > img {
+.chat-dice {
+    &.rerolled > img {
+        border-bottom: 3px solid orangered;
+    }
+
+    &.swapped > img {
+        border-bottom: 3px solid fuchsia;
+    }
+
+    img {
         border: 1px solid transparent;
         background-repeat: no-repeat;
         background-position: center;
@@ -24,6 +33,7 @@
         flex: 0 0 20px;
         display: inline-block;
     }
+}
 
 .chat-profil {
     text-align: center;

system/templates/dice/chat-roll.html

@@ -49,7 +49,7 @@
             {{#each l5r5e.dices}}
                 {{#if this.diceTypeL5r}}
                     {{#each this.rolls}}
-                        <span class="l5r5e chat-dice">{{{this.result}}}</span>
+                        <span class="l5r5e chat-dice {{this.classes}}">{{{this.result}}}</span>
                     {{/each}}
                 {{/if}}
             {{/each}}