From d8399b56c94880dffc3fcdca18e5da98c6308d67 Mon Sep 17 00:00:00 2001
From: stephhou <steph.housden1@nhs.net>
Date: Thu, 12 Mar 2026 18:08:15 +0000
Subject: [PATCH 1/8] Feat: added phase banner to layout.jinja

---
 .../assets/sass/components/_phase_banner.scss | 20 +++++++++++++++++++
 lung_cancer_screening/assets/sass/main.scss   |  2 ++
 .../core/jinja2/layout.jinja                  |  3 ++-
 .../core/jinja2/phase_banner.jinja            | 15 ++++++++++++++
 4 files changed, 39 insertions(+), 1 deletion(-)
 create mode 100644 lung_cancer_screening/assets/sass/components/_phase_banner.scss
 create mode 100644 lung_cancer_screening/core/jinja2/phase_banner.jinja

diff --git a/lung_cancer_screening/assets/sass/components/_phase_banner.scss b/lung_cancer_screening/assets/sass/components/_phase_banner.scss
new file mode 100644
index 00000000..037baa6e
--- /dev/null
+++ b/lung_cancer_screening/assets/sass/components/_phase_banner.scss
@@ -0,0 +1,20 @@
+.lung-nhsuk-phase-banner__container{
+  border-bottom: 1px solid var(--nhsuk-grey-3-colour);
+  padding-bottom: 10px;
+  padding-top: 10px;
+}
+
+.lung-nhsuk-phase-banner__content {
+  display: table;
+  margin: 0;
+}
+
+.lung-nhsuk-phase-banner__text {
+  display: table-cell;
+  vertical-align: middle;
+  margin-left: 10px;
+}
+
+.nhsuk-tag__content{
+  margin-right: 10px;
+}
diff --git a/lung_cancer_screening/assets/sass/main.scss b/lung_cancer_screening/assets/sass/main.scss
index 5efa3de7..2a1e3272 100644
--- a/lung_cancer_screening/assets/sass/main.scss
+++ b/lung_cancer_screening/assets/sass/main.scss
@@ -3,3 +3,5 @@
 
 // Components that are not in the NHS.UK frontend library
 @forward "components/multi_field_input";
+
+@forward 'components/phase_banner';
diff --git a/lung_cancer_screening/core/jinja2/layout.jinja b/lung_cancer_screening/core/jinja2/layout.jinja
index d8530ab0..acebc70b 100644
--- a/lung_cancer_screening/core/jinja2/layout.jinja
+++ b/lung_cancer_screening/core/jinja2/layout.jinja
@@ -31,8 +31,9 @@
     }
   }) }}
 {% endblock header %}
-
 {% block beforeContent %}
