<%
  items ||= []
  see_more_link ||= false
  small ||= false
  margin_bottom ||= false
  component_size = "app-c-topic-list--small" if small
  margin_bottom_class = "app-c-topic-list--margin-bottom" if margin_bottom

  brand ||= false
  brand_helper = GovukPublishingComponents::AppHelpers::BrandHelper.new(brand)
%>
<% if items.any? %>
  <ol class="app-c-topic-list <%= component_size %> <%= brand_helper.brand_class %> <%= margin_bottom_class %>">
    <% items.each do |item| %>
      <li class="app-c-topic-list__item">
        <%=
          link_to(
            item[:text],
            item[:path],
            data: item[:data_attributes],
            "aria-label": item[:aria_label],
            class: "govuk-link app-c-topic-list__link app-c-topic-list__link--no-underline #{brand_helper.color_class}"
          )
        %>
      </li>
    <% end %>
    <% if see_more_link %>
      <li class="app-c-topic-list__item">
        <%=
          link_to(
            see_more_link[:text],
            see_more_link[:path],
            data: see_more_link[:data_attributes],
            class: "govuk-link app-c-topic-list__link #{brand_helper.color_class}"
          )
        %>
      </li>
    <% end %>
  </ol>
<% end %>

<%
  items ||= []
  see_more_link ||= false
  small ||= false
  margin_bottom ||= false
  component_size = "app-c-topic-list--small" if small
  margin_bottom_class = "app-c-topic-list--margin-bottom" if margin_bottom

  brand ||= false
  brand_helper = GovukPublishingComponents::AppHelpers::BrandHelper.new(brand)
%>
<% if items.any? %>
  <%= tag.ul(class: ["app-c-topic-list", component_size, brand_helper.brand_class, margin_bottom_class]) do %>
    <% items.each do |item| %>
      <li class="app-c-topic-list__item">
        <%=
          link_to(
            item[:text],
            item[:path],
            data: item[:data_attributes],
            "aria-label": item[:aria_label],
            class: "govuk-link app-c-topic-list__link app-c-topic-list__link--no-underline #{brand_helper.color_class}"
          )
        %>
      </li>
    <% end %>
    <% if see_more_link %>
      <li class="app-c-topic-list__item">
        <%=
          link_to(
            see_more_link[:text],
            see_more_link[:path],
            data: see_more_link[:data_attributes],
            class: "govuk-link app-c-topic-list__link #{brand_helper.color_class}"
          )
        %>
      </li>
    <% end %>
  <% end %>
<% end %>

Change topic-list list type from ol to ul
Change topic-list list type from ol to ul

I've inspected each use of the topic-list component in this app. There is no
clear benefit as far as I can see to having these as an ordered list.

Org pages use it to present lists of corporate information links. They are
not in any discernable order, and it would not make sense to number them.

Org and courts pages use it to present featured links. They are short lists
of "useful" things, but there's no real order to them.

The same goes for [topic pages](gov.uk/topic/benefits-credits/child-benefit)
and the finders list at the foot of the [transition landing page](https://www.gov.uk/transition).

These are failures of WCAG SC 1.3.1

https://trello.com/c/O3Mck5D5/441-transition-page-incorrect-list-type
