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

Diary entries #index: Simplify bootstrap pagination

Browse source 
- We can use a span instead of a disabled link as well
- Update to use double quotes everywhere
This commit is contained in:
Tobias 2021-01-02 22:24:46 +01:00 • committed by GitHub
parent abc83298cd
commit 36a41a8cc5
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 10 additions and 10 deletions
Download patch file Download diff file Expand all files Collapse all files

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

@ -5,7 +5,7 @@
<% end %>
<h1><%= @title %></h1>
<ul class='secondary-actions clearfix'>
<ul class="secondary-actions clearfix">
<% unless params[:friends] or params[:nearby] -%>
<li><%= rss_link_to :action => "rss", :language => params[:language] %></li>
<% end -%>
@ -29,24 +29,24 @@
<%= render @entries %>
<nav>
<ul class='pagination'>
<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 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 class="page-item disabled">
<span class="page-link"><%= t(".older_entries") %></span>
</li>
<% end -%>
<% if @page > 1 -%>
<li class='page-item'>
<%= link_to_if @page > 1, t(".newer_entries"), @params.merge(:page => @page - 1), :class => 'page-link' %>
<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 class="page-item disabled">
<span class="page-link"><%= t(".newer_entries") %></span>
</li>
<% end -%>
</ul>