spree_admin 5.0.0.rc1 → 5.0.1

data/app/views/spree/admin/digital_assets/new.html.erb

@@ -1,20 +1,12 @@
 <%= turbo_frame_tag :dialog_modal do %>
   <%= form_for :digital_asset, url: spree.admin_product_digital_assets_path(@product) do |f| %>
     <div class="modal-content">
-      <div class="modal-header">
-        <h5 class="modal-title" id="exampleModalLabel"><%= Spree.t(:new_digital_asset) %></h5>
-        <button
-          type="button"
-          class="btn-close"
-          data-dismiss="modal"
-          aria-label="<%= Spree.t(:close) %>"
-        />
-      </div>
+      <%= modal_header(Spree.t(:new_digital_asset)) %>
       <div class="modal-body">
         <%= render "spree/admin/digital_assets/form", f: f %>
       </div>
-      <div class="modal-footer d-flex justify-content-between">
-        <button type="button" class="btn btn-light" data-dismiss="modal"><%= Spree.t("actions.discard") %></button>
+      <div class="modal-footer">
+        <%= modal_discard_button %>
         <%= turbo_save_button_tag Spree.t("actions.save"), data: { turbo_frame: :_top } %>
       </div>
     </div>

data/app/views/spree/admin/exports/index.html.erb

@@ -1,30 +1,28 @@
 <%= render 'spree/admin/shared/audit_nav' %>
 
-<% if @collection.present? %>
-  <div class="table-responsive card-lg p-0">
-    <table class="table">
-      <thead class="text-muted">
-        <tr>
-          <th><%= Spree.t(:number) %></th>
-          <th class="text-center"><%= Spree.t(:kind) %></th>
-          <th class="text-center"><%= Spree.t(:status) %></th>
-          <th><%= Spree.t(:user) %></th>
-          <th><%= Spree.t(:filename) %></th>
-          <th><%= Spree.t(:date) %></th>
-          <th><%= Spree.t(:size) %></th>
-          <th class="text-center"><%= Spree.t(:download) %></th>
-        </tr>
-      </thead>
+<div class="card-lg">
+  <% if @collection.any? %>
+    <div class="table-responsive">
+      <table class="table">
+        <thead class="text-muted">
+          <tr>
+            <th><%= Spree.t(:number) %></th>
+            <th class="text-center"><%= Spree.t(:kind) %></th>
+            <th class="text-center"><%= Spree.t(:status) %></th>
+            <th><%= Spree.t(:user) %></th>
+            <th><%= Spree.t(:filename) %></th>
+            <th><%= Spree.t(:date) %></th>
+            <th><%= Spree.t(:size) %></th>
+            <th class="text-center"><%= Spree.t(:download) %></th>
+          </tr>
+        </thead>
         <tbody>
-          <%= render partial: 'export', collection: @exports, cached: true %>
+          <%= render partial: 'export', collection: @collection, cached: true %>
         </tbody>
       </table>
+      <%= render 'spree/admin/shared/index_table_options', collection: @collection %>
     </div>
-  </div>
-<% else %>
-  <div class="text-muted p-5 d-flex align-items-center w-100 justify-content-center">
-    <%= Spree.t(:no_resource_found, resource: plural_resource_name(model_class)) %>
-  </div>
-<% end %>
-
-<%= render 'spree/admin/shared/index_table_options', collection: @imports, simple: true if @imports.present? %>
+  <% else %>
+    <%= render 'spree/admin/shared/no_resource_found', new_object_url: nil %>
+  <% end %>
+</div>

data/app/views/spree/admin/exports/new.html.erb

@@ -3,13 +3,7 @@
     <%= f.hidden_field :type %>
     <%= f.hidden_field :search_params %>
     <div class="modal-content">
-      <div class="modal-header">
-        <h5 class="modal-title" id="exampleModalLabel">
-          <%= Spree.t(:export) %>
-          <%= plural_resource_name(@export.model_class) %>
-        </h5>
-        <button type="button" class="btn-close" data-dismiss="modal" aria-label="<%= Spree.t(:close) %>" />
-      </div>
+      <%= modal_header(Spree.t(:export) + ' ' + plural_resource_name(@export.model_class)) %>
       <div class="modal-body">
         <%= render 'spree/admin/shared/error_messages', target: @export %>
         <div class="custom-control custom-radio">
