From: Tobias <t@tobiasjordans.de>
Date: Sat, 2 Jan 2021 21:24:46 +0000 (+0100)
Subject: Diary entries #index: Simplify bootstrap pagination
X-Git-Tag: live~2710^2~1
X-Git-Url: https://git.openstreetmap.org./rails.git/commitdiff_plain/36a41a8cc5259a8f1aa6c3708cf0d27fabfa8622

Diary entries #index: Simplify bootstrap pagination

- We can use a span instead of a disabled link as well
- Update to use double quotes everywhere
---

diff --git a/app/views/diary_entries/index.html.erb b/app/views/diary_entries/index.html.erb
index 28cb69b24..40c3e585e 100644
--- a/app/views/diary_entries/index.html.erb
+++ b/app/views/diary_entries/index.html.erb
@@ -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>