+  {% include 'phase_banner.jinja' %}
+
   {% if back_link_url %}
     <nav>
       {{
diff --git a/lung_cancer_screening/core/jinja2/phase_banner.jinja b/lung_cancer_screening/core/jinja2/phase_banner.jinja
new file mode 100644
index 00000000..e22f8dad
--- /dev/null
+++ b/lung_cancer_screening/core/jinja2/phase_banner.jinja
@@ -0,0 +1,15 @@
+<section aria-label="Service phase">
+<div class="nhsuk-grid-row">
+  <div class="nhsuk-grid-column-full lung-nhsuk-phase-banner__container">
+    <div class="nhsuk-phase-banner">
+      <p class="lung-nhsuk-phase-banner__content">
+        <span class="nhsuk-tag nhsuk-tag__content nhsuk-tag--blue">Pilot</span>
+        <span class="lung-nhsuk-phase-banner__text nhsuk-body-s">
+          We are testing a new service &ndash; your <a
+            href="https://feedback.digital.nhs.uk/jfe/form/SV_6V8dAlFT0sxJnzo" target="_blank"
+            rel="noopener">feedback</a> will help us improve it.</span>
+      </p>
+    </div>
+  </div>
+</div>
+</section>

From 0ea15ff348d07552f798aa923731f52b68fcdbf4 Mon Sep 17 00:00:00 2001
From: stephhou <steph.housden1@nhs.net>
Date: Thu, 12 Mar 2026 19:17:22 +0000
Subject: [PATCH 2/8] test: Added sample tests for phase header

---
 features/static.feature                       | 16 ++++++++++----
 features/steps/page_assertion_steps.py        | 10 +++++++++
 .../core/jinja2/layout.jinja                  |  2 ++
 .../core/jinja2/phase_banner.jinja            | 22 +++++++++----------
 4 files changed, 35 insertions(+), 15 deletions(-)

diff --git a/features/static.feature b/features/static.feature
index 27f6092c..423c7e5a 100644
--- a/features/static.feature
+++ b/features/static.feature
@@ -2,15 +2,23 @@
 Feature: Static Pages
   Scenario: The static pages are available without logging in
     When I go to "/cookies"
-    Then I see a page title "NHS check if you need a lung scan cookies policy"
+    Then I see a phase header "Pilot"
+    And I see a link named "feedback"
+    And I see a page title "NHS check if you need a lung scan cookies policy"
     When I go to "/privacy-policy"
-    Then I see a page title "NHS check if you need a lung scan privacy policy"
+    Then I see a phase header "Pilot"
+    And I see a link named "feedback"
+    And I see a page title "NHS check if you need a lung scan privacy policy"
 
   Scenario: The static pages are available when logged in and accessible
     Given I am logged in
     When I go to "/cookies"
-    Then there are no accessibility violations
+    Then I see a phase header "Pilot"
+    And I see a link named "feedback"
+    And there are no accessibility violations
     And I see a page title "NHS check if you need a lung scan cookies policy"
     When I go to "/privacy-policy"
-    Then there are no accessibility violations
+    Then I see a phase header "Pilot"
+    And I see a link named "feedback"
+    And there are no accessibility violations
     And I see a page title "NHS check if you need a lung scan privacy policy"
diff --git a/features/steps/page_assertion_steps.py b/features/steps/page_assertion_steps.py
index e3884ad4..681b9969 100644
--- a/features/steps/page_assertion_steps.py
+++ b/features/steps/page_assertion_steps.py
@@ -14,3 +14,13 @@ def then_i_see_a_back_link_to(context, url):
 @then(u'I see a page title "{title}"')
 def then_i_see_a_page_title(context, title):
     expect(context.page.locator('h1')).to_have_text(title)
+
+@then(u'I see a phase header "{phase}"')
+def then_i_see_a_phase_header(context, phase):
+    expect(context.page.locator('.lung-nhsuk-phase-banner__text')).to_have_count(1)
+    expect(context.page.locator('.nhsuk-tag')).to_have_text(phase)
+
+@then(u'I see a link named "{link_text}"')
+def then_i_see_a_link_named(context, link_text):
+    link = context.page.locator(f'a:has-text("{link_text}")')
+    expect(link).to_have_count(1)
diff --git a/lung_cancer_screening/core/jinja2/layout.jinja b/lung_cancer_screening/core/jinja2/layout.jinja
index acebc70b..42dee240 100644
--- a/lung_cancer_screening/core/jinja2/layout.jinja
+++ b/lung_cancer_screening/core/jinja2/layout.jinja
@@ -31,7 +31,9 @@
     }
   }) }}
 {% endblock header %}
+
 {% block beforeContent %}
+
   {% include 'phase_banner.jinja' %}
 
   {% if back_link_url %}
diff --git a/lung_cancer_screening/core/jinja2/phase_banner.jinja b/lung_cancer_screening/core/jinja2/phase_banner.jinja
index e22f8dad..6554f154 100644
--- a/lung_cancer_screening/core/jinja2/phase_banner.jinja
+++ b/lung_cancer_screening/core/jinja2/phase_banner.jinja
@@ -1,15 +1,15 @@
 <section aria-label="Service phase">
-<div class="nhsuk-grid-row">
-  <div class="nhsuk-grid-column-full lung-nhsuk-phase-banner__container">
-    <div class="nhsuk-phase-banner">
-      <p class="lung-nhsuk-phase-banner__content">
-        <span class="nhsuk-tag nhsuk-tag__content nhsuk-tag--blue">Pilot</span>
-        <span class="lung-nhsuk-phase-banner__text nhsuk-body-s">
-          We are testing a new service &ndash; your <a
-            href="https://feedback.digital.nhs.uk/jfe/form/SV_6V8dAlFT0sxJnzo" target="_blank"
-            rel="noopener">feedback</a> will help us improve it.</span>
-      </p>
+  <div class="nhsuk-grid-row">
+    <div class="nhsuk-grid-column-full lung-nhsuk-phase-banner__container">
+      <div class="nhsuk-phase-banner">
+        <p class="lung-nhsuk-phase-banner__content">
+          <span class="nhsuk-tag nhsuk-tag__content nhsuk-tag--blue">Pilot</span>
+          <span class="lung-nhsuk-phase-banner__text nhsuk-body-s">
+            We are testing a new service &ndash; your <a
+              href="https://feedback.digital.nhs.uk/jfe/form/SV_6V8dAlFT0sxJnzo" target="_blank"
+              rel="noopener">feedback</a> will help us improve it.</span>
+        </p>
+      </div>
     </div>
   </div>