@@ -33,8 +27,8 @@
           <% end %>
         </div>
       </div>
-      <div class="modal-footer d-flex justify-content-between">
-        <button type="button" class="btn btn-light" data-dismiss="modal"><%= Spree.t('actions.discard') %></button>
+      <div class="modal-footer">
+        <%= modal_discard_button %>
         <%= turbo_save_button_tag Spree.t(:export), class: 'btn btn-primary' %>
       </div>
     </div>

data/app/views/spree/admin/line_items/new.html.erb

@@ -1,9 +1,6 @@
 <%= turbo_frame_tag :dialog_modal do %>
   <div class="modal-content" data-controller="search-picker">
-    <div class="modal-header">
-      <h5 class="modal-title" id="exampleModalLabel"><%= Spree.t(:add_variant) %></h5>
-      <button type="button" class="btn-close" data-dismiss="modal" aria-label="<%= Spree.t(:close) %>" />
-    </div>
+    <%= modal_header(Spree.t(:add_variant)) %>
     <div class="modal-body border-bottom">
       <%= form_with url: spree.admin_variants_search_path, method: :post, data: { controller: 'auto-submit', turbo_frame: '_top' } do |form| %>
         <%= form.hidden_field :omit_ids, value: @order.variant_ids.join(',') %>
@@ -20,7 +17,7 @@
           </p>
         </div>
       </div>
-      <div class="modal-footer d-flex justify-content-between border-top">
+      <div class="modal-footer border-top">
         <div class="d-flex gap-2 m-0 align-items-center">
           <%= form.label :quantity, Spree.t(:quantity), class: "mb-0" %>
           <%= form.number_field :quantity, class: "form-control", min: 1 %>

data/app/views/spree/admin/oauth_applications/index.html.erb

@@ -3,7 +3,7 @@
 <%= content_for(:title, Spree.t('admin.oauth_applications.list')) %>
 
 <% content_for :page_actions do %>
-  <%= button_link_to Spree.t('admin.oauth_applications.new'), new_object_url, class: "btn-primary", icon: 'plus' %>
+  <%= link_to_with_icon 'plus', Spree.t('admin.oauth_applications.new'), new_object_url, class: "btn btn-primary" %>
 <% end if can? :create, Spree::OauthApplication %>
 
 <%= content_for :page_alerts do %>
@@ -15,61 +15,60 @@
   </div>
 <% end %>
 
-<% if @oauth_applications.any? %>
-  <div class="table-responsive rounded border mt-4 bg-white">
-    <table class="table">
-      <thead class="text-muted">
-        <tr>
-          <th><%= Spree.t(:name) %></th>
-          <th>Client ID</th>
-          <th><%= Spree.t('admin.oauth_applications.scopes') %></th>
-          <th><%= Spree.t('admin.oauth_applications.last_used') %></th>
-          <th class="actions"></th>
-        </tr>
-      </thead>
-      <tbody>
-        <% @oauth_applications.each do |application| %>
-          <tr id="<%= spree_dom_id application %>">
-            <td class="w-20"><code><%= application.name %></code></td>
-            <td class="w-40">
-              <div class="input-group" data-controller="password-toggle">
-                <%= button_tag class: 'btn hover-gray px-1', data: { action: 'click->password-toggle#password' } do %>
-                  <%= icon 'eye', class: 'mr-0' %>
-                <% end %>
-                <%= password_field_tag :password, application.uid, class: 'form-control-plaintext', data: { password_toggle_target: 'unhide' }, readonly: true %>
-              </div>
-            </td>
-            <td class="w-15">
-              <% if application.scopes.include?('write') %>
-                <span class="badge badge-warning">
-                  <%= icon('pencil', class: 'mr-1') %>
-                  <%= Spree.t('admin.oauth_applications.read_and_write') %>
-                </span>
-              <% else %>
-                <span class="badge badge-light">
-                  <%= icon('eye', class: 'mr-1') %>
-                  <%= Spree.t('admin.oauth_applications.read_only') %>
-                </span>
-              <% end %>
-            </td>
-            <td class="w-15">
-              <% if application.last_used_at %>
-                <%= local_time(application.last_used_at) %>
-              <% else %>
-                <span class="text-muted"><%= Spree.t('admin.oauth_applications.never') %></span>
-              <% end %>
-            </td>
-            <td class="actions w-10">
-              <%= link_to_edit(application, no_text: true) if can? :edit, application %>
-            </td>
+<div class="card-lg">
+  <% if @collection.any? %>
+    <div class="table-responsive">
+      <table class="table">
+        <thead class="text-muted">
+          <tr>
+            <th><%= Spree.t(:name) %></th>
+            <th>Client ID</th>
+            <th><%= Spree.t('admin.oauth_applications.scopes') %></th>
+            <th><%= Spree.t('admin.oauth_applications.last_used') %></th>
+            <th></th>
           </tr>
