From e9148e56eb331038df29feec091ebe571326e92c Mon Sep 17 00:00:00 2001
From: Anton Khorev <tony29@yandex.ru>
Date: Tue, 3 Sep 2024 02:10:43 +0300
Subject: [PATCH] Use svg icons in pagination for note lists

---
 app/views/notes/_notes_paging_nav.html.erb | 19 ++++++++++++++-----
 config/locales/en.yml                      |  4 ++--
 2 files changed, 16 insertions(+), 7 deletions(-)

diff --git a/app/views/notes/_notes_paging_nav.html.erb b/app/views/notes/_notes_paging_nav.html.erb
index cb8591246..5209256d6 100644
--- a/app/views/notes/_notes_paging_nav.html.erb
+++ b/app/views/notes/_notes_paging_nav.html.erb
@@ -1,26 +1,35 @@
 <nav>
+  <% link_class = "page-link icon-link text-center text-nowrap" %>
   <ul class="pagination">
+    <% previous_link_content = capture do %>
+      <%= previous_page_svg_tag :class => "flex-shrink-0" %>
+      <%= t(".previous") %>
+    <% end %>
     <% if @page > 1 %>
       <li class="page-item">
-        <%= link_to t(".previous"), @params.merge(:page => @page - 1), :class => "page-link" %>
+        <%= link_to previous_link_content, @params.merge(:page => @page - 1), :class => link_class %>
       </li>
     <% else %>
       <li class="page-item disabled">
-        <%= tag.span t(".previous"), :class => "page-link" %>
+        <%= tag.span previous_link_content, :class => link_class %>
       </li>
     <% end %>
 
     <li class="page-item active">
-      <%= tag.span t(".showing_page", :page => @page), :class => "page-link" %>
+      <%= tag.span t(".showing_page", :page => @page), :class => link_class %>
     </li>
 
+    <% next_link_content = capture do %>
+      <%= t(".next") %>
+      <%= next_page_svg_tag :class => "flex-shrink-0" %>
+    <% end %>
     <% if @notes.size < @page_size %>
       <li class="page-item disabled">
-        <%= tag.span t(".next"), :class => "page-link" %>
+        <%= tag.span next_link_content, :class => link_class %>
       </li>
     <% else %>
       <li class="page-item">
-        <%= link_to t(".next"), @params.merge(:page => @page + 1), :class => "page-link" %>
+        <%= link_to next_link_content, @params.merge(:page => @page + 1), :class => link_class %>
       </li>
     <% end %>
   </ul>
diff --git a/config/locales/en.yml b/config/locales/en.yml
index d9bb56fbf..3d27d0256 100644
--- a/config/locales/en.yml
+++ b/config/locales/en.yml
@@ -3027,8 +3027,8 @@ en:
       add: Add Note
     notes_paging_nav:
       showing_page: "Page %{page}"
-      next: "Next »"
-      previous: "« Previous"
+      next: "Next"
+      previous: "Previous"
   javascripts:
     close: Close
     share:
-- 
2.39.5