spree_admin 5.0.6 → 5.1.0.beta

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

@@ -0,0 +1,32 @@
+<%= turbo_frame_tag :dialog_modal do %>
+  <%= form_for @invitation,
+    url: spree.admin_invitations_path,
+    data: { controller: "enable-button" } do |f| %>
+    <div class="modal-content">
+      <%= modal_header Spree.t('actions.send_invitation') %>
+      <div class="modal-body">
+        <%= render 'spree/admin/shared/error_messages', target: @invitation %>
+        <div class="form-group">
+          <%= f.label :email, Spree.t(:email) %>
+          <%= f.email_field :email, class: 'form-control', required: :required, data: { 'enable-button-target': 'input' }, placeholder: 'eg. steve@apple.com', autofocus: true %>
+        </div>
+        <% if @roles.any? %>
+          <div class="form-group">
+            <%= f.label :role, Spree.t(:role_id) %>
+            <%= f.select :role_id, @roles.map { |role| [role.name, role.id] }, {}, { data: { controller: 'autocomplete-select', 'enable-button-target': 'input' }, required: true } %>
+          </div>
+        <% end %>
+        <div class="form-group">
+          <%= f.label :expires_at %>
+          <%= f.date_field :expires_at, class: 'form-control', required: true, data: { 'enable-button-target': 'input' } %>
+        </div>
+      </div>
+      <div class="modal-footer">
+        <%= turbo_save_button_tag nil, class: 'btn btn-primary', data: { 'enable-button-target': 'button' } do %>
+          <%= icon 'send' %>
+          <%= Spree.t('actions.send_invitation') %>
+        <% end %>
+      </div>
+    </div>
+  <% end %>
+<% end %>

data/app/views/spree/admin/invitations/show.html.erb

@@ -0,0 +1,9 @@
+<div class="d-flex align-items-center gap-2">
+  <div>
+    <%= render_avatar(@invitation.inviter, height: 32, width: 32) %>
+  </div>
+  <div>
+    <strong><%= @invitation.inviter.name %></strong> has invited you to join <strong><%= @invitation.resource.name %></strong>
+  </div>
+</div>
+<%= link_to Spree.t(:accept), spree.accept_admin_invitation_path(@invitation), class: 'btn btn-primary w-100', data: { turbo_method: :put } %>

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

@@ -17,8 +17,8 @@
           </p>
         </div>
       </div>
-      <div class="modal-footer border-top">
-        <div class="d-flex gap-2 m-0 align-items-center">
+      <div class="modal-footer">
+        <div class="d-flex gap-2 m-0 align-items-center mr-auto">
           <%= form.label :quantity, Spree.t(:quantity), class: "mb-0" %>
           <%= form.number_field :quantity, class: "form-control", min: 1 %>
         </div>

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

@@ -19,7 +19,7 @@
   <% if @collection.any? %>
     <div class="table-responsive">
       <table class="table">
-        <thead class="text-muted">
+        <thead>
           <tr>
             <th><%= Spree.t(:name) %></th>
             <th>Client ID</th>

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

@@ -1,6 +1,6 @@
 <% content_for :page_title do %>
   <%= page_header_back_button spree.admin_option_types_url %>
-  <%= @option_type.name %>
+  <%= @option_type.presentation %>
 <% end %>
 
 <%= render partial: 'spree/admin/shared/error_messages', locals: { target: @option_type } %>

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

@@ -18,7 +18,7 @@
   <% if @option_types.any? %>
     <div class="table-responsive">
       <table class="table" id="listing_option_types">
-        <thead class="text-muted">
+        <thead>
           <tr>
             <th class="no-border handel-head"></th>
             <th><%= Spree.t(:internal_name) %></th>

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

@@ -5,7 +5,7 @@
     </h5>
     <% if can?(:update_customer, @order) || can?(:update_addresses, @order) %>
       <div class="dropdown ml-auto" id="customer-edit-dropdown">
-        <%= link_to_with_icon 'dots', nil, '#', class: 'btn btn-light btn-sm px-2', data: { toggle: 'dropdown', test_id: 'dropdown-toggle' } %>
+        <%= link_to_with_icon 'dots-vertical', nil, '#', class: 'btn btn-light btn-sm px-2', data: { toggle: 'dropdown', test_id: 'dropdown-toggle' } %>
         <div class="dropdown-menu dropdown-menu-right" >
           <div data-toggle="modal" data-target="#modal">
             <% if can?(:update_customer, @order) && !@order.user.present? %>
