From f252a070266f5a5aa8112c3630df590bb77fd33b Mon Sep 17 00:00:00 2001
From: Ian James <ian.james@digital.cabinet-office.gov.uk>
Date: Thu, 4 Mar 2021 13:56:08 +0000
Subject: [PATCH] Update header search button to be more specific

Having the heading search button labelled "Search" clashed with a lot of the
finders, which also had a button labelled "Search" on it. This is beause screen
readers can collate and read out form fields, so having duplicate submit button
text could lead to confusion. Changing this to "Search GOV.UK" makes the intent
of this form clearer.
---
 .../components/layout_header/_search.html.erb     | 15 ++++++++++++---
 1 file changed, 12 insertions(+), 3 deletions(-)

diff --git a/app/views/govuk_publishing_components/components/layout_header/_search.html.erb b/app/views/govuk_publishing_components/components/layout_header/_search.html.erb
index d7ddf9eec2..24d9f12e41 100644
--- a/app/views/govuk_publishing_components/components/layout_header/_search.html.erb
+++ b/app/views/govuk_publishing_components/components/layout_header/_search.html.erb
@@ -1,8 +1,17 @@
-<button class="search-toggle js-header-toggle" data-search-toggle-for="search">Show search</button>
-<form id="search" class="gem-c-layout-header__search-form govuk-clearfix" action="/search" method="get" role="search">
+<button class="search-toggle js-header-toggle" data-search-toggle-for="search">
+  Show search
+</button>
+<form
+  action="/search"
+  class="gem-c-layout-header__search-form govuk-clearfix"
+  id="search"
+  method="get"
+  role="search"
+>
   <%= render "govuk_publishing_components/components/search", {
+    button_text: "Search GOV.UK",
     id: "site-search-text",
+    margin_bottom: 0,
     no_border: true,
-    margin_bottom: 0
   } %>
 </form>