spree_admin 5.0.0.rc1 → 5.0.1

data/app/views/spree/admin/products/_product.html.erb

@@ -1,9 +1,6 @@
 <tr id="<%= dom_id product %>" data-controller="row-link">
   <td class="pr-0 w-5">
-    <div class="custom-control custom-checkbox ml-1">
-      <%= check_box_tag "ids[]", product.id, false, class: "custom-control-input", id: "ids_#{product.id}", data: { bulk_operation_target: "checkbox" } %>
-      <label class="custom-control-label" for="ids_<%= product.id %>"><span></span></label>
-    </div>
+    <%= bulk_operations_checkbox(product) %>
   </td>
   <td scope="row" class="w-40 py-0 text-wrap" data-action="click->row-link#openLink">
     <%= link_to spree.edit_admin_product_path(product), class: 'd-flex align-items-center w-100 text-decoration-none h-100 py-2', data: { row_link_target: :link, 'turbo-frame': '_top' } do %>

data/app/views/spree/admin/products/bulk_modal.html.erb

@@ -1,11 +1,6 @@
 <%= turbo_frame_tag :bulk_modal do %>
   <div class="modal-content">
-    <div class="modal-header">
-      <h5 class="modal-title" id="exampleModalLabel">
-        <%= @title %>
-      </h5>
-      <button type="button" class="btn-close" data-dismiss="modal" aria-label="<%= Spree.t(:close) %>" />
-    </div>
+    <%= modal_header(@title) %>
     <div class="modal-body">
       <p><%= @body %></p>
 
@@ -15,8 +10,8 @@
         <%= tom_select_tag 'tags', multiple: true, class: 'w-100', options: tags_json_array, required: true, value_field: :name, create: params[:kind] == 'add_tags' %>
       <% end %>
     </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 %>
     </div>
   </div>

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

@@ -6,6 +6,9 @@
     spree.admin_product_digital_assets_path(@product),
     class: "dropdown-item",
   ) if can?(:manage, Spree::Digital) %>
+
+  <%= render_admin_partials(:product_dropdown_partials, product: @product) %>
+
   <div class="dropdown-divider"></div>
 
   <%= external_page_preview_link(@product) %>

data/app/views/spree/admin/products/form/_variants.html.erb

@@ -9,6 +9,7 @@
         data-controller="variants-form"
         class="variants-form"
         data-action="product-form:toggle-quantity-tracked@window->variants-form#toggleQuantityTracked"
+        data-variants-form-locale-value="<%= I18n.locale %>"
         <% if @product.persisted? %> data-variants-form-product-id-value="<%= @product.slug %>" <% end %>
         data-variants-form-current-currency-value="<%= current_currency %>"
         data-variants-form-currencies-value="<%= supported_currencies.map(&:to_s).to_json %>"

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

@@ -1,9 +1,9 @@
 <%= render 'index_header' %>
 
-<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: :start, autoscroll_behavior: :smooth } do %>
-      <%= render 'filters' %>
+      <%= render 'filters', frame_name: params[:frame_name] %>
       <%= render 'list' %>
     <% end %>
   <% else %>
@@ -12,7 +12,7 @@
   <% end %>
 </div>
 
-<p class="text-muted small text-center py-2 px-3 mt-4 rounded-pill mx-auto d-flex-inline">
-  Learn more about 
-  <%= external_link_to 'managing products', 'https://spreecommerce.org/docs/user/manage-products', class: 'text-decoration-none' %>
+<p class="documentation-link-container">
+  Learn more about
+  <%= external_link_to 'managing products', 'https://spreecommerce.org/docs/user/manage-products' %>
 </p>

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

@@ -1,16 +1,11 @@
 <%= turbo_frame_tag :dialog_modal do %>
   <%= form_with model: @promotion_action, url: spree.admin_promotion_actions_path(@promotion), method: :post, scope: :promotion_action do |f| %>
     <div class="modal-content">