@@ -17,7 +17,6 @@
             <% end %>
           </div>
           <% if can?(:update_customer, @order) %>
-            <div class="dropdown-divider"></div>
             <% if @order.user.present? %>
               <%= link_to Spree.t('admin.orders.remove_user_from_this_order'), spree.admin_order_user_path(@order), class: 'dropdown-item btn-danger', data: { turbo_method: :delete, turbo_confirm: Spree.t(:are_you_sure), turbo_frame: :_top } %>
             <% else %>

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

@@ -23,28 +23,24 @@
   <div class="d-flex flex-column flex-lg-row gap-2">
     <%= render 'spree/admin/shared/filters_search_bar', param: :multi_search %>
 
-    <div class="d-flex align-items-center justify-content-between">
-      <div class="mr-2 h-100">
-        <% if controller_name == 'checkouts' %>
-          <%= render "spree/admin/shared/calendar_range_picker",
-          date_from_input_name: "q[created_at_gt]",
-          date_to_input_name: "q[created_at_lt]",
-          date_from_value: params.dig(:q, :created_at_gt),
-          date_to_value: params.dig(:q, :created_at_lt),
-          dropdown_direction: "right" %>
-        <% else %>
-          <%= render "spree/admin/shared/calendar_range_picker",
-          date_from_input_name: "q[completed_at_gt]",
-          date_to_input_name: "q[completed_at_lt]",
-          date_from_value: params.dig(:q, :completed_at_gt),
-          date_to_value: params.dig(:q, :completed_at_lt),
-          dropdown_direction: "right" %>
-        <% end %>
-      </div>
+    <% if controller_name == 'checkouts' %>
+      <%= render "spree/admin/shared/calendar_range_picker",
+      date_from_input_name: "q[created_at_gt]",
+      date_to_input_name: "q[created_at_lt]",
+      date_from_value: params.dig(:q, :created_at_gt),
+      date_to_value: params.dig(:q, :created_at_lt),
+      dropdown_direction: "right" %>
+    <% else %>
+      <%= render "spree/admin/shared/calendar_range_picker",
+      date_from_input_name: "q[completed_at_gt]",
+      date_to_input_name: "q[completed_at_lt]",
+      date_from_value: params.dig(:q, :completed_at_gt),
+      date_to_value: params.dig(:q, :completed_at_lt),
+      dropdown_direction: "right" %>
+    <% end %>
 
-      <%= render "spree/admin/orders/table_filter_dropdown" %>
-      <%= render 'spree/admin/shared/filters_button' %>
-    </div>
+    <%= render "spree/admin/orders/table_filter_dropdown" %>
+    <%= render 'spree/admin/shared/filters_button' %>
   </div>
   <div data-reveal-target="item" class="d-none" id="table-filter">
     <%= f.hidden_field :payment_state_eq, value: params.dig(:q, :payment_state_eq) %>

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

@@ -1,7 +1,7 @@
 <% if @orders.any? %>
   <div class="table-responsive">
     <table class="table position-relative" id="listing_orders">
-      <thead class="text-muted">
+      <thead>
         <tr>
           <th><%= I18n.t(:number, scope: 'activerecord.attributes.spree/order') %></th>
           <% if controller_name == 'orders' %>

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

@@ -10,7 +10,7 @@
     <div class="card-body p-0">
       <div class="table-responsive">
         <table class="table table-condensed" id='refunds' data-order-id='<%= @order.number %>'>
-          <thead class="text-muted">
+          <thead>
             <tr data-hook="refunds_header" class="border-bottom">
               <th><%= Spree.t(:amount) %></th>
               <th><%= Spree.t(:status) %></th>

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

@@ -17,9 +17,7 @@
             </tr>
           </thead>
           <tbody>
-            <%= render partial: "spree/admin/orders/return_authorizations/return_authorization",
-            collection: @return_authorizations,
-            cached: ->(r) { [spree_base_cache_key, r.cache_key_with_version, r.return_items.maximum(:updated_at)] } %>
+            <%= render partial: 'spree/admin/orders/return_authorizations/return_authorization', collection: @return_authorizations, cached: spree_base_cache_scope %>
           </tbody>
         </table>
       </div>

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

@@ -65,7 +65,7 @@
           <strong><%= Spree.t(:tracking) %>: </strong>
           <% if shipment.tracked? %>
             <% if shipment.tracking_url.present? %>
