]> git.openstreetmap.org Git - rails.git/commitdiff
Diary entries #index: Simplify bootstrap pagination
authorTobias <t@tobiasjordans.de>
Sat, 2 Jan 2021 21:24:46 +0000 (22:24 +0100)
committerGitHub <noreply@github.com>
Sat, 2 Jan 2021 21:24:46 +0000 (22:24 +0100)
- We can use a span instead of a disabled link as well
- Update to use double quotes everywhere

app/views/diary_entries/index.html.erb

index 28cb69b24b3aa916aa53792fac8e031f3e96695e..40c3e585e7d159f85470a5946bc2de3c2a61fad1 100644 (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 -%>
   <%= 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>