From 4adcea7361f01e257e08e2928bc90a848541c94f Mon Sep 17 00:00:00 2001
From: Jason Irish <jirish@dynamicagency.com>
Date: Tue, 13 Aug 2024 21:57:05 -0500
Subject: [PATCH 1/2] FEATURE Image position options

---
 src/Element/ElementCard.php                   | 21 ++++++++++----
 .../Elements/Card/Elements/ElementCard.ss     | 28 ++++++++++++-------
 2 files changed, 33 insertions(+), 16 deletions(-)

diff --git a/src/Element/ElementCard.php b/src/Element/ElementCard.php
index 979ef5e..a378c2f 100644
--- a/src/Element/ElementCard.php
+++ b/src/Element/ElementCard.php
@@ -51,7 +51,7 @@ class ElementCard extends BaseElement
      * @var string
      * @config
      */
-    private static string $icon = 'font-icon-block-content';
+    private static string $icon = 'font-icon-block-content'; 
 
     /**
      * @var array|string[]
@@ -59,6 +59,7 @@ class ElementCard extends BaseElement
      */
     private static array $db = [
         'Content' => 'HTMLText',
+        'Position' => 'Enum("Left, Right, Top", "Left")',
     ];
 
     /**
@@ -90,6 +91,9 @@ public function fieldLabels($includerelations = true)
         $labels['Title'] = _t(__CLASS__ . '.TitleLabel', 'Title');
         $labels['Content'] = _t(__CLASS__ . '.ContentLabel', 'Description');
         $labels['ElementLink'] = _t(__CLASS__ . '.ElementLinkLabel', 'Link');
+        $labels['Position'] = _t(__CLASS__ . '.PositionLabel', 'Image Position');
+        $labels['Image'] = _t(__CLASS__ . '.Image', 'Image');
+
 
         return $labels;
     }
@@ -100,15 +104,20 @@ public function fieldLabels($includerelations = true)
     public function getCMSFields(): FieldList
     {
         $this->beforeUpdateCMSFields(function (FieldList $fields) {
+            // @phpstan-ignore-next-line
+            $fields->dataFieldByName('Content')
+                ->setRows(8);
+
             // @phpstan-ignore-next-line
             $fields->dataFieldByName('Image')
                 ->setFolderName('Uploads/Elements/Card');
+            $fields->insertBefore('Position', $fields->dataFieldByName('Image'));
 
-            $fields->insertBefore('Content', $fields->dataFieldByName('Image'));
-
-            // @phpstan-ignore-next-line
-            $fields->dataFieldByName('Content')
-                ->setRows(5);
+            $fields->dataFieldByName('Position')
+                ->setDescription(_t(
+                    __CLASS__ . 'PositionDescription', 
+                    'Position of the image in relation to the content. Image will display on top at smaller breakpoints.'
+                ));
 
             $fields->replaceField(
                 'ElementLinkID',
diff --git a/templates/Dynamic/Elements/Card/Elements/ElementCard.ss b/templates/Dynamic/Elements/Card/Elements/ElementCard.ss
index d1b56e4..a7cd18a 100644
--- a/templates/Dynamic/Elements/Card/Elements/ElementCard.ss
+++ b/templates/Dynamic/Elements/Card/Elements/ElementCard.ss
@@ -1,14 +1,22 @@
-<div class="element__card">
-    <div class="card h-100">
+<div class="card h-100">
+    <div class="row g-0">
         <% if $Image %>
-            <% if $ElementLink %><a href="$ElementLink.LinkURL" title="Go to $ElementLink.Title.ATT"><% end_if %>
-                <img src="$Image.FocusFill(500,330).URL" class="card-img-top" alt="$Image.Title.ATT">
-            <% if $ElementLink %></a><% end_if %>
+            <% if $Position == 'Top' %>
+                <img src="$Image.FocusFill(800,530).URL" class="card-img-top" alt="$Image.Title.ATT">
+            <% else %>
+                <div class="col-lg-7">
+                    <img src="$Image.FocusFill(800,530).URL" class="img-fluid" alt="$Image.Title.ATT">
+                </div>
+            <% end_if %>
         <% end_if %>
-        <div class="card-body">
-            <% if $Title && $ShowTitle %><$TitleTag class="element__title $TitleSizeClass">$Title</$TitleTag><% end_if %>
-            <% if $Content %><div class="card-text">$Content</div><% end_if %>
-            <% if $ElementLink %><p><a href="$ElementLink.URL" class="btn btn-primary" title="$ElementLink.Title"<% if $ElementLink.OpenInNew %> target="_blank" rel="noopener noreferrer"<% end_if %>>$ElementLink.Title</a></p><% end_if %>
+        <div class="<% if $Position != 'Top' %> col-lg-5<% end_if %><% if $Position == 'Right' %> order-md-first<% end_if %>">
+            <div class="card-body h-100 d-flex align-items-center">
+                <div>
+                    <% if $Title && $ShowTitle %><$TitleTag class="element__title $TitleSizeClass">$Title</$TitleTag><% end_if %>
+                    <% if $Content %><div class="card-text">$Content</div><% end_if %>
+                    <% if $ElementLink %><p><a href="$ElementLink.URL" class="btn btn-primary" title="$ElementLink.Title"<% if $ElementLink.OpenInNew %> target="_blank" rel="noopener noreferrer"<% end_if %>>$ElementLink.Title</a></p><% end_if %>
+                </div>
+            </div>
         </div>
     </div>
-</div>
+</div>
\ No newline at end of file

From 5696e4656abf969e6d148485177af6000d566e81 Mon Sep 17 00:00:00 2001
From: Jason Irish <jirish@dynamicagency.com>
Date: Tue, 13 Aug 2024 22:04:12 -0500
Subject: [PATCH 2/2] phpcs

---
 src/Element/ElementCard.php | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/src/Element/ElementCard.php b/src/Element/ElementCard.php
index a378c2f..e53580e 100644
--- a/src/Element/ElementCard.php
+++ b/src/Element/ElementCard.php
@@ -51,7 +51,7 @@ class ElementCard extends BaseElement
      * @var string
      * @config
      */
-    private static string $icon = 'font-icon-block-content'; 
+    private static string $icon = 'font-icon-block-content';
 
     /**
      * @var array|string[]
@@ -115,8 +115,9 @@ public function getCMSFields(): FieldList
 
             $fields->dataFieldByName('Position')
                 ->setDescription(_t(
-                    __CLASS__ . 'PositionDescription', 
-                    'Position of the image in relation to the content. Image will display on top at smaller breakpoints.'
+                    __CLASS__ . 'PositionDescription',
+                    'Position of the image in relation to the content. 
+                        Image will display on top at smaller breakpoints.'
                 ));
 
             $fields->replaceField(