-      <div class="modal-header">
-        <h5 class="modal-title">
-          <% if @promotion_action.type.nil? %>
-            <%= Spree.t(:add_action_of_type) %>
-          <% else %>
-            <%= @promotion_action.human_name %>
-          <% end %>
-        </h5>
-        <button type="button" class="btn-close" data-dismiss="modal" aria-label="<%= Spree.t(:close) %>" />
-      </div>
+      <% if @promotion_action.type.nil? %>
+        <%= modal_header(Spree.t(:add_action_of_type)) %>
+      <% else %>
+        <%= modal_header(@promotion_action.human_name) %>
+      <% end %>
       <div class="modal-body">
         <%= render partial: 'spree/admin/shared/error_messages', locals: { target: @promotion_action } %>
         <% if @promotion_action.type.nil? %>
@@ -38,8 +33,8 @@
         <% end %>
       </div>
       <% if @promotion_action.type.present? %>
-        <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.create'), data: { turbo_frame: '_top' } %>
         </div>
       <% end %>

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

@@ -1,16 +1,11 @@
 <%= turbo_frame_tag :dialog_modal do %>
   <%= form_with model: @promotion_rule, url: spree.admin_promotion_rules_path(@promotion), method: :post, scope: :promotion_rule do |f| %>
     <div class="modal-content">
-      <div class="modal-header">
-        <h5 class="modal-title">
-          <% if @promotion_rule.type.nil? %>
-            <%= Spree.t(:add_rule_of_type) %>
-          <% else %>
-            <%= @promotion_rule.human_name %>
-          <% end %>
-        </h5>
-        <button type="button" class="btn-close" data-dismiss="modal" aria-label="<%= Spree.t(:close) %>" />
-      </div>
+      <% if @promotion_rule.type.nil? %>
+        <%= modal_header(Spree.t(:add_rule_of_type)) %>
+      <% else %>
+        <%= modal_header(@promotion_rule.human_name) %>
+      <% end %>
       <div class="modal-body">
         <%= render partial: 'spree/admin/shared/error_messages', locals: { target: @promotion_rule } %>
         <% if @promotion_rule.type.nil? %>
@@ -38,8 +33,8 @@
         <% end %>
       </div>
       <% if @promotion_rule.type.present? %>
-        <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.create'), data: { turbo_frame: '_top' } %>
         </div>
       <% end %>

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

@@ -0,0 +1,27 @@
+<%= search_form_for [:admin, @search], class: "filter-wrap", data: {controller: "filters reveal", reveal_hidden_class: "d-none"} do |f| %>
+  <div class="d-flex flex-column flex-lg-row gap-2">
+    <%= render 'spree/admin/shared/filters_search_bar', param: :name_cont, label: Spree.t(:name) %>
+
+    <div class="d-flex align-items-center justify-content-between">
+      <%= render "spree/admin/promotions/table_filter_dropdown" %>
+
+      <%= render 'spree/admin/shared/filters_button' %>
+    </div>
+  </div>
+  <div data-reveal-target="item" class="d-none" id="table-filter">
+    <div class="row">
+      <div class="col-12 col-lg-6">
+        <div class="form-group">
+          <%= label_tag :q_code_or_coupon_codes_code_eq, Spree.t(:code) %>
+          <%= f.search_field :code_or_coupon_codes_code_eq, class: "form-control", data: { filters_target: :input } %>
+        </div>
+      </div>
+    </div>
+
+    <%= render 'spree/admin/shared/filter_submit' %>
+  </div>
+
+  <%= render "spree/admin/shared/filter_badge_template" %>
+
+  <div data-filters-target="badgesContainer" class="filter-badges-container"></div>
+<% end %>

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

@@ -3,41 +3,13 @@
 <% end %>
 
 <% content_for :page_actions do %>
-  <%= button_link_to Spree.t(:new_promotion), new_object_url, class: "btn-primary", icon: "plus" %>
+  <%= link_to_with_icon 'plus', Spree.t(:new_promotion), new_object_url, class: "btn btn-primary" %>
 <% end if can?(:create, Spree::Promotion) %>
 
-<div class="card-lg p-0">
-  <%= search_form_for [:admin, @search], class: "filter-wrap border-bottom", data: {controller: "filters reveal", reveal_hidden_class: "d-none"} do |f| %>
-    <div class="d-flex flex-column flex-lg-row gap-2">
-      <%= render 'spree/admin/shared/filters_search_bar', param: :name_cont, label: Spree.t(:name) %>
+<div class="card-lg">
+  <%= render 'filters' %>
 