-        <% end %>
-      </tbody>
-    </table>
-  </div>
-<% else %>
-  <div class="text-muted p-5 d-flex align-items-center w-100 justify-content-center">
-    <%= Spree.t(:no_resource_found, resource: 'API keys') %>
-    <%= link_to_with_icon('plus', Spree.t(:add_one), new_object_url, class: 'btn btn-link ml-3') if can? :create, Spree::OauthApplication %>
-  </div>
-<% end %>
+        </thead>
+        <tbody>
+          <% @collection.each do |application| %>
+            <tr id="<%= spree_dom_id application %>">
+              <td class="w-20"><code><%= application.name %></code></td>
+              <td class="w-40">
+                <div class="input-group" data-controller="password-toggle">
+                  <%= button_tag class: 'btn hover-gray px-1', data: { action: 'click->password-toggle#password' } do %>
+                    <%= icon 'eye', class: 'mr-0' %>
+                  <% end %>
+                  <%= password_field_tag :password, application.uid, class: 'form-control-plaintext', data: { password_toggle_target: 'unhide' }, readonly: true %>
+                </div>
+              </td>
+              <td class="w-15">
+                <% if application.scopes.include?('write') %>
+                  <span class="badge badge-warning">
+                    <%= icon('pencil', class: 'mr-1') %>
+                    <%= Spree.t('admin.oauth_applications.read_and_write') %>
+                  </span>
+                <% else %>
+                  <span class="badge badge-light">
+                    <%= icon('eye', class: 'mr-1') %>
+                    <%= Spree.t('admin.oauth_applications.read_only') %>
+                  </span>
+                <% end %>
+              </td>
+              <td class="w-15">
+                <% if application.last_used_at %>
+                  <%= local_time(application.last_used_at) %>
+                <% else %>
+                  <span class="text-muted"><%= Spree.t('admin.oauth_applications.never') %></span>
+                <% end %>
+              </td>
+              <td class="actions w-10">
+                <%= link_to_edit(application, no_text: true) if can? :edit, application %>
+              </td>
+            </tr>
+          <% end %>
+        </tbody>
+      </table>
+    </div>
+  <% else %>
+    <%= render 'spree/admin/shared/no_resource_found', resource_name: 'API keys' %>
+  <% end %>
+</div>

data/app/views/spree/admin/option_types/_filter.html.erb

@@ -1,4 +1,4 @@
-<%= search_form_for [:admin, @search], class: "filter-wrap border-bottom", data: {controller: "filters reveal", reveal_hidden_class: "d-none"} do |f| %>
+<%= search_form_for [:admin, @search], class: "filter-wrap", data: {controller: "filters reveal", reveal_hidden_class: "d-none"} do |f| %>
   <%= render 'spree/admin/shared/filters_search_bar', param: :name_cont, label: Spree.t(:name) %>
 
   <%= render "spree/admin/shared/filter_badge_template" %>

data/app/views/spree/admin/option_types/index.html.erb

@@ -3,11 +3,7 @@
 <% end %>
 
 <% content_for :page_actions do %>
-  <span id="new_ot_link">
-    <%= button_link_to Spree.t(:new_option_type),
-    new_object_url,
-    { class: "btn-primary", icon: "plus", id: "new_option_type_link" } %>
-  </span>
+  <%= link_to_with_icon 'plus', Spree.t(:new_option_type), new_object_url, class: 'btn btn-primary' %>
 <% end if can?(:create, Spree::OptionType) %>
 
 <%= content_for(:page_alerts) do %>
@@ -17,12 +13,8 @@
   </div>
 <% end %>
 
-<div id="new_option_type"></div>
-
-<div class="card-lg p-0">
-
+<div class="card-lg">
   <%= render partial: "spree/admin/option_types/filter" %>
