From: Andy Allan <git@gravitystorm.co.uk>
Date: Wed, 20 Jul 2022 16:36:01 +0000 (+0100)
Subject: Rework trace index to use secondary-actions
X-Git-Tag: live~2022^2
X-Git-Url: https://git.openstreetmap.org./rails.git/commitdiff_plain/4e06fe054a38e7262f14ef555de9a1639404c7ba?ds=sidebyside

Rework trace index to use secondary-actions

The border between items is familiar UI, so we can keep it, but lets
use the secondary-actions approach instead.

This avoids using left-right specific border, margin and padding
classes
---

diff --git a/app/views/traces/index.html.erb b/app/views/traces/index.html.erb
index 693de7e1a..840069a02 100644
--- a/app/views/traces/index.html.erb
+++ b/app/views/traces/index.html.erb
@@ -1,12 +1,18 @@
 <% content_for :heading_class, "pb-0" %>
 <% content_for :heading do %>
   <h1><%= @title %></h1>
-  <p>
-    <%= t(".description") %>
-    <% if params[:tag] %>
-      <%= link_to t(".remove_tag_filter", :tag => params[:tag]), { :controller => "traces", :action => "index", :display_name => nil, :tag => nil, :page => nil }, { :class => "border-left ml-2 pl-2" } %>
-    <% end %>
-  </p>
+  <nav class="secondary-actions mb-3">
+    <ul>
+      <li>
+        <%= t(".description") %>
+      </li>
+      <% if params[:tag] %>
+        <li>
+          <%= link_to t(".remove_tag_filter", :tag => params[:tag]), { :controller => "traces", :action => "index", :display_name => nil, :tag => nil, :page => nil } %>
+        </li>
+      <% end %>
+    </ul>
+  </nav>
   <ul class="nav nav-tabs flex-column flex-sm-row">
     <% if @target_user.blank? %>
       <!-- public traces -->