-              <span><%= link_to shipment.tracking, shipment.tracking_url, target: '_blank', rel: "noopener noreferrer" %></span>
+              <span><%= link_to ERB::Util.html_escape(shipment.tracking), ERB::Util.html_escape(shipment.tracking_url), target: '_blank', rel: "noopener noreferrer" %></span>
             <% elsif shipment.tracking.present? %>
               <span><%= shipment.tracking %></span>
             <% end %>

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

@@ -1,6 +1,6 @@
 <% if controller_name == 'orders' %>
-  <div class="dropdown h-100 mr-2">
-    <button class="btn btn-light h-100 dropdown-toggle" type="button" data-toggle="dropdown" aria-expanded="false">
+  <div class="dropdown">
+    <button class="btn btn-light dropdown-toggle" type="button" data-toggle="dropdown" aria-expanded="false">
       <span class="text-muted font-weight-normal mr-1">
         <%= Spree.t(:show) %>
       </span>

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

@@ -35,7 +35,7 @@
             data-toggle="dropdown"
             aria-expanded="false"
           >
-            <%= icon("dots", class: "mr-0") %>
+            <%= icon('dots-vertical', class: "mr-0") %>
           </button>
           <div class="dropdown-menu">
             <%= link_to_edit(

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

@@ -1,5 +1,5 @@
 <table class="show return-items-table table table-condensed index">
-  <thead class="text-muted">
+  <thead>
     <tr>
       <th><%= Spree.t(:product) %></th>
       <th><%= Spree.t(:pre_tax_amount) %></th>

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

@@ -3,7 +3,7 @@
 <div data-controller="return-items">
   <div class="table-responsive card-lg mb-4">
     <table class="table table-condensed return-items-table">
-      <thead class="text-muted">
+      <thead>
         <tr>
           <th>
             <% if allow_return_item_changes %>

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

@@ -27,7 +27,7 @@ locals: {
 <div class="card-lg mb-4">
   <div class="table-responsive">
     <table class="table table-condensed return-items-table">
-      <thead class="text-muted">
+      <thead>
         <tr>
           <th><%= Spree.t(:product) %></th>
           <th><%= Spree.t(:status) %></th>
@@ -102,7 +102,7 @@ locals: {
     </div>
     <div class="card-body p-0">
       <table class="table table-condensed">
-        <thead class="text-muted">
+        <thead>
           <tr>
             <th><%= Spree.t(:reimbursement_type) %></th>
             <th><%= Spree.t(:description) %></th>

data/app/views/spree/admin/page_blocks/_form_tab_buttons.html.erb

@@ -1,7 +1,5 @@
-<div class="position-fixed border-top px-3 py-2 d-flex bottom-bar" style="bottom: 0px; left: 0px; width: 344px; z-index: 9999">
-  <% if @page_block.can_be_deleted? %>
-    <div data-action="click->page-builder#hideSidebarMobile">
-      <%= link_to_delete(@page_block, url: spree.admin_page_section_block_path(@page_block.section, @page_block)) %>
-    </div>
-  <% end %>
-</div>
+<% if @page_block.can_be_deleted? %>
+  <div class="sidebar-footer">
+    <%= link_to_delete(@page_block, url: spree.admin_page_section_block_path(@page_block.section, @page_block)) %>
+  </div>
+<% end %>

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

@@ -9,7 +9,7 @@
   <div class="px-3 pb-5 mb-5 edit-block" style="margin-bottom: 6rem;">
     <%= form_for @page_block, url: spree.admin_page_section_block_path(@page_block.section, @page_block), data: { controller: 'auto-submit' }, as: :page_block do |f| %>
       <div data-controller="bootstrap-tabs">
-        <ul class="nav nav-pills nav-fill mb-3 px-0" id="pills-tab" role="tablist">
+        <ul class="nav nav-pills nav-fill mb-3" id="pills-tab" role="tablist">
           <li class="nav-item" role="presentation">
             <button class="btn nav-link w-100 <%= tab_selected == :content ? 'active' : '' %>" id="pills-home-tab" data-toggle="pill" data-target="#pills-home" type="button" role="tab" aria-controls="pills-home" aria-selected="<%= tab_selected == :content %>" data-action="bootstrap-tabs#change" data-bootstrap-tabs-tab-param="content">Content</button>
           </li>
@@ -36,10 +36,8 @@
           <%= render 'spree/admin/page_sections/fields/button', f: f %>
           <%= yield(:design_tab) %>
 
-          <div class="position-fixed border-top bg-light bottom-bar" style="bottom: 0px; width:344px; z-index: 9999">
-            <div class="px-3 py-2" data-action="click->page-builder#hideSidebarMobile">
-              <%= link_to_delete(@page_block, url: spree.admin_page_section_block_path(@page_block.section, @page_block)) %>
-            </div>
+          <div class="sidebar-footer">
+            <%= link_to_delete(@page_block, url: spree.admin_page_section_block_path(@page_block.section, @page_block)) %>
           </div>
         </div>
       </div>

data/app/views/spree/admin/page_builder/_add_block.html.erb

@@ -1,4 +1,4 @@
-<%= form_for section.blocks.new, url: spree.admin_page_section_blocks_path(section), html: { class: 'list-group-item sidebar-page-block px-0 py-0 border-0 rounded'}, data: { controller: 'block-form', block_form_target: :form } do |f| %>
+<%= form_for section.blocks.new, url: spree.admin_page_section_blocks_path(section), html: { class: 'list-group-item sidebar-page-block px-0 py-1 border-0 rounded'}, data: { controller: 'block-form', block_form_target: :form } do |f| %>
   <%= f.hidden_field :type, data: { block_form_target: 'type' } %>
 
   <button class="btn btn-sm btn-link d-flex-inline" data-toggle="dropdown">

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

@@ -1,31 +1,38 @@
 <header id="page-builder-header" class="fixed-top">
   <nav class="navbar navbar-expand-lg">
-    <%= link_to spree.admin_themes_path, class: 'btn btn-light with-tip px-2', title: 'Go back to dashboard' do %>
+    <%= link_to spree.admin_themes_path, class: 'btn hover-gray with-tip px-2', title: Spree.t('admin.back_to_dashboard') do %>
       <%= icon('x', class: 'mr-0') %>
     <% end %>
-    <div class="px-3">
-      <span class="text-muted mr-1"><%= Spree.t(:theme) %>:</span>
-      <strong class="mr-1"><%= @theme.name %></strong>
-      &nbsp;
-      <% if @theme.default? %>
-        <span class="badge badge-success">
-          <%= icon('check') %>
-          Live
-        </span>
-      <% else %>
-        <span class="badge badge-warning">Draft</span>
-      <% end %>
-    </div>
-    <%= render 'spree/admin/page_builder/pages_dropdown' %>
+    <% if @page.custom? %>
+      <div class="px-3">
+        <span class="text-muted mr-1"><%= I18n.t('activerecord.models.spree/page.one') %>:</span>
+        <strong><%= @page.display_name %></strong>
+      </div>
 
-    <ul class="nav nav-pills" role="tablist">
-      <% if @page.custom? %>
-        <li class="nav-item" role="presentation" data-toggle="modal" data-target="#modal">
-          <%= link_to_with_icon 'settings', Spree.t(:page_settings), spree.edit_admin_page_path(@page.id), class: 'btn btn-light', data: { turbo_frame: 'dialog_modal' } %>
-        </li>
-      <% else %>
+      <span data-toggle="modal" data-target="#modal">
+        <%= link_to_with_icon 'settings', Spree.t(:settings), spree.edit_admin_page_path(@page.id), class: 'btn btn-light', data: { turbo_frame: 'dialog_modal' } %>
+      </span>
+    <% else %>
+      <div class="px-3">
+        <span class="text-muted mr-1"><%= Spree.t(:theme) %>:</span>
+        <strong class="mr-1"><%= @theme.name %></strong>
+        &nbsp;
+        <% if @theme.default? %>
+          <span class="badge badge-success">
+            <%= icon('check') %>
+            <%= Spree.t(:live) %>
+          </span>
+        <% else %>
+          <span class="badge badge-warning">
+            <%= Spree.t(:draft) %>
+          </span>
+        <% end %>
+      </div>
+      <%= render 'spree/admin/page_builder/pages_dropdown' %>
+
+      <ul class="nav nav-pills" role="tablist">
         <li class="nav-item" role="presentation">
-          <button class="nav-link active"
+          <button class="nav-link active with-tip"
                         title="Page sections"
                         data-toggle="tab"
                         data-target="#sections"
@@ -39,8 +46,8 @@
           </button>
         </li>
         <li class="nav-item" role="presentation">
-          <button class="nav-link"
-                  title="Theme"
+          <button class="nav-link with-tip"
+                  title="<%= Spree.t(:theme) %>"
                   data-toggle="tab"
                   data-target="#colors"
                   type="button"
@@ -53,7 +60,7 @@
           </button>
         </li>
         <li class="nav-item" role="presentation">
-          <button class="nav-link"
+          <button class="nav-link with-tip"
                 title="Fonts"
                 type="button"
                 data-toggle="tab"
@@ -67,8 +74,8 @@
             <%= icon 'typography', class: 'mr-0' %>
           </button>
         </li>
-      <% end %>
-    </ul>
+      </ul>
+    <% end %>
 
     <ul class="nav nav-pills ml-auto" role="tablist">
       <li class="nav-item" role="presentation">
@@ -94,6 +101,6 @@
     </ul>
 
     <%= link_to_with_icon 'eye', Spree.t(:preview), @page.preview_url(@theme_preview, @page_preview), class: 'btn btn-light ml-auto', target: '_blank' %>
-    <%= turbo_save_button_tag(@theme.default? ? Spree.t('actions.publish') : Spree.t('actions.save')) %>
+    <%= turbo_save_button_tag(@theme.default? ? icon('cloud-up') + Spree.t('actions.publish') : icon('pencil-check') + Spree.t('actions.save')) %>
   </nav>
 </header>

data/app/views/spree/admin/page_builder/_pages_dropdown.html.erb

@@ -1,23 +1,17 @@
-<% if @page.custom? %>
-  <div class="d-flex justify-content-center h-100 align-items-center px-3">
-    <p class="mb-0 font-weight-bold"><%= truncate(@page.display_name, length: 150) %></p>
-  </div>
-<% else %>
-  <div class="dropdown h-100" id="pages_dropdown">
-    <button class="btn btn-light" type="button" data-toggle="dropdown" aria-expanded="false">
-      <span class="d-flex align-items-center pr-5">
-        <%= icon @page.icon_name, class: 'mr-2' %>
-        <%= @page.display_name %>
-      </span>
-      <%= icon 'selector', class: 'mr-0 ml-2 mb-0 text-muted' %>
-    </button>
-    <div class="dropdown-menu" style="min-width: 300px">
-      <% @theme.pages.standard.order(name: :asc).find_all(&:customizable?).each do |page| %>
-        <%= active_link_to spree.edit_admin_theme_path(@theme, theme_preview_id: @theme_preview.id, page_id: page.id), class: 'dropdown-item', active: @page == page do %>
-          <%= icon page.icon_name, class: 'mr-2' %>
-          <%= page.display_name %>
-        <% end %>
+<div class="dropdown h-100" id="pages_dropdown">
+  <button class="btn btn-light" type="button" data-toggle="dropdown" aria-expanded="false">
+    <span class="d-flex align-items-center pr-5">
+      <%= icon @page.icon_name, class: 'mr-2' %>
+      <%= @page.display_name %>
+    </span>
+    <%= icon 'selector', class: 'mr-0 ml-2 mb-0 text-muted' %>
+  </button>
+  <div class="dropdown-menu" style="min-width: 300px">
+    <% @theme.pages.standard.order(name: :asc).find_all(&:customizable?).each do |page| %>
+      <%= active_link_to spree.edit_admin_theme_path(@theme, theme_preview_id: @theme_preview.id, page_id: page.id), class: 'dropdown-item', active: @page == page do %>
+        <%= icon page.icon_name, class: 'mr-2' %>
+        <%= page.display_name %>
       <% end %>
-    </div>
+    <% end %>
   </div>
-<% end %>
+</div>

data/app/views/spree/admin/page_builder/_sidebar.html.erb

@@ -1,4 +1,4 @@
-<aside id="page-builder-sidebar" class="fixed-top h-100 overflow-auto pb-5">
+<aside id="page-builder-sidebar">
   <div class="tab-content mb5">
     <div class="tab-pane fade show active" id="sections" role="tabpanel" aria-labelledby="sidebar-icon-page">
       <%= render 'spree/admin/page_builder/sidebar_sections' %>

data/app/views/spree/admin/page_builder/_sidebar_block.html.erb

@@ -20,7 +20,7 @@
       </div>
       <div class="d-flex align-items-center d-none">
         <% if block.section.can_sort_blocks? %>
-          <button class="btn btn-sm pr-0 handle hover-gray-200 h-100 mr-1 px-1 handle-block">
+          <button class="btn btn-sm pr-0 handle hover-gray-200 h-100 px-1 handle-block">
             <%= icon('grip-vertical', class: 'mr-0') %>
           </button>
         <% end %>

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

@@ -26,9 +26,9 @@
 
   <% if show_add_link_button %>
     <% if parent.is_a?(Spree::PageSection) %>
-      <% add_url = admin_page_section_links_path(parent) %>
+      <% add_url = spree.admin_page_section_links_path(parent) %>
     <% elsif parent.is_a?(Spree::PageBlock) %>
-      <% add_url = admin_page_section_block_links_path(parent.section, parent) %>
+      <% add_url = spree.admin_page_section_block_links_path(parent.section, parent) %>
     <% end %>
 
     <%= link_to_with_icon 'plus', 'Add link', add_url, class: 'btn btn-primary w-100', data: { turbo_method: :post } if add_url && can?(:create, Spree::PageLink.new(parent: parent)) %>

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

@@ -10,9 +10,7 @@
       <%= render 'spree/admin/page_links/form', f: f %>
     <% end %>
   </div>
-  <div class="position-fixed border-top bg-light bottom-bar" style="bottom: 0px; width:344px; z-index: 9999">
-    <div class="px-3 py-2" data-action="click->page-builder#hideSidebarMobile">
-      <%= link_to_delete(@page_link, url: spree.admin_page_link_path(@page_link)) %>
-    </div>
+  <div class="sidebar-footer">
+    <%= link_to_delete(@page_link, url: spree.admin_page_link_path(@page_link)) %>
   </div>
 <% end %>

data/app/views/spree/admin/page_sections/_form_tab_buttons.html.erb

@@ -1,12 +1,8 @@
 <div class="sidebar-footer">
-  <div class="d-flex align-items-center justify-content-end w-100">
-    <% if tab == :design %>
-      <%= link_to_with_icon "restore", Spree.t(:restore_defaults), spree.restore_design_settings_to_defaults_admin_page_section_path(@page_section), {data: {turbo_confirm: Spree.t(:are_you_sure), turbo_method: :patch }, class: 'btn btn-light'} %>
-    <% end %>
-    <% if @page_section.can_be_deleted? %>
-      <div data-action="click->page-builder#hideSidebarMobile" class="ml-auto">
-        <%= link_to_delete(@page_section, url: spree.admin_page_section_path(@page_section)) %>
-      </div>
-    <% end %>
-  </div>
+  <% if @page_section.can_be_deleted? %>
+    <%= link_to_delete(@page_section, url: spree.admin_page_section_path(@page_section)) %>
+  <% end %>
+  <% if tab == :design %>
+    <%= link_to_with_icon "restore", Spree.t(:restore_defaults), spree.restore_design_settings_to_defaults_admin_page_section_path(@page_section), {data: {turbo_confirm: Spree.t(:are_you_sure), turbo_method: :patch }, class: 'btn btn-light btn-sm ml-auto'} %>
+  <% end %>
 </div>

data/app/views/spree/admin/page_sections/forms/_featured_posts.html.erb

@@ -21,6 +21,11 @@
   <%= f.select :preferred_description_alignment, options_for_select([ ["Left", "left"], ["Center", "center"], ["Right", "right"] ], @page_section.preferred_description_alignment ), {}, data: { action: 'auto-submit#submit' }, class: "custom-select" %>
 </div>
 
+<div class="form-group">
+  <%= f.label :preferred_posts_to_show, Spree.t('admin.page_builder.posts_to_show') %>
+  <%= f.select :preferred_posts_to_show, options_for_select([["Newest", "newest"], ["Most popular", "most_popular"]], @page_section.preferred_posts_to_show), {}, data: { action: 'auto-submit#submit' }, class: "custom-select" %>
+</div>
+
 <div class="form-group">
   <div data-controller="better-slider">
     <label><%= Spree.t('admin.page_builder.maximum_posts_to_show') %>: <span data-better-slider-target="currentValueLabel"></span></label>

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

@@ -11,7 +11,7 @@
       <%= f.hidden_field :pageable_id %>
       <%= f.hidden_field :pageable_type %>
       <% cache spree_base_cache_scope.call('new-section-list'), expires_in: 1.hour do %>
-        <ul class="list-group list-group-flush m-2 mx-3">
+        <ul class="list-group list-group-flush m-2 mx-3 border p-1 rounded-lg bg-white">
           <% @theme.available_page_sections.each do |section_class| %>
             <li class="list-group-item p-2 list-group-item-action cursor-pointer d-flex align-items-center hover-action text-primary"
               data-action="click->section-form#addSection"

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

@@ -1,14 +1,14 @@
 <%= turbo_frame_tag :dialog_modal do %>
   <%= form_with model: @page, url: spree.admin_page_path(@page), data: { controller: 'slug-form seo-form' }, scope: :page do |f| %>
     <div class="modal-content">
-      <%= modal_header(Spree.t(:page_settings)) %>
+      <%= modal_header(@page.display_name) %>
       <div class="modal-body">
         <%= render 'spree/admin/shared/error_messages', target: @page %>
         <%= render 'form', f: f, vertical: true %>
       </div>
       <div class="modal-footer">
         <%= turbo_save_button_tag Spree.t('actions.save'), data: {turbo_frame: :_top} %>
-        <%= link_to Spree.t('actions.destroy'), spree.admin_page_path(@page.id), data: { turbo_method: :delete, turbo_frame: '_top', turbo_confirm: Spree.t(:are_you_sure )}, class: 'btn btn-danger' %>
+        <%= link_to Spree.t('actions.destroy'), spree.admin_page_path(@page.id), data: { turbo_method: :delete, turbo_frame: '_top', turbo_confirm: Spree.t(:are_you_sure )}, class: 'btn btn-danger ml-auto' %>
       </div>
     </div>
   <% end %>

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

@@ -19,8 +19,7 @@
           </tr>
         </thead>
         <tbody>
-          <% default_theme = current_store.default_theme %>
-          <%= render collection: @collection, partial: "spree/admin/pages/page", cached: ->(page) { [*spree_base_cache_scope.call(page), default_theme] } %>
+          <%= render collection: @collection, partial: "spree/admin/pages/page", cached: spree_base_cache_scope %>
         </tbody>
       </table>
       <%= render "spree/admin/shared/index_table_options", collection: @collection %>

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

@@ -5,27 +5,28 @@
 <div class="card-lg">
   <div class="table-responsive">
     <table class="table">
-      <thead class="text-muted">
-      <tr>
-        <th></th>
-        <th><%= Spree.t(:payment_method) %></th>
-        <th><%= Spree.t(:visibility) %></th>
-        <th class="text-center"><%= Spree.t(:active) %></th>
-        <th></th>
-      </tr>
-    </thead>
-    <tbody data-controller="sortable" data-sortable-handle-value=".move-handle" data-sortable-resource-name-value="payment_method" data-sortable-response-kind-value="turbo-stream">
-      <%= render partial: 'spree/admin/payment_methods/payment_method', collection: @payment_methods %>
-    </tbody>
-  </table>
+      <thead>
+        <tr>
+          <th></th>
+          <th><%= Spree.t(:payment_method) %></th>
+          <th><%= Spree.t(:visibility) %></th>
+          <th class="text-center"><%= Spree.t(:active) %></th>
+          <th></th>
+        </tr>
+      </thead>
+      <tbody data-controller="sortable" data-sortable-handle-value=".move-handle" data-sortable-resource-name-value="payment_method" data-sortable-response-kind-value="turbo-stream">
+        <%= render partial: 'spree/admin/payment_methods/payment_method', collection: @payment_methods %>
+      </tbody>
+    </table>
+  </div>
 </div>
 
 <% if available_payment_methods.present? && can?(:create, Spree::PaymentMethod) %>
-  <h5 class="pb-2 pt-3">Available Payment Methods</h5>
+  <h6 class="pb-2 pt-3">Available Payment Methods</h6>
   <div class="card-lg">
     <div class="table-responsive">
       <table class="table">
-        <thead class="text-muted">
+        <thead>
         <tr>
           <th><%= Spree.t(:name) %></th>
           <th><%= Spree.t(:description) %></th>

data/app/views/spree/admin/payments/_payment.html.erb

@@ -16,7 +16,7 @@
 
     <div class="dropdown">
       <button class="btn btn-light btn-sm" type="button" data-toggle="dropdown" aria-expanded="false">
-        <%= icon('dots', class: 'mr-0') %>
+        <%= icon('dots-vertical', class: 'mr-0') %>
       </button>
       <div class="dropdown-menu">
         <% if payment.try(:gateway_dashboard_payment_url).present? %>