fixup! CLDC-4365: Use a ternary class for two thirds conditional

fixup! CLDC-4365: Fix notification banner call
CLDC-4365: Fix notification banner call
2 changed files with 7 additions and 10 deletions
--- a/app/helpers/application_helper.rb
+++ b/app/helpers/application_helper.rb
@ -35,7 +35,8 @@ module ApplicationHelper
  def notification_banner
    govuk_notification_banner(
      title_text: "Success",
-      success: true, title_heading_level: 3,
+      success: true,
+      title_heading_level: 3,
      title_id: "flash-notice",
      role: "alert",
    ) do |notification_banner|
--- a/app/views/layouts/application.html.erb
+++ b/app/views/layouts/application.html.erb
@ -120,15 +120,11 @@

      <main class="govuk-main-wrapper govuk-main-wrapper--auto-spacing" id="main-content" role="main">
        <% if flash.notice && !flash.notice.include?("translation missing") %>
-          <% if flash[:notification_banner_two_thirds] %>
          <div class="govuk-grid-row">
-              <div class="govuk-grid-column-two-thirds-from-desktop">
+            <div class="<%= flash[:notification_banner_two_thirds] ? "govuk-grid-column-two-thirds-from-desktop" : "govuk-grid-column-full" %>">
              <%= notification_banner %>
            </div>
          </div>
-          <% else %>
-            <%= notification_banner %>
-          <% end %>
        <% end %>
        <%= content_for?(:content) ? yield(:content) : yield %>
      </main>
Author	SHA1	Message	Date
samyou-softwire	ac5063bced	fixup! CLDC-4365: Use a ternary class for two thirds conditional	4 weeks ago
samyou-softwire	ae2114fb23	fixup! CLDC-4365: Fix notification banner call	4 weeks ago
samyou-softwire	136d0f2e56	CLDC-4365: Fix notification banner call	4 weeks ago
samyou-softwire	25cb48980d	CLDC-4365: Use a ternary class for two thirds conditional	4 weeks ago
samyou-softwire	8128dae473	CLDC-4365: Make the interruptuon screen notice two thirds	4 weeks ago
samyou-softwire	d53f12a586	CLDC-4365: Allow for two thirds notifications	4 weeks ago