-      <div class="d-flex align-items-center justify-content-between">
-        <%= render "spree/admin/promotions/table_filter_dropdown" %>
-
-        <%= render 'spree/admin/shared/filters_button' %>
-      </div>
-    </div>
-    <div data-reveal-target="item" class="d-none" id="table-filter">
-      <div class="row">
-        <div class="col-12 col-lg-6">
-          <div class="form-group">
-            <%= label_tag :q_code_or_coupon_codes_code_eq, Spree.t(:code) %>
-            <%= f.search_field :code_or_coupon_codes_code_eq, class: "form-control", data: { filters_target: :input } %>
-          </div>
-        </div>
-      </div>
-
-      <div class="form-actions">
-        <%= button Spree.t(:filter_results), "search" %>
-      </div>
-    </div>
-
-    <%= render "spree/admin/shared/filter_badge_template" %>
-
-    <div data-filters-target="badgesContainer" class="filter-badges-container"></div>
-  <% end %>
-
-  <% if @promotions.any? %>
+  <% if @collection.any? %>
     <div class="table-responsive">
       <table class="table">
         <thead class="text-muted">
@@ -47,15 +19,15 @@
             <th><%= Spree.t(:kind) %></th>
             <th><%= Spree.t(:usage_limit) %></th>
             <th><%= Spree.t(:status) %></th>
-            <th class="actions"></th>
+            <th></th>
           </tr>
         </thead>
         <tbody>
-          <%= render collection: @promotions, partial: "spree/admin/promotions/promotion", cached: spree_base_cache_scope %>
+          <%= render collection: @collection, partial: "spree/admin/promotions/promotion", cached: spree_base_cache_scope %>
         </tbody>
       </table>
     </div>
-    <%= render "spree/admin/shared/index_table_options", collection: @promotions, simple: true %>
+    <%= render "spree/admin/shared/index_table_options", collection: @collection %>
   <% else %>
     <%= render "spree/admin/shared/no_resource_found" %>
   <% end %>

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

@@ -9,22 +9,19 @@
 <% end %>
 
 <% content_for :page_actions do %>
-  <%= button_link_to Spree.t(:new_property), new_object_url, { class: "btn-primary", icon: "plus" } %>
+  <%= link_to_with_icon 'plus', Spree.t(:new_property), new_object_url, class: "btn btn-primary" %>
 <% end if can?(:create, Spree::Property)  %>
 
-<div class="card-lg p-0">
-
-  <%= search_form_for [:admin, @search], class: "filter-wrap border-bottom", data: {controller: "filters reveal", reveal_hidden_class: "d-none"} do |f| %>
+<div class="card-lg">
+  <%= 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" %>
-
     <div data-filters-target="badgesContainer" class="filter-badges-container"></div>
   <% end %>
 
   <% if @properties.present? %>
     <div class="table-responsive">
-      <table class="table" id='listing_properties'>
+      <table class="table">
         <thead class="text-muted">
           <tr>
             <th class="no-border handel-head"></th>
@@ -33,7 +30,7 @@
             <th><%= Spree.t(:kind) %></th>
             <th><%= Spree.t(:products) %></th>
             <th><%= Spree.t(:visibility) %></th>
-            <th class="actions"></th>
+            <th></th>
           </tr>
         </thead>
         <tbody
@@ -45,9 +42,13 @@
           <%= render partial: "spree/admin/properties/property", collection: @collection, cached: spree_base_cache_scope %>
         </tbody>
       </table>
-      <%= render "spree/admin/shared/index_table_options", collection: @collection, simple: true %>
+      <%= render "spree/admin/shared/index_table_options", collection: @collection %>
     </div>
   <% else %>
     <%= render "spree/admin/shared/no_resource_found" %>
   <% end %>
 </div>
+
+<p class="documentation-link-container">
+  <%= external_link_to "Learn more about properties", "https://spreecommerce.org/docs/user/manage-products/product-properties" %>
+</p>

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

@@ -1,40 +1,42 @@
 <% content_for :page_actions do %>
-  <%= button_link_to Spree.t(:new_refund_reason), new_object_url, { icon: 'plus', id: 'admin_new_named_type', class: "btn-primary" } %>
+  <%= link_to_with_icon 'plus', Spree.t(:new_refund_reason), new_object_url, class: "btn btn-primary" %>
 <% end if can? :create, Spree::RefundReason %>
 
 <%= render 'spree/admin/shared/returns_and_refunds_nav' %>
 
