govuk_publishing_components 34.0.0 → 34.5.0

data/app/assets/stylesheets/govuk_publishing_components/components/_subscription-links.scss

@@ -123,7 +123,6 @@
   .gem-c-subscription-links__item--link {
     padding: govuk-spacing(2) govuk-spacing(2) govuk-spacing(2) 0;
     border: 1px solid transparent;
-    border-bottom: 1px solid govuk-colour("dark-grey", $legacy: "grey-1");
 
     &:focus {
       border-bottom-color: transparent;

data/app/assets/stylesheets/govuk_publishing_components/components/govspeak/_steps.scss

@@ -3,21 +3,38 @@
   .steps {
     padding-left: 0;
     margin-left: 0;
-    overflow: hidden;
 
     > li {
-      background-position: 0 .87em;
-      background-repeat: no-repeat;
-      list-style-type: decimal;
+      list-style-type: none;
       margin-left: 0;
       padding: .75em 0 .75em 2.5em;
+      position: relative;
+
+      // IE doesn't render this correctly with this element as display:block
+      // (it leaves a space for the :before pseudo element) by setting to
+      // display inline block and a width of 100% it works correctly.
+      display: inline-block;
+      width: 100%;
+      box-sizing: border-box;
+
+      &:before {
+        background-color: govuk-colour("black");
+        border-radius: 50%;
+        color: govuk-colour("white");
+        float: left;
+        font-size: 80%;
+        height: 1.75em;
+        left: .3em;
+        line-height: 1.75em;
+        position: absolute;
+        text-align: center;
+        top: .9em;
+        width: 1.75em;
+      }
 
       @for $i from 1 through 30 {
-        &:nth-child(#{$i}) {
-          background-image: url("data:image/svg+xml, %3Csvg xmlns='http://www.w3.org/2000/svg' viewBox='0 0 250 250' preserveAspectRatio='xMinYMin meet'%3E%3Cg%3E%3Ccircle r='50%25' cx='50%25' cy='50%25' class='circle-back'%3E%3C/circle%3E%3Ctext x='50%25' y='50%25' text-anchor='middle' dy='0.3em' font-family='nta,arial,sans-serif' font-size='8rem' fill='%23ffffff'%3E#{$i}%3C/text%3E%3C/g%3E%3C/svg%3E ");
-          background-repeat: no-repeat;
-          background-position: .2em .7em;
-          background-size: 1.4em 1.4em;
+        &:nth-child(#{$i}):before {
+          content: "#{$i}";
         }
       }
     }

data/app/assets/stylesheets/govuk_publishing_components/components/helpers/_brand-colours.scss

@@ -57,12 +57,12 @@
 
 .brand--prime-ministers-office-10-downing-street {
   .brand__color {
-    color: govuk-colour("bright-purple");
+    color: govuk-colour("black");
 
     &:link,
     &:visited,
     &:active {
-      color: govuk-colour("bright-purple");
+      color: govuk-colour("black");
     }
 
     &:hover,

data/app/controllers/govuk_publishing_components/application_controller.rb

@@ -5,19 +5,6 @@ module GovukPublishingComponents
     before_action :set_x_frame_options_header
     before_action :set_disable_slimmer_header
 
-    if defined? content_security_policy
-      content_security_policy do |p|
-        # don't do anything if the app doesn't have a content security policy
-        next unless p.directives.any?
-
-        # Unfortunately the axe core script uses a dependency that uses eval
-        # see: https://github.com/dequelabs/axe-core/issues/1175
-        # Thus all components shown by govuk_publishing_components need this
-        # enabled
-        p.script_src(*p.script_src, :unsafe_eval)
-      end
-    end
-
   private
 
     def set_x_frame_options_header

data/app/controllers/govuk_publishing_components/audit_controller.rb

@@ -34,7 +34,7 @@ module GovukPublishingComponents
       @in_application = false
       @in_application = true unless ENV["MAIN_COMPONENT_GUIDE"]
 
-      components = AuditComponents.new(gem_path, false)
+      components = AuditComponents.new(gem_path)
       applications = analyse_applications(host_dir, application_dirs)
       compared_data = AuditComparer.new(components.data, applications)
 

data/app/models/govuk_publishing_components/audit_comparer.rb

@@ -252,7 +252,7 @@ module GovukPublishingComponents
       results = []
       found_something = false
 
-      @gem_data[:component_listing].each do |component|
+      @gem_data[:component_file_details].each do |component|
         component_name = component[:name]
         locations = []
 

data/app/models/govuk_publishing_components/audit_components.rb

@@ -2,65 +2,123 @@ module GovukPublishingComponents
   class AuditComponents
     attr_reader :data
 
-    def initialize(path, simple)
-      @data = {
-        gem_found: false,
-      }
-      @data = compile_data(path, simple) if Dir.exist?(path)
-    end
-
-  private
-
-    def compile_data(path, simple)
-      # simple is used to reduce effort (and therefore page load time) required
-      # when loading auditing summary on the main component guide page
-      @simple = simple
-
+    def initialize(path)
       # paths to key file locations
       @templates_path = "app/views/govuk_publishing_components/components"
       @stylesheets_path = "app/assets/stylesheets/govuk_publishing_components/components"
       @print_stylesheets_path = "app/assets/stylesheets/govuk_publishing_components/components/print"
       @javascripts_path = "app/assets/javascripts/govuk_publishing_components/components"
       @tests_path = "spec/components"
-      @js_tests_path = "spec/javascripts/components"
+      @javascript_tests_path = "spec/javascripts/components"
       @helpers_path = "lib/govuk_publishing_components/presenters"
 
-      # get all files in key file locations
-      templates = Dir["#{path}/#{@templates_path}/*.erb"]
-      stylesheets = Dir["#{path}/#{@stylesheets_path}/*.scss"]
-      print_stylesheets = Dir["#{path}/#{@print_stylesheets_path}/*.scss"]
-      javascripts = Dir["#{path}/#{@javascripts_path}/*.js"]
-      tests = Dir["#{path}/#{@tests_path}/*.rb"]
-      js_tests = Dir["#{path}/#{@js_tests_path}/*.js"]
-      helpers = Dir["#{path}/#{@helpers_path}/*_helper.rb"]
-
+      @all_templates = Dir["#{path}/#{@templates_path}/*.erb"].sort
       @templates_full_path = "#{path}/#{@templates_path}/"
 
-      # find the cleaned names of components in key file locations
-      # i.e. will show that 'component name' has a stylesheet
-      # standardised like this to be used later for easier comparison
-      @components = clean_files(templates, [path, @templates_path].join("/"))
-      @component_stylesheets = clean_files(stylesheets, [path, @stylesheets_path].join("/"))
-      @component_print_stylesheets = clean_files(print_stylesheets, [path, @print_stylesheets_path].join("/"))
-      @component_javascripts = clean_files(javascripts, [path, @javascripts_path].join("/"))
-      @component_tests = clean_files(tests, [path, @tests_path].join("/"))
-      @component_js_tests = clean_files(js_tests, [path, @js_tests_path].join("/"))
-      @component_helpers = clean_files(helpers, [path, @helpers_path].join("/"))
+      @component_numbers = {
+        template: 0,
+        stylesheet: 0,
+        print_stylesheet: 0,
+        javascript: 0,
+        test: 0,
+        javascript_test: 0,
+        helper: 0,
+      }
 
+      @data = {
+        gem_found: false,
+      }
+      @data = compile_data(path) if Dir.exist?(path)
+    end
+
+  private
+
+    def compile_data(path)
       {
         gem_found: true,
-        component_code: @components,
-        component_stylesheets: @component_stylesheets,
-        component_print_stylesheets: @component_print_stylesheets,
-        component_javascripts: @component_javascripts,
-        component_tests: @component_tests,
-        component_js_tests: @component_js_tests,
-        component_helpers: @component_helpers,
-        components_containing_components: find_all_partials_in(templates),
-        component_listing: list_all_component_details,
+        component_code: clean_files(@all_templates, @templates_full_path).sort,
+        components_containing_components: find_all_partials_in(@all_templates),
+        component_file_details: get_component_file_details(path),
+        component_numbers: @component_numbers,
       }
     end
 
+    def get_component_file_details(path)
+      component_templates = clean_files(@all_templates, @templates_full_path).sort
+      details = []
+      component_templates.each do |component|
+        component_detail = {
+          name: component,
+          link: get_component_link(component),
+        }
+        file_details = [
+          {
+            type: "template",
+            file: "#{[path, @templates_path].join('/')}/_#{component.gsub(' ', '_')}.html.erb",
+          },
+          {
+            type: "stylesheet",
+            file: "#{[path, @stylesheets_path].join('/')}/_#{component.gsub(' ', '-')}.scss",
+          },
+          {
+            type: "print_stylesheet",
+            file: "#{[path, @stylesheets_path].join('/')}/_#{component.gsub(' ', '-')}.scss",
+          },
+          {
+            type: "javascript",
+            file: "#{[path, @javascripts_path].join('/')}/#{component.gsub(' ', '-')}.js",
+          },
+          {
+            type: "test",
+            file: "#{[path, @tests_path].join('/')}/#{component.gsub(' ', '_')}_spec.rb",
+          },
+          {
+            type: "javascript_test",
+            file: "#{[path, @javascript_tests_path].join('/')}/#{component.gsub(' ', '-')}-spec.js",
+          },
+          {
+            type: "helper",
+            file: "#{[path, @helpers_path].join('/')}/#{component.gsub(' ', '_')}_helper.rb",
+          },
+        ]
+        file_details.each do |detail|
+          component_detail = component_detail.merge(get_component_asset_detail(detail, component))
+        end
+
+        details << component_detail
+      end
+
+      details
+    end
+
+    def get_component_asset_detail(detail, component)
+      details = {}
+      type = detail[:type]
+      file = detail[:file]
+      details["#{type}_exists".to_sym] = false
+      if File.file?(file)
+        # we don't have separate print stylesheets anymore
+        # so check the main stylesheet for print styles
+        if type == "print_stylesheet"
+          unless File.foreach(file).grep(/(media-type: print)/).empty?
+            details["#{type}_exists".to_sym] = true
+            @component_numbers[type.to_sym] += 1
+          end
+        else
+          @component_numbers[type.to_sym] += 1
+          details["#{type}_exists".to_sym] = true
+          details["#{type}_lines".to_sym] = count_lines_in(file)
+          details["#{type}_link".to_sym] = get_asset_link(type, component)
+        end
+      end
+
+      details
+    end
+
+    def count_lines_in(file)
+      File.read(file).each_line.count
+    end
+
     def clean_files(files, replace)
       files.map { |file| clean_file_name(file.gsub(replace, "")) }.sort
     end
@@ -91,8 +149,6 @@ module GovukPublishingComponents
     end
 
     def find_all_partials_in(templates)
-      return [] if @simple
-
       components = []
 
       templates.each do |template|
@@ -144,52 +200,17 @@ module GovukPublishingComponents
       end
     end
 
-    def list_all_component_details
-      return [] if @simple
-
-      all_component_information = []
-
-      @components.each do |component|
-        all_component_information << {
-          name: component,
-          link: get_component_link(component),
-          stylesheet: check_component_has("stylesheet", component),
-          print_stylesheet: check_component_has("print_stylesheet", component),
-          javascript: check_component_has("javascript", component),
-          tests: check_component_has("test", component),
-          js_tests: check_component_has("js_test", component),
-          helper: check_component_has("helper", component),
-        }
-      end
-
-      all_component_information
-    end
-
-    def check_component_has(a_thing, component)
-      look_in = @component_stylesheets if a_thing == "stylesheet"
-      look_in = @component_print_stylesheets if a_thing == "print_stylesheet"
-      look_in = @component_javascripts if a_thing == "javascript"
-      look_in = @component_tests if a_thing == "test"
-      look_in = @component_js_tests if a_thing == "js_test"
-      look_in = @component_helpers if a_thing == "helper"
-
-      if look_in.include?(component)
-        return get_asset_link(a_thing, component)
-      end
-
-      false
-    end
-
     def get_asset_link(a_thing, component)
       url = "https://github.com/alphagov"
       repo = "govuk_publishing_components"
       blob = "blob/main"
       link = nil
+      link = "#{url}/#{repo}/#{blob}/#{@templates_path}/_#{component.gsub(' ', '_')}.html.erb" if a_thing == "template"
       link = "#{url}/#{repo}/#{blob}/#{@stylesheets_path}/_#{component.gsub(' ', '-')}.scss" if a_thing == "stylesheet"
       link = "#{url}/#{repo}/#{blob}/#{@print_stylesheets_path}/_#{component.gsub(' ', '-')}.scss" if a_thing == "print_stylesheet"
       link = "#{url}/#{repo}/#{blob}/#{@javascripts_path}/#{component.gsub(' ', '-')}.js" if a_thing == "javascript"
       link = "#{url}/#{repo}/#{blob}/#{@tests_path}/#{component.gsub(' ', '_')}_spec.rb" if a_thing == "test"
-      link = "#{url}/#{repo}/#{blob}/#{@js_tests_path}/#{component.gsub(' ', '-')}-spec.js" if a_thing == "js_test"
+      link = "#{url}/#{repo}/#{blob}/#{@javascript_tests_path}/#{component.gsub(' ', '-')}-spec.js" if a_thing == "javascript_test"
       link = "#{url}/#{repo}/#{blob}/#{@helpers_path}/#{component.gsub(' ', '_')}_helper.rb" if a_thing == "helper"
 
       link

data/app/views/govuk_publishing_components/audit/_applications.html.erb

@@ -12,7 +12,7 @@
   <%= render "govuk_publishing_components/components/details", {
     title: "How to use this information"
   } do %>
-    <p class="govuk-body">This page shows information about component use on GOV.UK. This information has been cross referenced with the components in the gem to produce warnings where e.g. a print stylesheet for a component exists but has not been included in an application.</p>
+    <p class="govuk-body">This page shows information about component use on GOV.UK. This information has been cross referenced with the components in the gem to produce warnings where e.g. JavaScript for a component exists but has not been included in an application.</p>
     <p class="govuk-body">Warnings should be investigated, although there may be a reason why the application has been configured as it is. Note that 'code' can refer to templates or ruby code.</p>
   <% end %>
 

data/app/views/govuk_publishing_components/audit/_component_contents.html.erb

@@ -1,78 +1,86 @@
-<%
-  total_components = components.length
-  components_with_stylesheet = 0
-  components_with_print_stylesheet = 0
-  components_with_javascript = 0
-  components_with_test = 0
-  components_with_javascript_test = 0
-  components_with_helper = 0
-
-  components.each do |component|
-    components_with_stylesheet += 1 if component[:stylesheet]
-    components_with_print_stylesheet += 1 if component[:print_stylesheet]
-    components_with_javascript += 1 if component[:javascript]
-    components_with_test += 1 if component[:tests]
-    components_with_javascript_test += 1 if component[:js_tests]
-    components_with_helper += 1 if component[:helper]
-  end
-%>
 <table class="govuk-table">
   <thead class="govuk-table__head">
     <tr class="govuk-table__row">
-      <th scope="col" class="govuk-table__header sticky-table-header">Component (<%= total_components %>)</th>
-      <th scope="col" class="govuk-table__header sticky-table-header">CSS (<%= components_with_stylesheet %>)</th>
-      <th scope="col" class="govuk-table__header sticky-table-header">Print CSS (<%= components_with_print_stylesheet %>)</th>
-      <th scope="col" class="govuk-table__header sticky-table-header">JS (<%= components_with_javascript %>)</th>
-      <th scope="col" class="govuk-table__header sticky-table-header">Test (<%= components_with_test %>)</th>
-      <th scope="col" class="govuk-table__header sticky-table-header">JS test (<%= components_with_javascript_test %>)</th>
-      <th scope="col" class="govuk-table__header sticky-table-header">Helper (<%= components_with_helper %>)</th>
+      <th scope="col" class="govuk-table__header sticky-table-header">Component</th>
+      <th scope="col" class="govuk-table__header sticky-table-header">Template (<%= @components[:component_numbers][:template] %>)</th>
+      <th scope="col" class="govuk-table__header sticky-table-header">CSS (<%= @components[:component_numbers][:stylesheet] %>)</th>
+      <th scope="col" class="govuk-table__header sticky-table-header">Print styles (<%= @components[:component_numbers][:print_stylesheet] %>)</th>
+      <th scope="col" class="govuk-table__header sticky-table-header">JS (<%= @components[:component_numbers][:javascript] %>)</th>
+      <th scope="col" class="govuk-table__header sticky-table-header">Test (<%= @components[:component_numbers][:test] %>)</th>
+      <th scope="col" class="govuk-table__header sticky-table-header">JS test (<%= @components[:component_numbers][:javascript_test] %>)</th>
+      <th scope="col" class="govuk-table__header sticky-table-header">Helper (<%= @components[:component_numbers][:helper] %>)</th>
     </tr>
   </thead>
   <tbody class="govuk-table__body">
-    <% components.each do |component| %>
+    <% @components[:component_file_details].each do |component| %>
       <tr class="govuk-table__row">
         <th scope="row" class="govuk-table__header">
-          <a href="<%= component[:link] %>" class="govuk-link"><%= component[:name] %></a>
+          <a href="<%= component[:link] %>" class="govuk-link">
+            <%= component[:name] %>
+          </a>
         </th>
         <td class="govuk-table__cell">
-          <% if component[:stylesheet] %>
+          <% if component[:template_exists] %>
             <strong class="govuk-tag govuk-tag--green">
-              <a href="<%= component[:stylesheet] %>" class="govuk-link">Yes</a>
+              <a href="<%= component[:template_link] %>" class="govuk-link">
+                <%= component[:template_lines] %>
+                <span class="govuk-visually-hidden">lines of code in <%= component[:name] %> template</span>
+              </a>
             </strong>
           <% end %>
         </td>
         <td class="govuk-table__cell">
-          <% if component[:print_stylesheet] %>
+          <% if component[:stylesheet_exists] %>
             <strong class="govuk-tag govuk-tag--green">
-              <a href="<%= component[:print_stylesheet] %>" class="govuk-link">Yes</a>
+              <a href="<%= component[:stylesheet_link] %>" class="govuk-link">
+                <%= component[:stylesheet_lines] %>
+                <span class="govuk-visually-hidden">lines of code in <%= component[:name] %> stylesheet</span>
+              </a>
             </strong>
           <% end %>
         </td>
         <td class="govuk-table__cell">
-          <% if component[:javascript] %>
+          <% if component[:print_stylesheet_exists] %>
+            <strong class="govuk-tag govuk-tag--green">Yes</strong>
+          <% end %>
+        </td>
+        <td class="govuk-table__cell">
+          <% if component[:javascript_exists] %>
             <strong class="govuk-tag govuk-tag--green">
-              <a href="<%= component[:javascript] %>" class="govuk-link">Yes</a>
+              <a href="<%= component[:javascript_link] %>" class="govuk-link">
+                <%= component[:javascript_lines] %>
+                <span class="govuk-visually-hidden">lines of code in <%= component[:name] %> javascript</span>
+              </a>
             </strong>
           <% end %>
         </td>
         <td class="govuk-table__cell">
-          <% if component[:tests] %>
+          <% if component[:test_exists] %>
             <strong class="govuk-tag govuk-tag--green">
-              <a href="<%= component[:tests] %>" class="govuk-link">Yes</a>
+              <a href="<%= component[:test_link] %>" class="govuk-link">
+                <%= component[:test_lines] %>
+                <span class="govuk-visually-hidden">lines of code in <%= component[:name] %> test</span>
+              </a>
             </strong>
           <% end %>
         </td>
         <td class="govuk-table__cell">
-          <% if component[:js_tests] %>
+          <% if component[:javascript_test_exists] %>
             <strong class="govuk-tag govuk-tag--green">
-              <a href="<%= component[:js_tests] %>" class="govuk-link">Yes</a>
+              <a href="<%= component[:javascript_test_link] %>" class="govuk-link">
+                <%= component[:javascript_test_lines] %>
+                <span class="govuk-visually-hidden">lines of code in <%= component[:name] %> javascript test</span>
+              </a>
             </strong>
           <% end %>
         </td>
         <td class="govuk-table__cell">
-          <% if component[:helper] %>
+          <% if component[:helper_exists] %>
             <strong class="govuk-tag govuk-tag--green">
-              <a href="<%= component[:helper] %>" class="govuk-link">Yes</a>
+              <a href="<%= component[:helper_link] %>" class="govuk-link">
+                <%= component[:helper_lines] %>
+                <span class="govuk-visually-hidden">lines of code in <%= component[:name] %> helper</span>
+              </a>
             </strong>
           <% end %>
         </td>

data/app/views/govuk_publishing_components/audit/_components.html.erb

@@ -10,7 +10,7 @@
   <% component_items = [] %>
 
   <% component_files = capture do %>
-    <%= render partial: "component_contents", locals: { components: @components[:component_listing] } %>
+    <%= render partial: "component_contents" %>
   <% end %>
 
   <%

data/app/views/govuk_publishing_components/components/_accordion.html.erb

@@ -12,8 +12,8 @@
   accordion_classes << (shared_helper.get_margin_bottom)
 
   translations = {
-    show_text: "components.accordion.show",
-    hide_text: "components.accordion.hide",
+    show_text: "common.show",
+    hide_text: "common.hide",
     show_all_text: "components.accordion.show_all",
     hide_all_text: "components.accordion.hide_all",
     this_section_visually_hidden: "components.accordion.this_section_visually_hidden",

data/app/views/govuk_publishing_components/components/_contextual_footer.html.erb

@@ -1,10 +1,12 @@
 <% navigation = GovukPublishingComponents::Presenters::ContextualNavigation.new(content_item, request) %>
+<% ga4_tracking ||= false %>
 
 <% unless navigation.content_tagged_to_current_step_by_step? %>
   <div class="gem-c-contextual-footer">
     <%# Rendering related navigation because no step by step list %>
     <%= render 'govuk_publishing_components/components/related_navigation',
                content_item: content_item,
+               ga4_tracking: ga4_tracking,
                context: :footer %>
   </div>
 <% end %>

data/app/views/govuk_publishing_components/components/_contextual_sidebar.html.erb

@@ -1,5 +1,9 @@
-<% navigation = GovukPublishingComponents::Presenters::ContextualNavigation.new(content_item, request) %>
-<% shared_helper = GovukPublishingComponents::Presenters::SharedHelper.new(local_assigns) %>
+<%
+  ga4_tracking ||= false
+  request.query_parameters[:ga4_tracking] = ga4_tracking
+  navigation = GovukPublishingComponents::Presenters::ContextualNavigation.new(content_item, request)
+  shared_helper = GovukPublishingComponents::Presenters::SharedHelper.new(local_assigns)
+%>
 
 <div class="gem-c-contextual-sidebar">
   <% if navigation.content_tagged_to_a_reasonable_number_of_step_by_steps? %>
@@ -12,7 +16,7 @@
     <%= render 'govuk_publishing_components/components/step_by_step_nav', navigation.step_nav_helper.sidebar %>
   <% else %>
     <%# Rendering related navigation sidebar because no step by step list %>
-    <%= render 'govuk_publishing_components/components/related_navigation', content_item: content_item, context: :sidebar %>
+    <%= render 'govuk_publishing_components/components/related_navigation', content_item: content_item, context: :sidebar, ga4_tracking: ga4_tracking %>
   <% end %>
 
   <% if navigation.content_tagged_to_other_step_by_steps? %>

data/app/views/govuk_publishing_components/components/_cookie_banner.html.erb

@@ -14,9 +14,9 @@
   text = raw(text)
 
   cookie_preferences_href ||= "/help/cookies"
-  confirmation_message ||= raw(t("components.cookie_banner.confirmation_message_html",
+  confirmation_message ||= raw(t("components.cookie_banner.confirmation_message.html",
                                 link: link_to(
-                                  t("components.cookie_banner.confirmation_message_link"),
+                                  t("components.cookie_banner.confirmation_message.link"),
                                   cookie_preferences_href,
                                   class: "govuk-link",
                                   data: {
@@ -29,8 +29,8 @@
   css_classes = %w(gem-c-cookie-banner govuk-clearfix)
   css_classes << "gem-c-cookie-banner--services" if services_cookies
 %>
-<div id="<%= id %>" class="<%= css_classes.join(' ') %>" data-module="cookie-banner" role="region" aria-label="cookie banner" data-nosnippet>
-  <div class="govuk-cookie-banner js-banner-wrapper" role="region" aria-label="<%= title %>">
+<div id="<%= id %>" class="<%= css_classes.join(' ') %>" data-module="cookie-banner" data-nosnippet role="region" aria-label="<%= title %>">
+  <div class="govuk-cookie-banner js-banner-wrapper">
     <div class="gem-c-cookie-banner__message govuk-cookie-banner__message govuk-width-container">
       <div class="govuk-grid-row">
         <div class="govuk-grid-column-two-thirds">
@@ -79,13 +79,17 @@
                   "track-action": "Cookie banner rejected",
                 }
             } %>
-            <a class="govuk-link" href="<%= cookie_preferences_href %>">View cookies</a>
+            <a class="govuk-link" href="<%= cookie_preferences_href %>"><%= t("components.cookie_banner.buttons.view_cookies") %></a>
           </div>
         <% end %>
     </div>
   </div>
   <div class="gem-c-cookie-banner__confirmation govuk-width-container" tabindex="-1" hidden>
-    <p class="gem-c-cookie-banner__confirmation-message" role="alert"><%= confirmation_message %></p>
+    <p class="gem-c-cookie-banner__confirmation-message" role="alert">
+      <span class="gem-c-cookie-banner__confirmation-message__accepted" hidden><%= t("components.cookie_banner.confirmation_message.accepted") %>. </span>
+      <span class="gem-c-cookie-banner__confirmation-message__rejected" hidden><%= t("components.cookie_banner.confirmation_message.rejected") %>. </span>
+      <%= confirmation_message %>
+    </p>
     <div class="govuk-button-group">
       <button class="gem-c-cookie-banner__hide-button govuk-button" data-hide-cookie-banner="true" data-module="gem-track-click" data-track-category="cookieBanner" data-track-action="Hide cookie banner"><%= t("components.cookie_banner.hide") %></button>
     </div>

data/app/views/govuk_publishing_components/components/_error_message.html.erb

@@ -4,12 +4,13 @@
   css_classes = %w( gem-c-error-message govuk-error-message )
   css_classes << classes if classes
   items ||= []
+  right_to_left ||= false
 
   if items.any?
     errors = items.map { |item| capture { item[:text] } }
     text = raw(errors.join("<br />"))
   end
 %>
-<%= tag.p id: id, class: css_classes do %>
+<%= tag.p id: id, class: css_classes, dir: right_to_left ? "rtl" : nil do %>
   <span class="govuk-visually-hidden">Error:</span> <%= text %>
 <% end %>

data/app/views/govuk_publishing_components/components/_google_tag_manager_script.html.erb

@@ -7,7 +7,7 @@
     gtm_attributes << "gtm_preview=" + gtm_preview if gtm_preview
   gtm_attributes = gtm_attributes.join('&')
 %>
-<script>
+<%= javascript_tag nonce: true do -%>
 var doNotTrack = ( navigator.doNotTrack === '1' || navigator.doNotTrack === 'yes' || navigator.msDoNotTrack === '1' || window.doNotTrack === '1' )
 if (!doNotTrack) {
   (function(w,d,s,l,i){w[l]=w[l]||[];w[l].push({'gtm.start':
@@ -16,4 +16,4 @@ if (!doNotTrack) {
   'https://www.googletagmanager.com/gtm.js?id='+i+dl+'&<%= raw gtm_attributes %>';f.parentNode.insertBefore(j,f);
 })(window,document,'script','dataLayer','<%= gtm_id %>');
 }
-</script>
+<% end -%>

data/app/views/govuk_publishing_components/components/_hint.html.erb

@@ -1,11 +1,12 @@
 <%
   id ||= "hint-#{SecureRandom.hex(4)}"
+  right_to_left ||= false
   shared_helper = GovukPublishingComponents::Presenters::SharedHelper.new(local_assigns)
 
   css_classes = %w( gem-c-hint govuk-hint )
   css_classes << shared_helper.get_margin_bottom
 %>
 
-<%= tag.div id: id, class: css_classes do %>
+<%= tag.div id: id, class: css_classes, dir: right_to_left ? "rtl" : nil do %>
   <%= text %>
 <% end %>