cst1/openstreetmap-website
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Diary entries #index: Migrate to bootstrap pagination

Browse source 
Use https://getbootstrap.com/docs/4.5/components/pagination/ for the pagination.

I tried using link_to_if but this does not work well in this case since multiple changes need to happen for the disabled state.
This commit is contained in:
Tobias Jordans 2020-12-30 14:20:13 +01:00
parent e392556444
commit abc83298cd
1 changed files with 22 additions and 14 deletions
Download patch file Download diff file Expand all files Collapse all files

36
app/views/diary_entries/index.html.erb
View file

@ -28,21 +28,29 @@
<%= render @entries %>
<div class="pagination">
<% if @entries.size < @page_size -%>
<%= t(".older_entries") %>
<% else -%>
<%= link_to t(".older_entries"), @params.merge(:page => @page + 1) %>
<% end -%>
<nav>
<ul class='pagination'>
<% if @entries.size >= @page_size -%>
<li class='page-item'>
<%= link_to t('.older_entries'), @params.merge(:page => @page + 1), :class => 'page-link' %>
</li>
<% else -%>
<li class='page-item disabled'>
<%= link_to t('.older_entries'), '#', :class => 'page-link', :tabindex => '-1', :'aria-disabled' => true %>
</li>
<% end -%>
|
<% if @page > 1 -%>
<%= link_to t(".newer_entries"), @params.merge(:page => @page - 1) %>
<% else -%>
<%= t(".newer_entries") %>
<% end -%>
</div>
<% if @page > 1 -%>
<li class='page-item'>
<%= link_to_if @page > 1, t(".newer_entries"), @params.merge(:page => @page - 1), :class => 'page-link' %>
</li>
<% else -%>
<li class='page-item disabled'>
<%= link_to t('.newer_entries'), '#', :class => 'page-link', :tabindex => '-1', :'aria-disabled' => true %>
</li>
<% end -%>
</ul>
</nav>
<% end %>
<% unless params[:friends] or params[:nearby] -%>