-
   <% if @option_types.any? %>
     <div class="table-responsive">
       <table class="table" id="listing_option_types">
@@ -44,7 +36,7 @@
             <th>
               <%= Spree.t(:products) %>
             </th>
-            <th class="actions"></th>
+            <th></th>
           </tr>
         </thead>
         <tbody
@@ -61,3 +53,7 @@
     <%= render "spree/admin/shared/no_resource_found" %>
   <% end %>
 </div>
+
+<p class="documentation-link-container">
+  <%= external_link_to "Learn more about options", "https://spreecommerce.org/docs/user/manage-products/product-options" %>
+</p>

data/app/views/spree/admin/orders/_customer_summary.html.erb

@@ -1,8 +1,6 @@
 <% if order %>
   <% if order.user %>
     <span class="d-flex">
-      <%#= image_tag(avatar_url_for( order.user), width: 24, height: 24, class: 'mr-2 rounded') %>
-
       <% if can?(:manage, order.user) %>
         <%= link_to order.full_name, spree.admin_user_path(order.user), class: 'text-dark', data: { 'turbo-frame': '_top' } %>
       <% else %>

data/app/views/spree/admin/orders/_filters.html.erb

@@ -2,7 +2,7 @@
 
 <%= search_form_for [:admin, @vendor || @user, @search],
   url: controller_name == 'checkouts' ? spree.admin_checkouts_path : nil,