-</div>
 </section>

From f89b7943b8af7b80b37129fe9803883e6cf576c1 Mon Sep 17 00:00:00 2001
From: stephhou <steph.housden1@nhs.net>
Date: Mon, 16 Mar 2026 11:43:01 +0000
Subject: [PATCH 3/8] fix: Moved phase banner into header for consistency
 across pages

---
 lung_cancer_screening/core/jinja2/layout.jinja       | 4 ++--
 lung_cancer_screening/core/jinja2/phase_banner.jinja | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/lung_cancer_screening/core/jinja2/layout.jinja b/lung_cancer_screening/core/jinja2/layout.jinja
index 42dee240..92c50386 100644
--- a/lung_cancer_screening/core/jinja2/layout.jinja
+++ b/lung_cancer_screening/core/jinja2/layout.jinja
@@ -30,12 +30,12 @@
       ] if request.user.is_authenticated else []
     }
   }) }}
+
+    {% include 'phase_banner.jinja' %}
 {% endblock header %}
 
 {% block beforeContent %}
 
-  {% include 'phase_banner.jinja' %}
-
   {% if back_link_url %}
     <nav>
       {{
diff --git a/lung_cancer_screening/core/jinja2/phase_banner.jinja b/lung_cancer_screening/core/jinja2/phase_banner.jinja
index 6554f154..7b8e3b20 100644
--- a/lung_cancer_screening/core/jinja2/phase_banner.jinja
+++ b/lung_cancer_screening/core/jinja2/phase_banner.jinja
@@ -1,4 +1,4 @@
-<section aria-label="Service phase">
+<section aria-label="Service phase" class="nhsuk-width-container">
   <div class="nhsuk-grid-row">
     <div class="nhsuk-grid-column-full lung-nhsuk-phase-banner__container">
       <div class="nhsuk-phase-banner">

From 404e9506702e4aa231f8ac72654ca073a227af46 Mon Sep 17 00:00:00 2001
From: stephhou <steph.housden1@nhs.net>
Date: Mon, 16 Mar 2026 12:08:54 +0000
Subject: [PATCH 4/8] fix: fixed document format

---
 lung_cancer_screening/core/jinja2/layout.jinja | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/lung_cancer_screening/core/jinja2/layout.jinja b/lung_cancer_screening/core/jinja2/layout.jinja
index 92c50386..daa00570 100644
--- a/lung_cancer_screening/core/jinja2/layout.jinja
+++ b/lung_cancer_screening/core/jinja2/layout.jinja
@@ -31,11 +31,11 @@
     }
   }) }}
 
-    {% include 'phase_banner.jinja' %}
+  {% include 'phase_banner.jinja' %}
+
 {% endblock header %}
 
 {% block beforeContent %}
