[openstreetmap/openstreetmap-website] More padding for menu items on mobile devices (PR #6730)

[Pablo Brasero]

@pablobm commented on this pull request.



>        <% if Settings.status != "database_offline" && can?(:index, Issue) %>
-        <li class="nav-item">
+        <li class="nav-item pb-2 pb-md-0">
           <%= link_to issues_path(:status => "open"), :class => header_nav_link_class(issues_path) do %>
             <%= t("layouts.issues") %>
             <%= open_issues_count %>
           <% end -%>
         </li>
       <% end %>
-      <li class="nav-item">
+      <li class="nav-item pb-2 pb-md-0">
         <%= link_to t("layouts.history"), history_path, :class => header_nav_link_class(history_path) %>
       </li>
-      <li class="nav-item">
+      <li class="nav-item pb-2 pb-md-0">
         <%= link_to t("layouts.export"), export_path, :class => header_nav_link_class(export_path) %>
       </li>
-      <li class="nav-item">
+      <li class="nav-item pb-2 pb-md-0">
         <%= link_to t("layouts.gps_traces"), traces_path, :class => header_nav_link_class(traces_path) %>
       </li>
-      <li class="nav-item">
+      <li class="nav-item pb-2 pb-md-0">
         <%= link_to t("layouts.user_diaries"), diary_entries_path, :class => header_nav_link_class(diary_entries_path) %>
       </li>
-      <li class="nav-item">
+      <li class="nav-item pb-2 pb-md-0">
         <%= link_to t("layouts.communities"), communities_path, :class => header_nav_link_class(communities_path) %>
       </li>
-      <li class="nav-item">
+      <li class="nav-item pb-2 pb-md-0">
         <%= link_to t("layouts.copyright"), copyright_path, :class => header_nav_link_class(copyright_path) %>
       </li>
-      <li class="nav-item">
+      <li class="nav-item pb-2 pb-md-0">
         <%= link_to t("layouts.help"), help_path, :class => header_nav_link_class(help_path) %>
       </li>
-      <li class="nav-item">
+      <li class="nav-item pb-2 pb-md-0">
         <%= link_to t("layouts.about"), about_path, :class => header_nav_link_class(about_path) %>
       </li>

Not a fan of so much code in the template, but perhaps if we move `secondary_nav_items` into a helper, it can work.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6730#discussion_r2758862619
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/6730/review/3745019397 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20260203/39c5bca9/attachment-0001.htm>