From: Andy Allan <git@gravitystorm.co.uk>
Date: Wed, 6 Jan 2021 15:41:42 +0000 (+0000)
Subject: Merge branch 'pull/3035'
X-Git-Tag: live~2267
X-Git-Url: https://git.openstreetmap.org./rails.git/commitdiff_plain/85bf0af4886f155e8521a6ccffedfa845133b611?hp=9568c7fee85a615830ba78639b2ee8440da60668

Merge branch 'pull/3035'
---

diff --git a/app/assets/stylesheets/common.scss b/app/assets/stylesheets/common.scss
index d8369e3ce..0998b94a9 100644
--- a/app/assets/stylesheets/common.scss
+++ b/app/assets/stylesheets/common.scss
@@ -1273,10 +1273,6 @@ tr.turn:hover {
   }
 }
 
-.pagination {
-  padding-top: $lineheight;
-}
-
 /* Rules for the diary entry page */
 
 .diary_entries {
diff --git a/app/views/traces/_trace_paging_nav.html.erb b/app/views/traces/_trace_paging_nav.html.erb
index 4fcc7b4d8..25e35d226 100644
--- a/app/views/traces/_trace_paging_nav.html.erb
+++ b/app/views/traces/_trace_paging_nav.html.erb
@@ -1,18 +1,27 @@
-<p>
+<nav>
+  <ul class="pagination">
+    <% if page > 1 %>
+      <li class="page-item">
+        <%= link_to t(".newer"), params.merge(:page => page - 1), :class => "page-link" %>
+      </li>
+    <% else %>
+      <li class="page-item disabled">
+        <span class="page-link"><%= t(".newer") %></span>
+      </li>
+    <% end %>
 
-<% if @traces.size > 1 %>
-<% if @page > 1 %>
-<%= link_to t(".newer"), @params.merge(:page => @page - 1) %>
-<% else %>
-<%= t(".newer") %>
-<% end %>
+    <li class="page-item disabled">
+      <span class="page-link"><%= t(".showing_page", :page => page) %></span>
+    </li>
 
-| <%= t(".showing_page", :page => @page) %> |
-
-<% if @traces.size < @page_size %>
-<%= t(".older") %>
-<% else %>
-<%= link_to t(".older"), @params.merge(:page => @page + 1) %>
-<% end %>
-<% end %>
-</p>
+    <% if traces.size < page_size %>
+      <li class="page-item disabled">
+        <span class="page-link"><%= t(".older") %></span>
+      </li>
+    <% else %>
+      <li class="page-item">
+        <%= link_to t(".older"), params.merge(:page => page + 1), :class => "page-link" %>
+      </li>
+    <% end %>
+  </ul>
+</nav>
diff --git a/app/views/traces/index.html.erb b/app/views/traces/index.html.erb
index 609dbcede..fa69c48a2 100644
--- a/app/views/traces/index.html.erb
+++ b/app/views/traces/index.html.erb
@@ -23,7 +23,7 @@
 <% end %>
 
 <% if @traces.size > 0 %>
-  <%= render :partial => "trace_paging_nav" %>
+  <%= render "trace_paging_nav", :page => @page, :page_size => @page_size, :traces => @traces, :params => @params %>
 
   <table id="trace_list" class="table table-borderless table-striped">
     <tbody>
@@ -31,7 +31,7 @@
     </tbody>
   </table>
 
-  <%= render :partial => "trace_paging_nav" %>
+  <%= render "trace_paging_nav", :page => @page, :page_size => @page_size, :traces => @traces, :params => @params %>
 <% else %>
   <h4><%= t ".empty_html", :upload_link => new_trace_path %></h4>
 <% end %>