From 2360f57f1ed51dd51bf3347d310c11001b9e1477 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E2=88=87ince?= <vince-roll20@users.noreply.github.com>
Date: Tue, 10 Dec 2024 00:38:42 -0800
Subject: [PATCH] embedded image adjustment for roll templates

---
 ADnD_1E_Revised/1ESheet.css  | 10 +++++++---
 ADnD_1E_Revised/1ESheet.html |  2 ++
 2 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/ADnD_1E_Revised/1ESheet.css b/ADnD_1E_Revised/1ESheet.css
index 1b6af202a420..a73aa778b163 100644
--- a/ADnD_1E_Revised/1ESheet.css
+++ b/ADnD_1E_Revised/1ESheet.css
@@ -3731,14 +3731,18 @@ details[open].thac0-box {
 /*images*/
 .sheet-rolltemplate-attacks a img,
 .sheet-rolltemplate-general a img {
-  aspect-ratio: 1;
+  aspect-ratio: auto;
   display: flex;
   flex-direction: row;
-  max-height: 300px;
+  justify-self: center;
+  margin: auto;
+  max-height: 50vw;
+  max-width: fit-content;
   object-fit: contain;
   object-position: center;
+  outline: 2px solid #000;
   overflow: clip;
-  margin: auto;
+  width: calc(100% - 5px);
 }
 
 .sheet-rolltemplate-attacks .sheet-subheader a img,
diff --git a/ADnD_1E_Revised/1ESheet.html b/ADnD_1E_Revised/1ESheet.html
index be113ba104e9..8ccb0d37cfb2 100644
--- a/ADnD_1E_Revised/1ESheet.html
+++ b/ADnD_1E_Revised/1ESheet.html
@@ -66,6 +66,8 @@ <h5 class="right two-columns">
             <span class="right"> • 12/8/24'</span>
           </h5>
           <ul>
+            <li>Roll templates now have an increased max-height for embedded images based on the viewport width. Should help to display larger images.<br>
+              <strong>FYI:</strong> images can be included with sheet rolls by using roll20's normal chat image format. ie <strong>[x](URL that ends in a common image format such as .jpg, .png, .gif)</strong> So if you want to display an image with a given roll, simply include a properly formatted image code macro within any text-based field. ie Name, Notes, Description. Macro-text can also be modified to include an image. ie <strong>{{subtag=[x](URL)}}</strong></li>
             <li>Fix for crit damage rolls; I've added parenthesis around the crit damage dice so that the entire roll is multiplied x2. This is a non-issue unless you want bonuses included with the crit multiplier. While it's common for crit damage to only double the dice roll and then add bonuses, you can include bonuses with the damage dice if desired. Note that the damage modifier field will also be included after any multiplier. Use it however you like.</li>
             <li>Rollback of last weeks single column not spanning full width 'fix' and a proper redo that doesn't cause collateral issues with the spell header layout.</li>
             <li>Small fix for single column spells and special abilities not spanning the full width.</li>