From 184b73581ca35bc684edc8cf4e31391d6d2e549d Mon Sep 17 00:00:00 2001
From: Ray Addams <ra730@cam.ac.uk>
Date: Mon, 17 Mar 2025 08:20:01 +0000
Subject: [PATCH 1/3] =?UTF-8?q?AM-447=20Height=20of=20link=20grid=20cards?=
 =?UTF-8?q?=20is=20wrong=20=E2=80=93=C2=A0internal=20padding=20issues?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../link-grid/link-grid.stories.twig          | 49 ++++++++++++++++++-
 .../link-grid/link-with-description.twig      |  4 +-
 2 files changed, 50 insertions(+), 3 deletions(-)

diff --git a/web/themes/custom/cambridge_tailwind/components/link-grid/link-grid.stories.twig b/web/themes/custom/cambridge_tailwind/components/link-grid/link-grid.stories.twig
index 8e54a25..11f63da 100644
--- a/web/themes/custom/cambridge_tailwind/components/link-grid/link-grid.stories.twig
+++ b/web/themes/custom/cambridge_tailwind/components/link-grid/link-grid.stories.twig
@@ -22,7 +22,7 @@
 %}
 
 
-{% story link_grid_1 with
+  {% story link_grid_1 with
     {
       name: '1. Default',
       args: {
@@ -43,5 +43,52 @@
     {% include '@cambridge_tailwind/components/link-grid/link-grid.twig' with { content: { field_link_grid_title: { 0: title }, field_link_grid_description: { 0: description }, field_list_of_links:list_of_links }, nested_paragraph_count: number_of_links } %}
   {% endstory %}
 
+    {% story link_grid_2 with
+        {
+            name: '1. Mix of description and no description',
+            args: {
+            title: "Heading",
+            description: "Description of the whole link set",
+            field_link_url: '/home',
+            field_link_title: 'Link grid link',
+            link_description: "A link description"
+        }
+        } %}
+        {% set list_of_links %}
+            {% include "@cambridge_tailwind/components/link-grid/link-with-description.twig" with {
+                field_link_url,
+                field_link_title,
+                field_featured_links_description:
+                link_description
+            } %}
+            {% include "@cambridge_tailwind/components/link-grid/link-with-description.twig" with {
+                field_link_url,
+                field_link_title: "Special Students at Harvard University",
+                field_featured_links_description: false,
+                link_description
+            } %}
+            {% include "@cambridge_tailwind/components/link-grid/link-with-description.twig" with {
+                field_link_url,
+                field_link_title: "Visiting Student at Yale",
+                field_featured_links_description: false,
+                link_description
+            } %}
+            {% include "@cambridge_tailwind/components/link-grid/link-with-description.twig" with {
+                field_link_url,
+                field_link_title,
+                field_featured_links_description: field_featured_links_description ~ "Lorem ipsum dolor sit amet, consectetur adipiscing elit.",
+                link_description
+            } %}
+            {% include "@cambridge_tailwind/components/link-grid/link-with-description.twig" with {
+                field_link_url,
+                field_link_title,
+                field_featured_links_description:
+                link_description
+            } %}
+        {% endset %}
+        {# Here, use any Twig you want, including custom functions, filters, etc. #}
+        {% include '@cambridge_tailwind/components/link-grid/link-grid.twig' with { content: { field_link_grid_title: { 0: title }, field_link_grid_description: { 0: description }, field_list_of_links:list_of_links }, nested_paragraph_count: number_of_links } %}
+    {% endstory %}
+
 
 {% endstories %}
diff --git a/web/themes/custom/cambridge_tailwind/components/link-grid/link-with-description.twig b/web/themes/custom/cambridge_tailwind/components/link-grid/link-with-description.twig
index 45359dc..c57c947 100644
--- a/web/themes/custom/cambridge_tailwind/components/link-grid/link-with-description.twig
+++ b/web/themes/custom/cambridge_tailwind/components/link-grid/link-with-description.twig
@@ -11,9 +11,9 @@
 {% set imageCardLinkClasses = cardLinkClasses() | spaceless | split(" ") %}
 {% set linkId = field_link_title | truncate(10) | clean_unique_id %}
 
-<a href="{{ field_link_url }}" class="{{ imageCardLinkClasses | join(" ") }} py-20" aria-labelledby="link_grid_{{linkId}}_label" {% if field_featured_links_description %}aria-describedby="link_grid_{{linkId}}_desc"{% endif %}>
+<a href="{{ field_link_url }}" class="{{ imageCardLinkClasses | join(" ") }} py-20 flex flex-col justify-center gap-12" aria-labelledby="link_grid_{{linkId}}_label" {% if field_featured_links_description %}aria-describedby="link_grid_{{linkId}}_desc"{% endif %}>
   <p class="text-style-heading-xs" id="link_grid_{{linkId}}_label">{{ field_link_title }}</p>
   {% if field_featured_links_description %}
-    <p class="text-text-tertiary text-style-body-sm pt-12" id="link_grid_{{linkId}}_desc">{{ field_featured_links_description }}</p>
+    <p class="text-text-tertiary text-style-body-sm" id="link_grid_{{linkId}}_desc">{{ field_featured_links_description }}</p>
   {% endif %}
 </a>
-- 
GitLab


From 662d2ed927edd19d06d32b5c75e0ce7a1472e00b Mon Sep 17 00:00:00 2001
From: Ray Addams <ra730@cam.ac.uk>
Date: Wed, 19 Mar 2025 07:52:34 +0000
Subject: [PATCH 2/3] =?UTF-8?q?CAMWEBPLAT-167=20Height=20of=20link=20grid?=
 =?UTF-8?q?=20cards=20is=20wrong=20=E2=80=93=20internal=20padding=20issues?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../components/link-grid/link-grid.stories.twig | 17 ++++++++++++-----
 .../link-grid/link-with-description.twig        |  4 ++--
 2 files changed, 14 insertions(+), 7 deletions(-)

diff --git a/web/themes/custom/cambridge_tailwind/components/link-grid/link-grid.stories.twig b/web/themes/custom/cambridge_tailwind/components/link-grid/link-grid.stories.twig
index 11f63da..ac78986 100644
--- a/web/themes/custom/cambridge_tailwind/components/link-grid/link-grid.stories.twig
+++ b/web/themes/custom/cambridge_tailwind/components/link-grid/link-grid.stories.twig
@@ -51,25 +51,32 @@
             description: "Description of the whole link set",
             field_link_url: '/home',
             field_link_title: 'Link grid link',
-            link_description: "A link description"
+            link_description: "A link description",
+            number_of_links: 6
         }
         } %}
         {% set list_of_links %}
             {% include "@cambridge_tailwind/components/link-grid/link-with-description.twig" with {
                 field_link_url,
-                field_link_title,
-                field_featured_links_description:
+                field_link_title: "Special Students at Harvard University",
+                field_featured_links_description: false,
                 link_description
             } %}
             {% include "@cambridge_tailwind/components/link-grid/link-with-description.twig" with {
                 field_link_url,
-                field_link_title: "Special Students at Harvard University",
+                field_link_title: "Visiting Student at Yale",
                 field_featured_links_description: false,
                 link_description
             } %}
             {% include "@cambridge_tailwind/components/link-grid/link-with-description.twig" with {
                 field_link_url,
-                field_link_title: "Visiting Student at Yale",
+                field_link_title: "Wikipedia page - Henry Fellowship",
+                field_featured_links_description: false,
+                link_description
+            } %}
+            {% include "@cambridge_tailwind/components/link-grid/link-with-description.twig" with {
+                field_link_url,
+                field_link_title: "Wikipedia page - Procter Fellowship",
                 field_featured_links_description: false,
                 link_description
             } %}
diff --git a/web/themes/custom/cambridge_tailwind/components/link-grid/link-with-description.twig b/web/themes/custom/cambridge_tailwind/components/link-grid/link-with-description.twig
index c57c947..45359dc 100644
--- a/web/themes/custom/cambridge_tailwind/components/link-grid/link-with-description.twig
+++ b/web/themes/custom/cambridge_tailwind/components/link-grid/link-with-description.twig
@@ -11,9 +11,9 @@
 {% set imageCardLinkClasses = cardLinkClasses() | spaceless | split(" ") %}
 {% set linkId = field_link_title | truncate(10) | clean_unique_id %}
 
-<a href="{{ field_link_url }}" class="{{ imageCardLinkClasses | join(" ") }} py-20 flex flex-col justify-center gap-12" aria-labelledby="link_grid_{{linkId}}_label" {% if field_featured_links_description %}aria-describedby="link_grid_{{linkId}}_desc"{% endif %}>
+<a href="{{ field_link_url }}" class="{{ imageCardLinkClasses | join(" ") }} py-20" aria-labelledby="link_grid_{{linkId}}_label" {% if field_featured_links_description %}aria-describedby="link_grid_{{linkId}}_desc"{% endif %}>
   <p class="text-style-heading-xs" id="link_grid_{{linkId}}_label">{{ field_link_title }}</p>
   {% if field_featured_links_description %}
-    <p class="text-text-tertiary text-style-body-sm" id="link_grid_{{linkId}}_desc">{{ field_featured_links_description }}</p>
+    <p class="text-text-tertiary text-style-body-sm pt-12" id="link_grid_{{linkId}}_desc">{{ field_featured_links_description }}</p>
   {% endif %}
 </a>
-- 
GitLab


From 5c150addc8048e0122e9868dbbd796aa161303ff Mon Sep 17 00:00:00 2001
From: Ray Addams <ra730@cam.ac.uk>
Date: Fri, 21 Mar 2025 13:57:51 +0000
Subject: [PATCH 3/3] =?UTF-8?q?CAMWEBPLAT-167=20Height=20of=20link=20grid?=
 =?UTF-8?q?=20cards=20is=20wrong=20=E2=80=93=20internal=20padding=20issues?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../components/link-grid/link-with-description.twig        | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/web/themes/custom/cambridge_tailwind/components/link-grid/link-with-description.twig b/web/themes/custom/cambridge_tailwind/components/link-grid/link-with-description.twig
index 45359dc..338c895 100644
--- a/web/themes/custom/cambridge_tailwind/components/link-grid/link-with-description.twig
+++ b/web/themes/custom/cambridge_tailwind/components/link-grid/link-with-description.twig
@@ -7,13 +7,16 @@
  * field_featured_links_description
  */
 #}
+
+{% set field_featured_links_description %}{{ field_featured_links_description }}{% endset %}
+
 {% from '@cambridge_tailwind/components/link/link.twig' import cardLinkClasses %}
 {% set imageCardLinkClasses = cardLinkClasses() | spaceless | split(" ") %}
 {% set linkId = field_link_title | truncate(10) | clean_unique_id %}
 
-<a href="{{ field_link_url }}" class="{{ imageCardLinkClasses | join(" ") }} py-20" aria-labelledby="link_grid_{{linkId}}_label" {% if field_featured_links_description %}aria-describedby="link_grid_{{linkId}}_desc"{% endif %}>
+<a href="{{ field_link_url }}" class="{{ imageCardLinkClasses | join(" ") }} py-20" aria-labelledby="link_grid_{{linkId}}_label" {% if field_featured_links_description is not empty %}aria-describedby="link_grid_{{linkId}}_desc"{% endif %}>
   <p class="text-style-heading-xs" id="link_grid_{{linkId}}_label">{{ field_link_title }}</p>
-  {% if field_featured_links_description %}
+  {% if field_featured_links_description is not empty %}
     <p class="text-text-tertiary text-style-body-sm pt-12" id="link_grid_{{linkId}}_desc">{{ field_featured_links_description }}</p>
   {% endif %}
 </a>
-- 
GitLab