From 0eba0e8937cd4aec88d38613e5866ae25c85aa2f Mon Sep 17 00:00:00 2001
From: maxgds <max.harden@digital.cabinet-office.gov.uk>
Date: Wed, 14 Oct 2020 14:56:33 +0100
Subject: [PATCH] Replace bespoke nav with contents list component

---
 app/assets/stylesheets/helpers/_parts.scss     | 12 +++---------
 app/views/content_items/travel_advice.html.erb | 10 +---------
 test/integration/travel_advice_test.rb         |  1 -
 3 files changed, 4 insertions(+), 19 deletions(-)

diff --git a/app/assets/stylesheets/helpers/_parts.scss b/app/assets/stylesheets/helpers/_parts.scss
index 39c653e45..9089f982b 100644
--- a/app/assets/stylesheets/helpers/_parts.scss
+++ b/app/assets/stylesheets/helpers/_parts.scss
@@ -14,16 +14,10 @@
   }
 
   .part-navigation {
-    margin-bottom: 0;
-    @include govuk-font(19);
+    margin-left: 0;
 
-    @include govuk-media-query($from: tablet) {
-      @include govuk-font(16);
-    }
-
-    li {
-      list-style: none;
-      margin-bottom: 0.75em;
+    @include govuk-media-query($until: tablet) {
+      margin-left: govuk-spacing(1);
     }
   }
 
diff --git a/app/views/content_items/travel_advice.html.erb b/app/views/content_items/travel_advice.html.erb
index b86d751c9..c564f4950 100644
--- a/app/views/content_items/travel_advice.html.erb
+++ b/app/views/content_items/travel_advice.html.erb
@@ -36,15 +36,7 @@
 
     <aside class="part-navigation-container" role="complementary">
       <nav role="navigation" class="govuk-grid-row part-navigation" aria-label="Travel advice pages" data-module="track-click">
-        <%  @content_item.parts_navigation.each_with_index do |part_group, i| %>
-          <ol class="govuk-grid-column-one-half" <% if i == 1 %> start="<%=  @content_item.parts_navigation_second_list_start %>" <% end %>>
-            <% part_group.each do |part_nav_item| %>
-              <li>
-                <%= part_nav_item %>
-              </li>
-            <% end %>
-          </ol>
-        <% end %>
+        <%= render "govuk_publishing_components/components/contents_list", contents: @content_item.part_link_elements, underline_links: true %>
       </nav>
 
       <%= render 'govuk_publishing_components/components/subscription-links', email_signup_link: @content_item.email_signup_link, feed_link: @content_item.feed_link %>
diff --git a/test/integration/travel_advice_test.rb b/test/integration/travel_advice_test.rb
index 605d7e585..cb0011417 100644
--- a/test/integration/travel_advice_test.rb
+++ b/test/integration/travel_advice_test.rb
@@ -14,7 +14,6 @@ class TravelAdviceTest < ActionDispatch::IntegrationTest
     assert page.has_css?("a[href=\"#{@content_item['details']['email_signup_link']}\"]", text: "Get email alerts")
     assert page.has_css?("a[href=\"#{@content_item['base_path']}.atom\"]", text: "Subscribe to feed")
 
-    assert page.has_css?(".part-navigation ol", count: 2)
     assert page.has_css?(".part-navigation li", count: @content_item["details"]["parts"].size + 1)
     assert page.has_css?(".part-navigation li", text: "Summary")
     assert_not page.has_css?(".part-navigation li a", text: "Summary")