Remove redundant wrapper div

This is only used in tests, for which we can use the `app-phase-banner` class instead.
alphagov · Nov 19, 2021 · a04885b · a04885b
1 parent a5860d4
commit a04885b
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 4 deletions.
diff --git a/__tests__/accessiblity_audit.test.js b/__tests__/accessiblity_audit.test.js
@@ -13,7 +13,7 @@ async function audit (page) {
   const axe = new AxePuppeteer(page)
     .include('body')
     .exclude('#app-site-search__input') // axe reports there is "no label associated with the text field", when there is one.
-    .exclude('.app-phase-banner__wrapper') // axe reports that the phase banner is not inside a landmark, which is intentional.
+    .exclude('.app-phase-banner') // axe reports that the phase banner is not inside a landmark, which is intentional.
 
   const results = await axe.analyze()
 

diff --git a/views/partials/_banner.njk b/views/partials/_banner.njk
@@ -17,8 +17,6 @@
 {% if bannerText %}
   {% from "govuk/components/phase-banner/macro.njk" import govukPhaseBanner %}
 
-  <div class="app-phase-banner__wrapper">
-
     {% set bannerClasses = "app-phase-banner app-width-container" %}
     {% if masthead %}
       {% set bannerClasses = bannerClasses + " app-phase-banner--no-border" %}
@@ -32,5 +30,4 @@
       "classes": bannerClasses,
       "html": bannerText
     }) }}
-  </div>
 {% endif %}