-  class: "filter-wrap border-bottom",
+  class: "filter-wrap",
   data: {
     controller: "filters reveal",
     reveal_hidden_class: "d-none",
@@ -89,15 +89,10 @@
         </div>
       </div>
 
-      <%= render "spree/admin/orders/extra_filters", f: f %>
+      <%= render_admin_partials(:orders_filters_partials, f: f) %>
     </div>
 
-    <div class="form-actions">
-      <%= turbo_save_button_tag Spree.t(:filter_results) do %>
-        <%= icon("search") %>
-        <%= Spree.t(:filter_results) %>
-      <% end %>
-    </div>
+    <%= render 'spree/admin/shared/filter_submit' %>
   </div>
 
   <%= render "spree/admin/shared/filter_badge_template" %>

data/app/views/spree/admin/orders/_header.html.erb

@@ -63,6 +63,8 @@
     <% end %>
   <% end %>
 
+  <%= render_admin_partials(:order_page_dropdown_partials, order: @order) %>
+
   <% if can?(:cancel, @order) %>
     <div class="dropdown-divider"></div>
     <%= link_to spree.cancel_admin_order_path(@order),

data/app/views/spree/admin/orders/_list.html.erb

@@ -23,8 +23,6 @@
     </table>
   </div>
 <% else %>
-  <div class="text-muted p-5 d-flex align-items-center w-100 justify-content-center">
-    <%= Spree.t(:no_resource_found, resource: plural_resource_name(Spree::Order)) %>
-  </div>
+  <%= render 'spree/admin/shared/no_resource_found', new_object_url: nil, model_class: Spree::Order %>
 <% end %>
-<%= render 'spree/admin/shared/index_table_options', collection: @orders, simple: true if @orders.any? %>
+<%= render 'spree/admin/shared/index_table_options', collection: @orders if @orders.any? %>

data/app/views/spree/admin/orders/_summary.html.erb

@@ -74,7 +74,7 @@
       <%= render 'tax_lines', tax_lines: tax_lines, id: 'tax-lines-additional' %>
     </li>
 
-    <%= render 'summary_custom', order: @order %>
+    <%= render_admin_partials(:order_page_body_partials, order: @order) %>
 
     <li class="list-group-item d-flex justify-content-between align-items-center border-0">
       <span data-hook='admin_order_tab_total_title'><%= Spree.t(:total) %></span>

data/app/views/spree/admin/orders/billing_address/create.turbo_stream.erb

@@ -1,10 +1,7 @@
 <%= turbo_stream.update :dialog_modal do %>
   <%= form_for @address, url: spree.admin_order_billing_address_path(@order), data: {controller: 'auto-submit', form_target: 'form', turbo_frame: :_top}, method: :post do |f| %>
     <div class="modal-content">
-      <div class="modal-header">
-        <h5 class="modal-title" id="exampleModalLabel"><%= Spree.t(:new_billing_address) %></h5>
-        <button type="button" class="btn-close" data-dismiss="modal" aria-label="<%= Spree.t(:close) %>" ></button>
-      </div>
+      <%= modal_header(Spree.t(:new_billing_address)) %>
       <div class="modal-body d-flex flex-col gap-3" data-controller="order-billing-address">
         <div>
           <% if @order.shipping_address.present? %>
@@ -26,8 +23,8 @@
         </div>
         <%= render 'form', address: @address, f: f %>
       </div>
-      <div class="modal-footer d-flex justify-content-between">
-        <button type="button" class="btn btn-light" data-dismiss="modal"><%= Spree.t('actions.discard') %></button>
+      <div class="modal-footer">
+        <%= modal_discard_button %>
         <%= turbo_save_button_tag Spree.t('actions.save') %>
       </div>
     </div>

data/app/views/spree/admin/orders/billing_address/edit.html.erb

@@ -1,10 +1,7 @@
 <%= turbo_frame_tag :dialog_modal do %>
   <%= form_for @address, url: spree.admin_order_billing_address_path(@order), data: {controller: 'auto-submit', form_target: 'form', turbo_frame: :_top}, method: :put do |f| %>
     <div class="modal-content">
-      <div class="modal-header">
-        <h5 class="modal-title" id="exampleModalLabel"><%= Spree.t(:billing_address) %></h5>
-        <button type="button" class="btn-close" data-dismiss="modal" aria-label="<%= Spree.t(:close) %>"></button>
-      </div>
+      <%= modal_header(Spree.t(:billing_address)) %>
       <div class="modal-body d-flex flex-col gap-3" data-controller="order-billing-address">
         <div>
           <% if @order.shipping_address.present? %>
@@ -33,8 +30,8 @@
         </div>
         <%= render 'form', address: @address, f: f %>
       </div>
-      <div class="modal-footer d-flex justify-content-between">
-        <button type="button" class="btn btn-light" data-dismiss="modal"><%= Spree.t('actions.discard') %></button>
+      <div class="modal-footer">
+        <%= modal_discard_button %>
         <%= turbo_save_button_tag Spree.t('actions.update') %>
       </div>
     </div>

data/app/views/spree/admin/orders/billing_address/new.html.erb

@@ -1,10 +1,7 @@
 <%= turbo_frame_tag :dialog_modal do %>
   <%= form_for @address, url: spree.admin_order_billing_address_path(@order), data: {controller: 'auto-submit', form_target: 'form', turbo_frame: :_top}, method: :post do |f| %>
     <div class="modal-content">
-      <div class="modal-header">
-        <h5 class="modal-title" id="exampleModalLabel"><%= Spree.t(:billing_address) %></h5>
-        <button type="button" class="btn-close" data-dismiss="modal" aria-label="<%= Spree.t(:close) %>" ></button>
-      </div>
+      <%= modal_header(Spree.t(:billing_address)) %>
       <div class="modal-body d-flex flex-col gap-3" data-controller="order-billing-address">
         <div>
           <% if @order.shipping_address.present? %>
@@ -26,8 +23,8 @@
         </div>
         <%= render 'form', address: @address, f: f %>
       </div>
-      <div class="modal-footer d-flex justify-content-between">
-        <button type="button" class="btn btn-light" data-dismiss="modal"><%= Spree.t('actions.discard') %></button>
+      <div class="modal-footer">
+        <%= modal_discard_button %>
         <%= turbo_save_button_tag Spree.t('actions.save') %>
       </div>
     </div>

data/app/views/spree/admin/orders/billing_address/update.turbo_stream.erb

@@ -1,10 +1,7 @@
 <%= turbo_stream.update :dialog_modal do %>
   <%= form_for @address, url: spree.admin_order_billing_address_path(@order), data: {controller: 'auto-submit', form_target: 'form', turbo_frame: :_top}, method: :put do |f| %>
     <div class="modal-content">
-      <div class="modal-header">
-        <h5 class="modal-title" id="exampleModalLabel"><%= Spree.t(:update_billing_address) %></h5>
-        <button type="button" class="btn-close" data-dismiss="modal" aria-label="<%= Spree.t(:close) %>"></button>
-      </div>
+      <%= modal_header(Spree.t(:update_billing_address)) %>
       <div class="modal-body d-flex flex-col gap-3" data-controller="order-billing-address">
         <div>
           <% if @order.shipping_address.present? %>
@@ -26,8 +23,8 @@
         </div>
         <%= render 'form', address: @address, f: f %>
       </div>
-      <div class="modal-footer d-flex justify-content-between">
-        <button type="button" class="btn btn-light" data-dismiss="modal"><%= Spree.t('actions.discard') %></button>
+      <div class="modal-footer">
+        <%= modal_discard_button %>
         <%= turbo_save_button_tag Spree.t('actions.update') %>
       </div>
     </div>

data/app/views/spree/admin/orders/contact_information/edit.html.erb

@@ -1,18 +1,15 @@
 <%= turbo_frame_tag :dialog_modal do %>
   <%= form_for @order, url: spree.admin_order_contact_information_path(@order), method: :put, data: { turbo_frame: :_top } do |f| %>
     <div class="modal-content">
-      <div class="modal-header">
-        <h5 class="modal-title" id="exampleModalLabel"><%= Spree.t(:update_contact_information) %></h5>
-        <button type="button" class="btn-close" data-dismiss="modal" aria-label="<%= Spree.t(:close) %>" />
-      </div>
+      <%= modal_header(Spree.t(:update_contact_information)) %>
       <div class="modal-body">
         <div class="form-group">
           <%= f.label :email, Spree.t(:email) %>
           <%= f.email_field :email, class: 'form-control', required: true %>
         </div>
       </div>
-      <div class="modal-footer d-flex justify-content-between">
-        <button type="button" class="btn btn-light" data-dismiss="modal"><%= Spree.t('actions.discard') %></button>
+      <div class="modal-footer">
+        <%= modal_discard_button %>
         <%= turbo_save_button_tag Spree.t('actions.update') %>
       </div>
     </div>

data/app/views/spree/admin/orders/customer_returns/edit.html.erb

@@ -110,16 +110,7 @@
         reimbursements: @customer_return.reimbursements,
       } %>
     <% else %>
