From d391eb606c65218a78b1719274e2241bf7b9dae9 Mon Sep 17 00:00:00 2001
From: Colin Rotherham <work@colinr.com>
Date: Tue, 13 Feb 2024 17:00:54 +0000
Subject: [PATCH 1/3] Move Tab list items into macro for easy indenting

---
 src/govuk/components/tabs/template.njk | 18 +++++++++++-------
 1 file changed, 11 insertions(+), 7 deletions(-)

diff --git a/src/govuk/components/tabs/template.njk b/src/govuk/components/tabs/template.njk
index 99b1062cab..002e225169 100644
--- a/src/govuk/components/tabs/template.njk
+++ b/src/govuk/components/tabs/template.njk
@@ -1,5 +1,15 @@
 {% from "../../macros/attributes.njk" import govukAttributes %}
 
+{%- macro _tabListItem(params, item, index) %}
+{% set tabPanelId = item.id if item.id else idPrefix + "-" + index -%}
+<li class="govuk-tabs__list-item {%- if index == 1 %} govuk-tabs__list-item--selected{% endif %}">
+  <a class="govuk-tabs__tab" href="#{{ tabPanelId }}"
+    {{- govukAttributes(item.attributes) }}>
+    {{ item.label }}
+  </a>
+</li>
+{% endmacro -%}
+
 {#- If an id 'prefix' is not passed, fall back to using the name attribute
   instead. We need this for error messages and hints as well -#}
 {% set idPrefix = params.idPrefix if params.idPrefix -%}
@@ -13,13 +23,7 @@
   <ul class="govuk-tabs__list">
     {% for item in params.items %}
       {% if item %}
-        {% set id = item.id if item.id else idPrefix + "-" + loop.index %}
-        <li class="govuk-tabs__list-item {%- if loop.index == 1 %} govuk-tabs__list-item--selected{% endif %}">
-          <a class="govuk-tabs__tab" href="#{{ id }}"
-            {{- govukAttributes(item.attributes) }}>
-            {{ item.label }}
-          </a>
-        </li>
+        {{- _tabListItem(params, item, loop.index) }}
       {% endif %}
     {% endfor %}
   </ul>

From 1caeddcdf79b2e394b477bf0824bb1a7b653de4c Mon Sep 17 00:00:00 2001
From: Colin Rotherham <work@colinr.com>
Date: Tue, 13 Feb 2024 17:05:41 +0000
Subject: [PATCH 2/3] Move Tab panels into macro for easy indenting

---
 src/govuk/components/tabs/template.njk | 22 +++++++++++++---------
 1 file changed, 13 insertions(+), 9 deletions(-)

diff --git a/src/govuk/components/tabs/template.njk b/src/govuk/components/tabs/template.njk
index 002e225169..2ee7b5c664 100644
--- a/src/govuk/components/tabs/template.njk
+++ b/src/govuk/components/tabs/template.njk
@@ -10,6 +10,18 @@
 </li>
 {% endmacro -%}
 
+{%- macro _tabPanel(params, item, index) %}
+{% set tabPanelId = item.id if item.id else idPrefix + "-" + index -%}
+<div class="govuk-tabs__panel {%- if index > 1 %} govuk-tabs__panel--hidden{% endif %}" id="{{ tabPanelId }}"
+  {{- govukAttributes(item.panel.attributes) }}>
+  {% if item.panel.html %}
+    {{ item.panel.html | safe }}
+  {% elif item.panel.text %}
+    <p class="govuk-body">{{ item.panel.text }}</p>
+  {% endif %}
+</div>
+{% endmacro -%}
+
 {#- If an id 'prefix' is not passed, fall back to using the name attribute
   instead. We need this for error messages and hints as well -#}
 {% set idPrefix = params.idPrefix if params.idPrefix -%}
@@ -30,15 +42,7 @@
   {% endif %}
   {% for item in params.items %}
     {% if item %}
-      {% set id = item.id if item.id else idPrefix + "-" + loop.index %}
-      <div class="govuk-tabs__panel {%- if loop.index > 1 %} govuk-tabs__panel--hidden{% endif %}" id="{{ id }}"
-        {{- govukAttributes(item.panel.attributes) }}>
-        {% if item.panel.html %}
-          {{ item.panel.html | safe }}
-        {% elif item.panel.text %}
-          <p class="govuk-body">{{ item.panel.text }}</p>
-        {% endif %}
-      </div>
+      {{- _tabPanel(params, item, loop.index) }}
     {% endif %}
   {% endfor %}
 </div>

From 6a577e1721aaecab42e8449d3a40ebe3632eabb1 Mon Sep 17 00:00:00 2001
From: Colin Rotherham <work@colinr.com>
Date: Tue, 13 Feb 2024 17:11:37 +0000
Subject: [PATCH 3/3] Indent HTML for Tabs

---
 src/govuk/components/tabs/template.njk | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/src/govuk/components/tabs/template.njk b/src/govuk/components/tabs/template.njk
index 2ee7b5c664..77185e243b 100644
--- a/src/govuk/components/tabs/template.njk
+++ b/src/govuk/components/tabs/template.njk
@@ -14,11 +14,11 @@
 {% set tabPanelId = item.id if item.id else idPrefix + "-" + index -%}
 <div class="govuk-tabs__panel {%- if index > 1 %} govuk-tabs__panel--hidden{% endif %}" id="{{ tabPanelId }}"
   {{- govukAttributes(item.panel.attributes) }}>
-  {% if item.panel.html %}
-    {{ item.panel.html | safe }}
-  {% elif item.panel.text %}
-    <p class="govuk-body">{{ item.panel.text }}</p>
-  {% endif %}
+{% if item.panel.html %}
+  {{ item.panel.html | safe | trim | indent(2) }}
+{% elif item.panel.text %}
+  <p class="govuk-body">{{ item.panel.text }}</p>
+{% endif %}
 </div>
 {% endmacro -%}
 
@@ -31,18 +31,18 @@
   <h2 class="govuk-tabs__title">
     {{ params.title | default ("Contents") }}
   </h2>
-  {% if(params.items | length) %}
+{% if params.items | length %}
   <ul class="govuk-tabs__list">
     {% for item in params.items %}
       {% if item %}
-        {{- _tabListItem(params, item, loop.index) }}
+        {{- _tabListItem(params, item, loop.index) | trim | indent(4, true) }}
       {% endif %}
     {% endfor %}
   </ul>
-  {% endif %}
   {% for item in params.items %}
     {% if item %}
-      {{- _tabPanel(params, item, loop.index) }}
+      {{- _tabPanel(params, item, loop.index) | trim | indent(2, true) }}
     {% endif %}
   {% endfor %}
+{% endif %}
 </div>