-<% if @collection.any? %>
-  <div class="table-responsive card-lg p-0">
-    <table class="table">
-      <thead class="text-muted">
-        <tr>
-          <th><%= Spree.t(:name) %></th>
-          <th><%= Spree.t(:status) %></th>
-          <th><%= Spree.t(:mutable) %></th>
-          <th></th>
-        </tr>
-      </thead>
-      <tbody>
-      <% @collection.each do |refund_reason| %>
-        <tr id="<%= spree_dom_id refund_reason %>" data-controller="row-link" class="cursor-pointer">
-          <td class="w-40 align-center" data-action="click->row-link#openLink">
-            <%= refund_reason.name %>
-          </td>
-          <td class="w-20 align-center" data-action="click->row-link#openLink">
-            <%= active_badge(refund_reason.active, label: Spree.t(refund_reason.active? ? :active : :inactive)) %>
-          </td>
-          <td class="w-10 align-center" data-action="click->row-link#openLink">
-            <%= active_badge(refund_reason.mutable) %>
-          </td>
-          <td class="actions w-10" data-action="click->row-link#openLink">
-            <%= link_to_edit(refund_reason, no_text: true, data: { row_link_target: :link }) if can? :edit, refund_reason %>
-          </td>
-        </tr>
-      <% end %>
-      </tbody>
-    </table>
-  </div>
-<% else %>
-  <%= render 'spree/admin/shared/no_resource_found' %>
-<% end %>
+<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><%= Spree.t(:status) %></th>
+            <th><%= Spree.t(:mutable) %></th>
+            <th></th>
+          </tr>
+        </thead>
+        <tbody>
+        <% @collection.each do |refund_reason| %>
+          <tr id="<%= spree_dom_id refund_reason %>" data-controller="row-link" class="cursor-pointer">
+            <td class="w-40 align-center" data-action="click->row-link#openLink">
+              <%= refund_reason.name %>
+            </td>
+            <td class="w-20 align-center" data-action="click->row-link#openLink">
+              <%= active_badge(refund_reason.active, label: Spree.t(refund_reason.active? ? :active : :inactive)) %>
+            </td>
+            <td class="w-10 align-center" data-action="click->row-link#openLink">
+              <%= active_badge(refund_reason.mutable) %>
+            </td>
+            <td class="actions w-10" data-action="click->row-link#openLink">
+              <%= link_to_edit(refund_reason, no_text: true, data: { row_link_target: :link }) if can? :edit, refund_reason %>
+            </td>
+          </tr>
+        <% end %>
+        </tbody>
+      </table>
+    </div>
+  <% else %>
+    <%= render 'spree/admin/shared/no_resource_found' %>
+  <% end %>
+</div>

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

@@ -1,18 +1,13 @@
 <%= turbo_frame_tag :dialog_modal do %>
   <%= form_for [:admin, @refund.payment.order, @refund.payment, @refund], data: { turbo_frame: '_top' } do |f| %>
     <div class="modal-content">
-      <div class="modal-header">
-        <h5 class="modal-title" id="exampleModalLabel">
-          <%= Spree.t(:edit_refund) %>
-        </h5>
-        <button type="button" class="btn-close" data-dismiss="modal" aria-label="<%= Spree.t(:close) %>" />
-      </div>
+      <%= modal_header(Spree.t(:edit_refund)) %>
       <div class="modal-body">
         <%= render 'spree/admin/shared/error_messages', target: @refund %>
         <%= render 'spree/admin/refunds/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 %>
       </div>
     </div>

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

@@ -1,44 +1,46 @@
 <% content_for :page_actions do %>
-  <%= button_link_to Spree.t(:new_reimbursement_type), new_object_url, { :class => "btn-primary", :icon => 'plus', :id => 'admin_new_reimbursement_type' } %>
+  <%= link_to_with_icon 'plus', Spree.t(:new_reimbursement_type), new_object_url, class: "btn btn-primary" %>
 <% end if can?(:create, Spree::ReimbursementType) %>
 
-<%= render partial: 'spree/admin/shared/returns_and_refunds_nav' %>
+<%= render 'spree/admin/shared/returns_and_refunds_nav' %>
 
