Commit `a8a55cd2`

mo <mokha@cisco.com>

2017-09-02 04:30:31

fix kaminari paging.

master

1 parent 6735009

Changed files (9)

app

views

admin

photos

index.html.erb

kaminari

_first_page.html.erb

_gap.html.erb

_last_page.html.erb

_next_page.html.erb

_page.html.erb

_paginator.html.erb

_prev_page.html.erb

shared

_paging.html.erb

@@ -3,7 +3,7 @@
     <%= render partial: "my/shared/my_nav" %>
   </div>
   <div class="col-9">
-    <h1><%= @photos.count %> photos submitted</h1>
+    <h1><%= @photos.total_count %> photos submitted</h1>
     <div class="card-columns">
       <% @photos.each do |photo| %>
         <div class="card">
@@ -29,3 +29,4 @@
     </div>
   </div>
 </div>
+<%= render "shared/paging", items: @photos %>

@@ -7,7 +7,7 @@
     remote:        data-remote
 -%>
 <% unless current_page.first? %>
-<li class="first">
-  <%= link_to_unless current_page.first?, raw(t 'views.pagination.first'), url, :remote => remote %>
+<li class="page-item">
+  <%= link_to_unless current_page.first?, raw(t 'views.pagination.first'), url, class: 'page-link', :remote => remote %>
 </li>
 <% end %>

@@ -5,4 +5,4 @@
     per_page:      number of items to fetch per page
     remote:        data-remote
 -%>
-<li class="page gap disabled"><a href="#" onclick="return false;"><%= raw(t 'views.pagination.truncate') %></a></li>
+<li class="page-item disabled"><a href="#" onclick="return false;"><%= raw(t 'views.pagination.truncate') %></a></li>

@@ -7,7 +7,7 @@
     remote:        data-remote
 -%>
 <% unless current_page.last? %>
-<li class="last next"><%# "next" class present for border styling in twitter bootstrap %>
-  <%= link_to_unless current_page.last?, raw(t 'views.pagination.last'), url, {:remote => remote} %>
+<li class="page-item"><%# "next" class present for border styling in twitter bootstrap %>
+  <%= link_to_unless current_page.last?, raw(t 'views.pagination.last'), url, remote: remote, class: 'page-link' %>
 </li>
 <% end %>

@@ -7,7 +7,7 @@
     remote:        data-remote
 -%>
 <% unless current_page.last? %>
-<li class="next_page">
-  <%= link_to_unless current_page.last?, raw(t 'views.pagination.next'), url, :rel => 'next', :remote => remote %>
+<li class="page-item">
+  <%= link_to_unless current_page.last?, raw(t 'views.pagination.next'), url, class: 'page-link', :rel => 'next', :remote => remote %>
 </li>
 <% end %>

@@ -7,6 +7,6 @@
     per_page:      number of items to fetch per page
     remote:        data-remote
 -%>
-<li class="page<%= ' active' if page.current? %>">
-  <%= link_to page, url, opts = {:remote => remote, :rel => page.next? ? 'next' : page.prev? ? 'prev' : nil} %>
+<li class="page-item<%= ' active' if page.current? %>">
+  <%= link_to page, url, opts = {:remote => remote, class: 'page-link', :rel => page.next? ? 'next' : page.prev? ? 'prev' : nil} %>
 </li>

@@ -7,8 +7,8 @@
     paginator:     the paginator that renders the pagination tags inside
 -%>
 <%= paginator.render do -%>
-  <div class="pagination">
-    <ul>
+  <nav>
+    <ul class="pagination justify-content-center">
     <%= first_page_tag unless current_page.first? %>
     <%= prev_page_tag unless current_page.first? %>
     <% each_page do |page| -%>
@@ -21,5 +21,5 @@
     <%= next_page_tag unless current_page.last? %>
     <%= last_page_tag unless current_page.last? %>
     </ul>
-  </div>
+  </nav>
 <% end -%>

@@ -7,7 +7,7 @@
     remote:        data-remote
 -%>
 <% unless current_page.first? %>
-<li class="prev">
-  <%= link_to_unless current_page.first?, raw(t 'views.pagination.previous'), url, :rel => 'prev', :remote => remote %>
+<li class="page-item">
+  <%= link_to_unless current_page.first?, raw(t 'views.pagination.previous'), url, :rel => 'prev', :remote => remote, class: 'page-link' %>
 </li>
 <% end %>

@@ -1,3 +1,1 @@
-<div class="row-fluid">
-  <div class="span12 text-center"> <%= paginate items %> </div>
-</div>
+<%= paginate items %>

Commit a8a55cd2

Commit `a8a55cd2`