-      <div
-        class="
-          text-muted p-5 d-flex align-items-center w-100 justify-content-center
-        "
-      >
-        <%= Spree.t(
-          :no_resource_found,
-          resource: plural_resource_name(Spree::Reimbursement),
-        ) %>
-      </div>
+      <%= render 'spree/admin/shared/no_resource_found', new_object_url: nil %>
     <% end %>
   </div>
 </div>

data/app/views/spree/admin/orders/edit.html.erb

@@ -1,23 +1,21 @@
 <%= render 'header' %>
-<%= render 'header_custom' %>
+<%= render_admin_partials(:order_page_header_partials, order: @order) %>
 
 <div class="container px-0">
   <div class="row">
     <div class="col-lg-8">
-
       <%= render 'line_items' %>
       <%= render 'shipments' %>
       <%= render 'customer_returns' %>
       <%= render 'return_authorizations' %>
       <%= render 'payments'%>
       <%= render 'refunds'%>
-
-      <%= render 'body_custom' %>
+      <%= render_admin_partials(:order_page_body_partials, order: @order) %>
       <%= render 'summary' %>
     </div>
     <div class="col-lg-4">
-      <!-- TODO: add notes -->
       <%= render 'customer' %>
+      <%= render_admin_partials(:order_page_sidebar_partials, order: @order) %>
     </div>
   </div>
 </div>

data/app/views/spree/admin/orders/index.html.erb

@@ -10,7 +10,7 @@
   <%= link_to_with_icon 'plus', Spree.t(:new_order), spree.admin_orders_path, class: "btn btn-primary", data: { turbo_method: :post } if can?(:create, Spree::Order)  %>
 <% end %>
 
-<div class="card-lg p-0">
+<div class="card-lg">
   <% if params[:frame_name].present? %>
     <%= turbo_frame_tag params[:frame_name], autoscroll: true, data: { autoscroll_block: :nearest, autoscroll_behavior: :smooth } do %>
       <%= render 'spree/admin/orders/filters', frame_name: params[:frame_name] %>

data/app/views/spree/admin/orders/return_authorizations/_form.html.erb

@@ -1,7 +1,7 @@
 <% allow_return_item_changes = !@return_authorization.customer_returned_items? %>
 
 <div data-controller="return-items">
-  <div class="table-responsive card-lg p-0 mb-4">
+  <div class="table-responsive card-lg mb-4">
     <table class="table table-condensed return-items-table">
       <thead class="text-muted">
         <tr>