@tyrasd commented on this pull request.
> @@ -36,7 +36,7 @@
<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 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 || {}) %>
+ <%= link_to label, path, **(options || {}), :class => ["nav-link",
"px-1", "py-0", current_page?(path) ? "active text-secondary-emphasis" :
"text-secondary"] + (options&.dig(:class) || []) %>
thanks for the suggestion! applied :heavy_check_mark:
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/7062#discussion_r3207896527
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/7062/review/[email protected]>_______________________________________________
rails-dev mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/rails-dev
