From 6fd66b578cecf214172bb065342a9cd194d4cb10 Mon Sep 17 00:00:00 2001
From: Vanita Barrett <vanita.barrett@digital.cabinet-office.gov.uk>
Date: Thu, 9 Aug 2018 14:12:30 +0000
Subject: [PATCH] Fix branding

Moves the branding class to the document list item instead of the ul which may not always be rendered.
---
 .../components/_document_list.html.erb                        | 4 ++--
 spec/components/document_list_spec.rb                         | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/app/views/govuk_publishing_components/components/_document_list.html.erb b/app/views/govuk_publishing_components/components/_document_list.html.erb
index 3d3703153f..5d8eba31e8 100644
--- a/app/views/govuk_publishing_components/components/_document_list.html.erb
+++ b/app/views/govuk_publishing_components/components/_document_list.html.erb
@@ -11,10 +11,10 @@
 %>
 <% if items.any? %>
   <% unless within_multitype_list %>
-    <ol class="gem-c-document-list<%= margin_bottom_class %><%= margin_top_class %> <%= brand_helper.brand_class %>">
+    <ol class="gem-c-document-list<%= margin_bottom_class %><%= margin_top_class %>">
   <% end %>
     <% items.each do |item| %>
-      <li class="gem-c-document-list__item<%= within_multitype_list_class %>">
+      <li class="gem-c-document-list__item<%= within_multitype_list_class %> <%= brand_helper.brand_class %>">
         <%=
           link_to(
             item[:link][:text],
diff --git a/spec/components/document_list_spec.rb b/spec/components/document_list_spec.rb
index e75d84634d..416857f3d8 100644
--- a/spec/components/document_list_spec.rb
+++ b/spec/components/document_list_spec.rb
@@ -149,7 +149,7 @@ def component_name
       ]
     )
 
-    assert_select '.gem-c-document-list.brand--attorney-generals-office'
+    assert_select '.gem-c-document-list__item.brand--attorney-generals-office'
     assert_select '.gem-c-document-list .gem-c-document-list__item-title.brand__color'
   end