-<% if @reimbursement_types.any? %>
-  <div class="table-responsive card-lg p-0">
-    <table class="table" id='listing_reimbursement_types'>
-      <thead class="text-muted">
-        <tr>
-          <th><%= Spree.t(:name) %></th>
-          <th><%= Spree.t(:type) %></th>
-          <th><%= Spree.t(:status) %></th>
-          <th><%= Spree.t(:mutable) %></th>
-          <th class="actions"></th>
-        </tr>
-      </thead>
-      <tbody>
-      <% @reimbursement_types.each do |reimbursement_type| %>
-        <tr id="<%= spree_dom_id reimbursement_type %>" data-controller="row-link">
-          <td class="w-40 align-center cursor-pointer" data-action="click->row-link#openLink">
-            <%= reimbursement_type.name.humanize %>
-          </td>
-          <td class="w-30 cursor-pointer" data-action="click->row-link#openLink">
-            <code><%= reimbursement_type.type.demodulize %></code>
-          </td>
-          <td class="w-10 align-center cursor-pointer" data-action="click->row-link#openLink">
-            <%= active_badge(reimbursement_type.active, label: Spree.t(reimbursement_type.active? ? :active : :inactive)) %>
-          </td>
-          <td class="w-10 align-center cursor-pointer" data-action="click->row-link#openLink">
-            <%= active_badge(reimbursement_type.mutable) %>
-          </td>
-          <td class="actions w-10">
-            <%= link_to_edit reimbursement_type, no_text: true, data: { row_link_target: :link } if can?(:edit, reimbursement_type) %>
-          </td>
-        </tr>
-      <% end %>
-      </tbody>
-    </table>
-  </div>
-<% else %>
-  <%= render 'spree/admin/shared/no_resource_found' %>
-<% end %>
+<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><%= Spree.t(:type) %></th>
+            <th><%= Spree.t(:status) %></th>
+            <th><%= Spree.t(:mutable) %></th>
+            <th></th>
+          </tr>
+        </thead>
+        <tbody>
+          <% @collection.each do |reimbursement_type| %>
+            <tr id="<%= spree_dom_id reimbursement_type %>" data-controller="row-link">
+              <td class="w-40 align-center cursor-pointer" data-action="click->row-link#openLink">
+                <%= reimbursement_type.name.humanize %>
+              </td>
+              <td class="w-30 cursor-pointer" data-action="click->row-link#openLink">
+                <code><%= reimbursement_type.type.demodulize %></code>
+              </td>
+              <td class="w-10 align-center cursor-pointer" data-action="click->row-link#openLink">
+                <%= active_badge(reimbursement_type.active, label: Spree.t(reimbursement_type.active? ? :active : :inactive)) %>
+              </td>
+              <td class="w-10 align-center cursor-pointer" data-action="click->row-link#openLink">
+                <%= active_badge(reimbursement_type.mutable) %>
+              </td>
+              <td class="actions w-10">
+                <%= link_to_edit reimbursement_type, no_text: true, data: { row_link_target: :link } if can?(:edit, reimbursement_type) %>
+              </td>
+            </tr>
+          <% end %>
+        </tbody>
+      </table>
+    </div>
+  <% else %>
+    <%= render 'spree/admin/shared/no_resource_found' %>
+  <% end %>
+</div>

data/app/views/spree/admin/reports/_report.html.erb

@@ -1,4 +1,4 @@
-<div class="card-lg p-0">
+<div class="card-lg">
   <div class="table-responsive">
     <table class="table">
       <thead class="text-muted">

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

@@ -1,45 +1,45 @@
 <% content_for :page_actions do %>
-  <%= button_link_to Spree.t(:new_rma_reason),
-  new_object_url,
-  { icon: "plus", id: "admin_new_named_type", class: "btn-primary" } %>
-  <% end if can? :create, Spree::RefundReason %>
+  <%= link_to_with_icon 'plus', Spree.t(:new_rma_reason), new_object_url, class: "btn btn-primary" %>
+<% end if can? :create, Spree::ReturnAuthorizationReason %>
 
-    <%= render partial: "spree/admin/shared/returns_and_refunds_nav" %>
+<%= render"spree/admin/shared/returns_and_refunds_nav" %>
 
