From cc5603646e0f99ddcf151a0df437652a2a2c0059 Mon Sep 17 00:00:00 2001
From: Elia Schito <elia@schito.me>
Date: Mon, 16 Oct 2023 16:58:52 +0200
Subject: [PATCH] Extract the table toolbar to an internal component

---
 .../solidus_admin/ui/table/component.html.erb | 22 +++++++------------
 .../ui/table/toolbar/component.rb             | 14 ++++++++++++
 2 files changed, 22 insertions(+), 14 deletions(-)
 create mode 100644 admin/app/components/solidus_admin/ui/table/toolbar/component.rb

diff --git a/admin/app/components/solidus_admin/ui/table/component.html.erb b/admin/app/components/solidus_admin/ui/table/component.html.erb
index df6f1474c89..736c2ae1ecb 100644
--- a/admin/app/components/solidus_admin/ui/table/component.html.erb
+++ b/admin/app/components/solidus_admin/ui/table/component.html.erb
@@ -11,14 +11,8 @@
     <%= component("ui/table/ransack_filter").stimulus_id %>:showSearch-><%= stimulus_id %>#showSearch
   "
 >
-  <% toolbar_classes = "
-    h-14 p-2 bg-white border-b border-gray-100
-    justify-start items-center gap-2
-    visible:flex hidden:hidden
-    rounded-t-lg
-  " %>
   <div role="search">
-    <div class="<%= toolbar_classes %>" data-<%= stimulus_id %>-target="searchToolbar">
+    <%= render component("ui/table/toolbar").new("data-#{stimulus_id}-target": "searchToolbar") do %>
       <%= form_with(
         url: @search_url,
         method: :get,
@@ -47,17 +41,17 @@
           "data-action": "#{stimulus_id}#cancelSearch",
         ) %>
       </div>
-    </div>
+    <% end %>
 
     <% if @filters.any? %>
-      <div class="<%= toolbar_classes %>" data-<%= stimulus_id %>-target="filterToolbar">
+      <%= render component("ui/table/toolbar").new("data-#{stimulus_id}-target": "filterToolbar") do %>
         <% @filters.each_with_index do |filter, index| %>
           <%= render_ransack_filter_dropdown(filter, index) %>
         <% end %>
-      </div>
+      <% end %>
     <% end %>
 
-    <div class="<%= toolbar_classes %>" data-<%= stimulus_id %>-target="scopesToolbar">
+    <%= render component("ui/table/toolbar").new("data-#{stimulus_id}-target": "scopesToolbar") do %>
       <div class="flex-grow">
         <%= render component("ui/tab").new(text: "All", current: true, href: "") %>
       </div>
@@ -68,15 +62,15 @@
         scheme: :secondary,
         "data-action": "#{stimulus_id}#showSearch",
       ) %>
-    </div>
+    <% end %>
   </div>
 
-  <div class="<%= toolbar_classes %>" data-<%= stimulus_id %>-target="batchToolbar" role="toolbar" aria-label="<%= t(".batch_actions") %>">
+  <%= render component("ui/table/toolbar").new("data-#{stimulus_id}-target": "batchToolbar", role: "toolbar", "aria-label": t(".batch_actions")) do %>
     <%= form_tag '', id: batch_actions_form_id %>
     <% @batch_actions.each do |batch_action| %>
       <%= render_batch_action_button(batch_action) %>
     <% end %>
-  </div>
+  <% end %>
 
   <%= turbo_frame_tag table_frame_id, target: "_top" do %>
     <table class="table-fixed w-full border-collapse">
diff --git a/admin/app/components/solidus_admin/ui/table/toolbar/component.rb b/admin/app/components/solidus_admin/ui/table/toolbar/component.rb
new file mode 100644
index 00000000000..70e287ef759
--- /dev/null
+++ b/admin/app/components/solidus_admin/ui/table/toolbar/component.rb
@@ -0,0 +1,14 @@
+# frozen_string_literal: true
+
+class SolidusAdmin::UI::Table::Toolbar::Component < SolidusAdmin::BaseComponent
+  erb_template <<~ERB
+    <div class="
+      h-14 p-2 bg-white border-b border-gray-100
+      justify-start items-center gap-2
+      visible:flex hidden:hidden
+      rounded-t-lg
+    ">
+      <%= content %>
+    </div>
+  ERB
+end