From 8ec14eb06b04ebc65dbfb6831b8db285d9d7c71c Mon Sep 17 00:00:00 2001
From: Anton Khorev <tony29@yandex.ru>
Date: Sun, 28 Apr 2024 04:02:59 +0300
Subject: [PATCH] Add tooltips, make newest/oldest buttons tooltip only

---
 app/helpers/pagination_helper.rb      |  3 ++-
 app/views/shared/_pagination.html.erb | 14 ++++++++------
 2 files changed, 10 insertions(+), 7 deletions(-)

diff --git a/app/helpers/pagination_helper.rb b/app/helpers/pagination_helper.rb
index c75c6c2a56..46b49a90f8 100644
--- a/app/helpers/pagination_helper.rb
+++ b/app/helpers/pagination_helper.rb
@@ -1,11 +1,12 @@
 module PaginationHelper
-  def pagination_item(params, &block)
+  def pagination_item(params, title, &block)
     link_class = "page-link icon-link text-center"
     page_link_content = capture(&block)
     if params
       page_link = link_to page_link_content,
                           params,
                           :class => link_class,
+                          :title => title,
                           :data => { "turbo" => true, "turbo-frame" => "pagination", "turbo-action" => "advance" }
       tag.li page_link, :class => "page-item d-flex"
     else
diff --git a/app/views/shared/_pagination.html.erb b/app/views/shared/_pagination.html.erb
index f4670aaab0..9bb0de60cf 100644
--- a/app/views/shared/_pagination.html.erb
+++ b/app/views/shared/_pagination.html.erb
@@ -3,24 +3,26 @@
 <% translation_scope ||= "shared.pagination.#{controller.controller_name}" %>
 <nav class="d-flex justify-content-between gap-2">
   <ul class="pagination">
-    <%= pagination_item(newer_id && @params.merge(:before => nil, :after => nil)) do %>
+    <%= pagination_item(newer_id && @params.merge(:before => nil, :after => nil),
+                        t(:newest, :scope => translation_scope)) do %>
       <%= previous_page_svg_tag :class => "flex-shrink-0", :count => 2 %>
-      <span class="d-none d-md-block"><%= t :newest, :scope => translation_scope %></span>
     <% end %>
   </ul>
   <ul class="pagination">
-    <%= pagination_item(newer_id && @params.merge(:before => nil, :after => newer_id)) do %>
+    <%= pagination_item(newer_id && @params.merge(:before => nil, :after => newer_id),
+                        t(:newer, :scope => translation_scope)) do %>
       <%= previous_page_svg_tag :class => "flex-shrink-0" %>
       <span class="d-none d-sm-block"><%= t :newer, :scope => translation_scope %></span>
     <% end %>
-    <%= pagination_item(older_id && @params.merge(:before => older_id, :after => nil)) do %>
+    <%= pagination_item(older_id && @params.merge(:before => older_id, :after => nil),
+                        t(:older, :scope => translation_scope)) do %>
       <span class="d-none d-sm-block"><%= t :older, :scope => translation_scope %></span>
       <%= next_page_svg_tag :class => "flex-shrink-0" %>
     <% end %>
   </ul>
   <ul class="pagination">
-    <%= pagination_item(older_id && @params.merge(:before => nil, :after => 0)) do %>
-      <span class="d-none d-md-block"><%= t :oldest, :scope => translation_scope %></span>
+    <%= pagination_item(older_id && @params.merge(:before => nil, :after => 0),
+                        t(:oldest, :scope => translation_scope)) do %>
       <%= next_page_svg_tag :class => "flex-shrink-0", :count => 2 %>
     <% end %>
   </ul>