From c6ce1958e1b12b407b9f553caaef82ea8cf6ac78 Mon Sep 17 00:00:00 2001
From: Roman Deev <roman.deev06@gmail.com>
Date: Wed, 21 Jan 2026 18:37:56 +0300
Subject: [PATCH] More padding for menu items on mobile devices

---
 app/assets/javascripts/application.js | 4 ++--
 app/assets/stylesheets/common.scss    | 4 ----
 app/views/layouts/_header.html.erb    | 8 ++++----
 3 files changed, 6 insertions(+), 10 deletions(-)

diff --git a/app/assets/javascripts/application.js b/app/assets/javascripts/application.js
index 499faa6a1..03fd908ee 100644
--- a/app/assets/javascripts/application.js
+++ b/app/assets/javascripts/application.js
@@ -157,7 +157,7 @@ $(function () {
   function expandSecondaryMenuItem($item) {
     $item.children("a")
       .removeClass("dropdown-item")
-      .addClass("nav-link")
+      .addClass("nav-link px-1 py-0")
       .addClass(function () {
         return $(this).hasClass("active") ? "text-secondary-emphasis" : "text-secondary";
       });
@@ -168,7 +168,7 @@ $(function () {
   function collapseSecondaryMenuItem($item) {
     $item.children("a")
       .addClass("dropdown-item")
-      .removeClass("nav-link text-secondary text-secondary-emphasis");
+      .removeClass("nav-link px-1 py-0 text-secondary text-secondary-emphasis");
     $item.removeClass("nav-item").appendTo($collapsedSecondaryMenu);
     toggleCompactSecondaryNav();
   }
diff --git a/app/assets/stylesheets/common.scss b/app/assets/stylesheets/common.scss
index fa4c801cb..fdfb37cb5 100644
--- a/app/assets/stylesheets/common.scss
+++ b/app/assets/stylesheets/common.scss
@@ -129,10 +129,6 @@ nav.primary {
 }
 
 nav.secondary {
-  .nav-link {
-    padding: 0 0.3rem;
-  }
-
   .user-menu, .login-menu {
     width: 100%;
   }
diff --git a/app/views/layouts/_header.html.erb b/app/views/layouts/_header.html.erb
index ec2354876..451cd0bb4 100644
--- a/app/views/layouts/_header.html.erb
+++ b/app/views/layouts/_header.html.erb
@@ -33,12 +33,12 @@
   <nav class="secondary d-flex flex-column flex-md-row gap-2 flex-grow-1 align-items-center">
     <ul id="secondary-nav-menu" class="nav flex-grow-1 justify-content-center justify-content-md-start" data-turbo-permanent>
       <% secondary_nav_items.each do |path, label, options| %>
-        <li class="nav-item">
-          <%= link_to label, path, :class => ["nav-link", current_page?(path) ? "active text-secondary-emphasis" : "text-secondary"], **(options || {}) %>
+        <li class="nav-item pb-2 pb-md-0">
+          <%= link_to label, path, :class => ["nav-link", "px-1", "py-0", current_page?(path) ? "active text-secondary-emphasis" : "text-secondary"], **(options || {}) %>
         </li>
       <% end %>
-      <li id="compact-secondary-nav" class="dropdown nav-item ms-auto collapse">
-        <button class="dropdown-toggle nav-link btn btn-outline-secondary border-0 bg-body text-secondary" type="button" data-bs-toggle="dropdown"><%= t ".more" %></button>
+      <li id="compact-secondary-nav" class="dropdown nav-item ms-auto pb-2 pb-md-0 collapse">
+        <button class="dropdown-toggle nav-link btn btn-outline-secondary px-1 py-0 border-0 bg-body text-secondary" type="button" data-bs-toggle="dropdown"><%= t ".more" %></button>
         <ul class="dropdown-menu">
         </ul>
       </li>
-- 
2.47.3