-
   {% if back_link_url %}
     <nav>
       {{

From ddad654b530e8efd48c997d1b1f67b78e8dbbe3b Mon Sep 17 00:00:00 2001
From: stephhou <steph.housden1@nhs.net>
Date: Mon, 16 Mar 2026 14:17:21 +0000
Subject: [PATCH 5/8] fix: made selections more specific for tests and app

---
 features/steps/page_assertion_steps.py        |  4 ++--
 .../assets/sass/components/_phase_banner.scss | 11 ++++------
 .../core/jinja2/phase_banner.jinja            | 22 +++++++++----------
 3 files changed, 16 insertions(+), 21 deletions(-)

diff --git a/features/steps/page_assertion_steps.py b/features/steps/page_assertion_steps.py
index 681b9969..4d5c5f66 100644
--- a/features/steps/page_assertion_steps.py
+++ b/features/steps/page_assertion_steps.py
@@ -17,8 +17,8 @@ def then_i_see_a_page_title(context, title):
 
 @then(u'I see a phase header "{phase}"')
 def then_i_see_a_phase_header(context, phase):
-    expect(context.page.locator('.lung-nhsuk-phase-banner__text')).to_have_count(1)
-    expect(context.page.locator('.nhsuk-tag')).to_have_text(phase)
+    expect(context.page.locator('.app-nhsuk-phase-banner__text')).to_have_count(1)
+    expect(context.page.locator('.app-nhsuk-phase-banner__content .nhsuk-tag')).to_have_text(phase)
 
 @then(u'I see a link named "{link_text}"')
 def then_i_see_a_link_named(context, link_text):
diff --git a/lung_cancer_screening/assets/sass/components/_phase_banner.scss b/lung_cancer_screening/assets/sass/components/_phase_banner.scss
index 037baa6e..c90a1b0e 100644
--- a/lung_cancer_screening/assets/sass/components/_phase_banner.scss
+++ b/lung_cancer_screening/assets/sass/components/_phase_banner.scss
@@ -1,20 +1,17 @@
-.lung-nhsuk-phase-banner__container{
+.app-nhsuk-phase-banner__container{
   border-bottom: 1px solid var(--nhsuk-grey-3-colour);
   padding-bottom: 10px;
   padding-top: 10px;
 }
 
-.lung-nhsuk-phase-banner__content {
+.app-nhsuk-phase-banner__content {
   display: table;
   margin: 0;
 }
 
-.lung-nhsuk-phase-banner__text {
+.app-nhsuk-phase-banner__text {
   display: table-cell;
   vertical-align: middle;
   margin-left: 10px;
-}
-
-.nhsuk-tag__content{
-  margin-right: 10px;
+  padding-left: 10px;
 }
diff --git a/lung_cancer_screening/core/jinja2/phase_banner.jinja b/lung_cancer_screening/core/jinja2/phase_banner.jinja
index 7b8e3b20..ea5ba373 100644
--- a/lung_cancer_screening/core/jinja2/phase_banner.jinja
+++ b/lung_cancer_screening/core/jinja2/phase_banner.jinja
@@ -1,15 +1,13 @@
-<section aria-label="Service phase" class="nhsuk-width-container">
-  <div class="nhsuk-grid-row">
-    <div class="nhsuk-grid-column-full lung-nhsuk-phase-banner__container">
-      <div class="nhsuk-phase-banner">
-        <p class="lung-nhsuk-phase-banner__content">
-          <span class="nhsuk-tag nhsuk-tag__content nhsuk-tag--blue">Pilot</span>
-          <span class="lung-nhsuk-phase-banner__text nhsuk-body-s">
-            We are testing a new service &ndash; your <a
-              href="https://feedback.digital.nhs.uk/jfe/form/SV_6V8dAlFT0sxJnzo" target="_blank"
-              rel="noopener">feedback</a> will help us improve it.</span>
-        </p>
-      </div>
+<section role="region" aria-label="Service phase" class="nhsuk-width-container">
+  <div class="app-nhsuk-phase-banner__container">
+    <div class="nhsuk-phase-banner">
+      <p class="app-nhsuk-phase-banner__content">
+        <span class="nhsuk-tag nhsuk-tag__content nhsuk-tag--blue">Pilot</span>
+        <span class="app-nhsuk-phase-banner__text nhsuk-body-s">
+          We are testing a new service &ndash; your <a
+            href="https://feedback.digital.nhs.uk/jfe/form/SV_6V8dAlFT0sxJnzo" target="_blank"
+            rel="noopener">feedback</a> will help us improve it.</span>
+      </p>
     </div>
   </div>
 </section>

From d8916bd02e4969a753ed29379fbc8fa2a8360763 Mon Sep 17 00:00:00 2001
From: stephhou <steph.housden1@nhs.net>
Date: Mon, 16 Mar 2026 14:26:56 +0000
Subject: [PATCH 6/8] fix: added lung prefix to css overrides

---
 .../assets/sass/components/_phase_banner.scss               | 6 +++---
 lung_cancer_screening/core/jinja2/phase_banner.jinja        | 6 +++---
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/lung_cancer_screening/assets/sass/components/_phase_banner.scss b/lung_cancer_screening/assets/sass/components/_phase_banner.scss
index c90a1b0e..8bf1931a 100644
--- a/lung_cancer_screening/assets/sass/components/_phase_banner.scss
+++ b/lung_cancer_screening/assets/sass/components/_phase_banner.scss
@@ -1,15 +1,15 @@
-.app-nhsuk-phase-banner__container{
+.lung-nhsuk-phase-banner__container{
   border-bottom: 1px solid var(--nhsuk-grey-3-colour);
   padding-bottom: 10px;
   padding-top: 10px;
 }
 
-.app-nhsuk-phase-banner__content {
+.lung-nhsuk-phase-banner__content {
   display: table;
   margin: 0;
 }
 
-.app-nhsuk-phase-banner__text {
+.lung-nhsuk-phase-banner__text {
   display: table-cell;
   vertical-align: middle;
   margin-left: 10px;
diff --git a/lung_cancer_screening/core/jinja2/phase_banner.jinja b/lung_cancer_screening/core/jinja2/phase_banner.jinja
index ea5ba373..4e463d1d 100644
--- a/lung_cancer_screening/core/jinja2/phase_banner.jinja
+++ b/lung_cancer_screening/core/jinja2/phase_banner.jinja
@@ -1,9 +1,9 @@
 <section role="region" aria-label="Service phase" class="nhsuk-width-container">
-  <div class="app-nhsuk-phase-banner__container">
+  <div class="lung-nhsuk-phase-banner__container">
     <div class="nhsuk-phase-banner">
-      <p class="app-nhsuk-phase-banner__content">
+      <p class="lung-nhsuk-phase-banner__content">
         <span class="nhsuk-tag nhsuk-tag__content nhsuk-tag--blue">Pilot</span>
-        <span class="app-nhsuk-phase-banner__text nhsuk-body-s">
+        <span class="lung-nhsuk-phase-banner__text nhsuk-body-s">
           We are testing a new service &ndash; your <a
             href="https://feedback.digital.nhs.uk/jfe/form/SV_6V8dAlFT0sxJnzo" target="_blank"
             rel="noopener">feedback</a> will help us improve it.</span>

From 2f064d0c9190434f2b9cca4f9ca1a5caf6e7a05e Mon Sep 17 00:00:00 2001
From: stephhou <steph.housden1@nhs.net>
Date: Mon, 16 Mar 2026 14:38:01 +0000
Subject: [PATCH 7/8] fix: updated tests to match css selectors

---
 features/steps/page_assertion_steps.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/features/steps/page_assertion_steps.py b/features/steps/page_assertion_steps.py
index 4d5c5f66..7e15a9a2 100644
--- a/features/steps/page_assertion_steps.py
+++ b/features/steps/page_assertion_steps.py
@@ -17,8 +17,8 @@ def then_i_see_a_page_title(context, title):
 
 @then(u'I see a phase header "{phase}"')
 def then_i_see_a_phase_header(context, phase):
-    expect(context.page.locator('.app-nhsuk-phase-banner__text')).to_have_count(1)
-    expect(context.page.locator('.app-nhsuk-phase-banner__content .nhsuk-tag')).to_have_text(phase)
+    expect(context.page.locator('.lung-nhsuk-phase-banner__text')).to_have_count(1)
+    expect(context.page.locator('.lung-nhsuk-phase-banner__content .nhsuk-tag')).to_have_text(phase)
 
 @then(u'I see a link named "{link_text}"')
 def then_i_see_a_link_named(context, link_text):

From da1611d5877337357461017654d90bf4f819a2d5 Mon Sep 17 00:00:00 2001
From: stephhou <steph.housden1@nhs.net>
Date: Mon, 16 Mar 2026 14:52:26 +0000
Subject: [PATCH 8/8] fix: removed unused margin

---
 lung_cancer_screening/assets/sass/components/_phase_banner.scss | 1 -
 1 file changed, 1 deletion(-)

diff --git a/lung_cancer_screening/assets/sass/components/_phase_banner.scss b/lung_cancer_screening/assets/sass/components/_phase_banner.scss
index 8bf1931a..ff63eb4b 100644
--- a/lung_cancer_screening/assets/sass/components/_phase_banner.scss
+++ b/lung_cancer_screening/assets/sass/components/_phase_banner.scss
@@ -12,6 +12,5 @@
 .lung-nhsuk-phase-banner__text {
   display: table-cell;
   vertical-align: middle;
-  margin-left: 10px;
   padding-left: 10px;
 }