Refactor rendering

app/helpers/notifications_helper.rb

@@ -14,4 +14,34 @@ module NotificationsHelper
       Notification.newest_active_unauthenticated_notification
     end
   end
+
+  def render_for_banner(title)
+    @@banner_renderer ||= NotificationTitleRenderer.new({ invert_link_colour: true, bold_all_text: true })
+    @@banner_markdown ||= Redcarpet::Markdown.new(@@banner_renderer, no_intra_emphasis: true)
+    @@banner_markdown.render(title)
+  end
+
+  def render_for_summary(title)
+    @@plain_title_renderer ||= NotificationTitleRenderer.new({ invert_link_colour: false, bold_all_text: false })
+    @@plain_title_markdown ||= Redcarpet::Markdown.new(@@plain_title_renderer, no_intra_emphasis: true)
+    @@plain_title_markdown.render(title)
+  end
+
+private
+
+  class NotificationTitleRenderer < Redcarpet::Render::HTML
+    def initialize(options = {})
+      link_class = "govuk-link"
+      link_class += " govuk-link--inverse" if options[:invert_link_colour]
+      @bold = options[:bold_all_text]
+      base_options = { escape_html: true, safe_links_only: true, link_attributes: { class: link_class } }
+      super base_options
+    end
+
+    def paragraph(text)
+      return %(<p class="govuk-!-font-weight-bold">#{text}</p>) if @bold
+
+      %(<p>#{text}</p>)
+    end
+  end
 end

app/models/notification.rb

@@ -19,19 +19,3 @@ class Notification < ApplicationRecord
     Redcarpet::Markdown.new(renderer, no_intra_emphasis: true).render(title)
   end
 end
-
-class NotificationTitleRenderer < Redcarpet::Render::HTML
-  def initialize(options = {})
-    link_class = "govuk-link"
-    link_class += " govuk-link--inverse" if options[:invert_link_colour]
-    @bold = options[:bold_all_text]
-    base_options = { escape_html: true, safe_links_only: true, link_attributes: { class: link_class } }
-    super base_options
-  end
-
-  def paragraph(text)
-    return %(<p class="govuk-!-font-weight-bold">#{text}</p>) if @bold
-
-    %(<p>#{text}</p>)
-  end
-end

app/views/notifications/_notification_banner.html.erb

@@ -6,7 +6,7 @@
         <% if notification_count > 1  && current_user.present? %>
           <p>Notification 1 of <%= notification_count %></p>
         <% end %>
-      <%== notification.rendered_title(invert_link_colour: true, bold_all_text: true) %>
+      <%== render_for_banner(notification.title) %>
         <% if notification.page_content.present? %>
           <div class="govuk-body">
             <%= govuk_link_to notification.link_text, notification_path(notification), class: "govuk-link--inverse govuk-!-font-weight-bold" %>

app/views/notifications/check_answers.html.erb

@@ -22,7 +22,7 @@
             <% summary_list.with_row do |row| %>
               <% row.with_key { "Title" } %>
               <% row.with_value do %>
-                <%== @notification.rendered_title %>
+                <%== render_for_summary(@notification.title) %>
               <% end %>
               <% row.with_action(text: "Change", href: edit_notification_path(@notification)) %>
             <% end %>