-    <% if @collection.any? %>
-      <div class="table-responsive card-lg p-0">
-        <table class="table" id='listing_named_types'>
-          <thead class="text-muted">
-            <tr>
-              <th><%= Spree.t(:name) %></th>
-              <th><%= Spree.t(:status) %></th>
-              <th></th>
+<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><%= Spree.t(:status) %></th>
+            <th></th>
+          </tr>
+        </thead>
+        <tbody id="<%= plural_resource_name(Spree::ReturnAuthorizationReason).to_s.parameterize.underscore %>">
+          <% @collection.each do |return_authorization_reason| %>
+            <tr id="<%= spree_dom_id return_authorization_reason %>" data-controller="row-link">
+              <td class="w-40 align-center cursor-pointer" data-action="click->row-link#openLink">
+                <%= return_authorization_reason.name %>
+              </td>
+              <td class="w-20 align-center cursor-pointer" data-action="click->row-link#openLink">
+                <%= active_badge(
+                  return_authorization_reason.active,
+                  label: Spree.t(return_authorization_reason.active? ? :active : :inactive),
+                ) %>
+              </td>
+              <td class="actions w-10">
+                <% if return_authorization_reason.mutable? %>
+                  <%= if can? :edit, return_authorization_reason
+                    link_to_edit(return_authorization_reason, no_text: true, data: { row_link_target: :link })
+                  end %>
+                <% end %>
+              </td>
             </tr>
-          </thead>
-          <tbody id="<%= plural_resource_name(Spree::ReturnAuthorizationReason).to_s.parameterize.underscore %>">
-            <% @collection.each do |return_authorization_reason| %>
-              <tr id="<%= spree_dom_id return_authorization_reason %>" data-controller="row-link">
-                <td class="w-40 align-center cursor-pointer" data-action="click->row-link#openLink">
-                  <%= return_authorization_reason.name %>
-                </td>
-                <td class="w-20 align-center cursor-pointer" data-action="click->row-link#openLink">
-                  <%= active_badge(
-                    return_authorization_reason.active,
-                    label: Spree.t(return_authorization_reason.active? ? :active : :inactive),
-                  ) %>
-                </td>
-                <td class="actions w-10">
-                  <% if return_authorization_reason.mutable? %>
-                    <%= if can? :edit, return_authorization_reason
-                      link_to_edit(return_authorization_reason, no_text: true, data: { row_link_target: :link })
-                    end %>
-                  <% end %>
-                </td>
-              </tr>
-            <% end %>
-          </tbody>
-        </table>
-      </div>
-    <% else %>
-      <%= render "spree/admin/shared/no_resource_found" %>
-    <% end %>
+          <% end %>
+        </tbody>
+      </table>
+    </div>
+  <% else %>
+    <%= render "spree/admin/shared/no_resource_found" %>
+  <% end %>
+</div>

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

@@ -1,4 +1,4 @@
-<%= search_form_for [:admin, @search], url: spree.admin_return_authorizations_path, class: "filter-wrap border-bottom", data: {controller: "filters reveal", reveal_hidden_class: "d-none"} do |f| %>
+<%= search_form_for [:admin, @search], url: spree.admin_return_authorizations_path, class: "filter-wrap", data: {controller: "filters reveal", reveal_hidden_class: "d-none"} do |f| %>
   <div class="d-flex flex-column flex-lg-row gap-2">
     <%= render "spree/admin/shared/filters_search_bar", param: :number_cont %>
 

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

@@ -17,14 +17,5 @@
     </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::ReturnAuthorization),
-    ) %>.
-  </div>
+  <%= render 'spree/admin/shared/no_resource_found', new_object_url: nil %>
 <% end %>

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

@@ -2,7 +2,7 @@
   <%= Spree.t(:return_authorizations) %>
 <% end %>
 
-<div class="card-lg p-0">
+<div class="card-lg">
   <%= render "spree/admin/return_authorizations/filters" %>
   <%= render "spree/admin/return_authorizations/list" %>
 

data/app/views/spree/admin/roles/_form.html.erb

@@ -2,5 +2,5 @@
   <%= f.label :name, raw(Spree.t(:name) + required_span_tag) %>
   <%= f.text_field :name, class: 'form-control' %>
   <%= f.error_message_on :name %>
-<% end %>
+</div>