From f70ce34acc0e8971b3e23d69a1e985298a953be8 Mon Sep 17 00:00:00 2001
From: Aaron Lutomski <sobran@gmail.com>
Date: Sat, 16 May 2026 21:06:35 -0700
Subject: [PATCH] Fixes for roll chat cards and info links

---
 css/deltagreen.css                   | 27 ++++++++++++++
 module/roll/roll.js                  | 10 ++++--
 templates/actor/parts/about-tab.html | 53 ++++++++++++++--------------
 templates/roll/percentile-roll.hbs   | 28 ++++++++-------
 4 files changed, 77 insertions(+), 41 deletions(-)

diff --git a/css/deltagreen.css b/css/deltagreen.css
index 58791d7..6c88506 100644
--- a/css/deltagreen.css
+++ b/css/deltagreen.css
@@ -737,6 +737,33 @@ div .exhausted-section {
   margin-bottom: 5px;
 }
 
+.tab.about .dg-about-links {
+  gap: 0.5rem;
+  margin-bottom: 1rem;
+}
+
+.tab.about .dg-about-links a {
+  display: inline-flex;
+  align-items: center;
+  gap: 0.5rem;
+  width: fit-content;
+  max-width: 100%;
+}
+
+.tab.about .dg-about-links a i {
+  flex: 0 0 auto;
+}
+
+.tab.about .dg-about-links a span {
+  flex: 1 1 auto;
+  line-height: 1.4;
+  text-align: left;
+}
+
+.tab.about .dg-about-legal p {
+  margin: 0.5rem 0;
+}
+
 textarea.wounds {
   resize: none;
   height: 100px;
diff --git a/module/roll/roll.js b/module/roll/roll.js
index 80f7fbc..db002e4 100644
--- a/module/roll/roll.js
+++ b/module/roll/roll.js
@@ -594,10 +594,11 @@ export class DGLethalityRoll extends DGPercentileRoll {
     }
 
     let html = "";
-    html += `<div class="dice-roll">`;
+    html += `<div class="dice-roll" data-action="expandRoll">`;
     html += `     <div class="dice-result">`;
     html += `     <div style="${styleOverride}" class="dice-formula">${resultString}</div>`;
     html += `     <div class="dice-tooltip">`;
+    html += `          <div class="wrapper">`;
     html += `          <section class="tooltip-part">`;
     html += `               <div class="dice">`;
     html += `                    <header class="part-header flexrow">`;
@@ -626,10 +627,12 @@ export class DGLethalityRoll extends DGPercentileRoll {
     html += `                    </ol>`;
     html += `               </div>`;
     html += `          </section>`;
+    html += `          </div>`;
     html += `     </div>`;
     html += `     <h4 class="dice-total">${this.total} (${
       nonLethalDamage.total
     } ${game.i18n.localize("DG.Roll.Damage")})</h4>`;
+    html += `     </div>`;
     html += `</div>`;
 
     return this.toMessage({ content: html, flavor: label });
@@ -788,10 +791,11 @@ export class DGSanityDamageRoll extends DGRoll {
     )}</b> For <b>${lowDie.formula} / ${highDie.formula}</b>`;
 
     let html = "";
-    html += `<div class="dice-roll">`;
+    html += `<div class="dice-roll" data-action="expandRoll">`;
     html += `     <div class="dice-result">`;
     html += `     <div class="dice-formula">${lowDie.formula} / ${highDie.formula}</div>`;
     html += `     <div class="dice-tooltip">`;
+    html += `          <div class="wrapper">`;
     html += `          <section class="tooltip-part">`;
     html += `               <div class="dice">`;
     html += `                    <header class="part-header flexrow">`;
@@ -819,8 +823,10 @@ export class DGSanityDamageRoll extends DGRoll {
     html += `                    </ol>`;
     html += `               </div>`;
     html += `          </section>`;
+    html += `          </div>`;
     html += `     </div>`;
     html += `     <h4 class="dice-total">${lowResult} / ${highResult}</h4>`;
+    html += `     </div>`;
     html += `</div>`;
     return this.toMessage({ content: html, flavor });
   }
diff --git a/templates/actor/parts/about-tab.html b/templates/actor/parts/about-tab.html
index c2f07b5..607354c 100644
--- a/templates/actor/parts/about-tab.html
+++ b/templates/actor/parts/about-tab.html
@@ -3,31 +3,32 @@
      data-tab="about">
     <div class="about-section-header"></div>
 
-    <div class="flex horizontal-flex-column">
-        <div>
-            <a class="fas fa-question-circle"
-               title="{{localize 'DG.About.OnlineHelpTooltip'}}"
-               href="https://github.com/deltagreen-foundryvtt/delta-green-foundry-vtt-system/blob/master/documentation/home.md">
-                {{localize 'DG.About.OnlineHelp'}}
-            </a>
-        </div>
-        <div>
-            <a class="fas fa-bug"
-               title="{{localize 'DG.About.OnlineBugTrackerTooltip'}}"
-               href="https://github.com/deltagreen-foundryvtt/delta-green-foundry-vtt-system/issues">
-                {{localize 'DG.About.OnlineBugTracker'}}
-            </a>
-        </div>
-        <div>
-            <a class="fas fa-plus-circle"
-               title="{{localize 'DG.About.Changelogtip'}}"
-               href="https://github.com/deltagreen-foundryvtt/delta-green-foundry-vtt-system/blob/master/CHANGELOG.md">
-                {{localize 'DG.About.Changelog'}}
-            </a>
-        </div>
-        <div>
-            <p>{{localize 'DG.About.Copyright'}}</p>
-            <p>{{{localize 'DG.About.License'}}}</p>
-        </div>
+    <div class="dg-about-links flex horizontal-flex-column">
+        <a href="https://github.com/deltagreen-foundryvtt/delta-green-foundry-vtt-system/blob/master/documentation/home.md"
+           target="_blank"
+           rel="noopener"
+           data-tooltip="{{localize 'DG.About.OnlineHelpTooltip'}}">
+            <i class="fas fa-question-circle" inert></i>
+            <span>{{localize 'DG.About.OnlineHelp'}}</span>
+        </a>
+        <a href="https://github.com/deltagreen-foundryvtt/delta-green-foundry-vtt-system/issues"
+           target="_blank"
+           rel="noopener"
+           data-tooltip="{{localize 'DG.About.OnlineBugTrackerTooltip'}}">
+            <i class="fas fa-bug" inert></i>
+            <span>{{localize 'DG.About.OnlineBugTracker'}}</span>
+        </a>
+        <a href="https://github.com/deltagreen-foundryvtt/delta-green-foundry-vtt-system/blob/master/CHANGELOG.md"
+           target="_blank"
+           rel="noopener"
+           data-tooltip="{{localize 'DG.About.Changelogtip'}}">
+            <i class="fas fa-plus-circle" inert></i>
+            <span>{{localize 'DG.About.Changelog'}}</span>
+        </a>
+    </div>
+
+    <div class="dg-about-legal">
+        <p>{{localize 'DG.About.Copyright'}}</p>
+        <p>{{{localize 'DG.About.License'}}}</p>
     </div>
 </div>
diff --git a/templates/roll/percentile-roll.hbs b/templates/roll/percentile-roll.hbs
index 1c83e39..cef8a41 100644
--- a/templates/roll/percentile-roll.hbs
+++ b/templates/roll/percentile-roll.hbs
@@ -1,18 +1,20 @@
-<div class="dice-roll">
+<div class="dice-roll" data-action="expandRoll">
   <div class="dice-result">
     <div style="{{styleOverride}}" class="dice-formula">{{resultString}}</div>
     <div class="dice-tooltip">
-      <section class="tooltip-part">
-        <div class="dice">
-          <header class="part-header flexrow">
-            <span class="part-formula">{{formula}}</span>
-            <span class="part-total">{{total}}</span>
-          </header>
-          <ol class="dice-rolls">
-            <li class="roll die {{formula}}">{{total}}</li>
-          </ol>
-        </div>
-      </section>
+      <div class="wrapper">
+        <section class="tooltip-part">
+          <div class="dice">
+            <header class="part-header flexrow">
+              <span class="part-formula">{{formula}}</span>
+              <span class="part-total">{{total}}</span>
+            </header>
+            <ol class="dice-rolls">
+              <li class="roll die {{formula}}">{{total}}</li>
+            </ol>
+          </div>
+        </section>
+      </div>
     </div>
     <h4 class="dice-total">{{total}}</h4>
   </div>
@@ -27,4 +29,4 @@
     </div>
   {{/if}}
 
-</div>
\ No newline at end of file
+</div>