@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/[email protected]>
_______________________________________________
rails-dev mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/rails-dev

Reply via email to