From a8623322069a0404dd764b9dc47743118c943642 Mon Sep 17 00:00:00 2001
From: Ricardo Marinovic <marinovic.ricardo@gmail.com>
Date: Tue, 17 Jun 2025 19:46:27 +0200
Subject: [PATCH 1/3] Add action items to show view

---
 .../demo_web/item_actions/user_soft_delete.ex |  3 +
 lib/backpex/html/resource.ex                  | 24 ++----
 .../resource/resource_index_table.html.heex   |  6 +-
 .../html/resource/resource_show.html.heex     | 22 ++++++
 lib/backpex/item_actions/delete.ex            |  6 +-
 lib/backpex/live_components/form_component.ex | 21 +++--
 lib/backpex/live_resource.ex                  | 28 +++----
 lib/backpex/live_resource/index.ex            |  8 +-
 lib/backpex/live_resource/show.ex             | 78 +++++++++++++++++++
 9 files changed, 146 insertions(+), 50 deletions(-)

diff --git a/demo/lib/demo_web/item_actions/user_soft_delete.ex b/demo/lib/demo_web/item_actions/user_soft_delete.ex
index da02b6c6c..ab9938866 100644
--- a/demo/lib/demo_web/item_actions/user_soft_delete.ex
+++ b/demo/lib/demo_web/item_actions/user_soft_delete.ex
@@ -75,9 +75,12 @@ defmodule DemoWeb.ItemActions.UserSoftDelete do
             Backpex.Resource.update_all(item.posts, [set: [user_id: nil]], "updated", DemoWeb.PostLive)
           end)
 
+        %{live_resource: live_resource, params: params} = socket.assigns
+
         socket
         |> clear_flash()
         |> put_flash(:info, success_message(socket.assigns, items))
+        |> assign(:return_to, Router.get_path(socket, live_resource, params, :index))
       rescue
         error ->
           Logger.error("An error occurred while deleting the resource: #{inspect(error)}")
diff --git a/lib/backpex/html/resource.ex b/lib/backpex/html/resource.ex
index c663ed53a..f6905ce7d 100644
--- a/lib/backpex/html/resource.ex
+++ b/lib/backpex/html/resource.ex
@@ -643,7 +643,7 @@ defmodule Backpex.HTML.Resource do
       <div :if={display_divider?(assigns)} class="border-base-300 my-0.5 border-r-2 border-solid" />
 
       <button
-        :for={{key, action} <- index_item_actions(@item_actions)}
+        :for={{key, action} <- item_actions_for(:index, @item_actions)}
         class="btn btn-sm btn-outline btn-primary"
         disabled={action_disabled?(assigns, key, @selected_items)}
         phx-click="item-action"
@@ -705,24 +705,22 @@ defmodule Backpex.HTML.Resource do
   end
 
   defp display_divider?(assigns) do
-    index_item_actions = index_item_actions(assigns.item_actions)
+    index_item_actions = item_actions_for(:index, assigns.item_actions)
     resource_actions = resource_actions(assigns, assigns.resource_actions)
 
     Enum.any?(index_item_actions) &&
       (Enum.any?(resource_actions) || assigns.live_resource.can?(assigns, :new, nil))
   end
 
-  defp index_item_actions(item_actions) do
+  def item_actions_for(place, item_actions) do
     Enum.filter(item_actions, fn {_key, action} ->
-      action_on_index?(action)
+      action_enabled?(action, place)
     end)
   end
 
-  defp row_item_actions(item_actions) do
-    Enum.filter(item_actions, fn {_key, action} ->
-      action_on_row?(action)
-    end)
-  end
+  defp action_enabled?(%{only: only}, place), do: place in only
+  defp action_enabled?(%{except: except}, place), do: place in except
+  defp action_enabled?(_action, _place), do: true
 
   defp action_disabled?(assigns, action_key, items) do
     Enum.filter(items, fn item ->
@@ -731,14 +729,6 @@ defmodule Backpex.HTML.Resource do
     |> Enum.empty?()
   end
 
-  defp action_on_row?(%{only: only}), do: :row in only
-  defp action_on_row?(%{except: except}), do: :row not in except
-  defp action_on_row?(_action), do: true
-
-  defp action_on_index?(%{only: only}), do: :index in only
-  defp action_on_index?(%{except: except}), do: :index not in except
-  defp action_on_index?(_action), do: true
-
   @doc """
   Renders an info block to indicate that no items are found.
   """
diff --git a/lib/backpex/html/resource/resource_index_table.html.heex b/lib/backpex/html/resource/resource_index_table.html.heex
index b6e45a66f..90c210ea4 100644
--- a/lib/backpex/html/resource/resource_index_table.html.heex
+++ b/lib/backpex/html/resource/resource_index_table.html.heex
@@ -1,7 +1,7 @@
 <table class="table">
   <thead class="bg-base-100 text-base-content uppercase">
     <tr id="row-headers" phx-hook="BackpexStickyActions" class="border-b-2">
-      <th :if={Enum.any?(index_item_actions(@item_actions))}>
+      <th :if={Enum.any?(item_actions_for(:index, @item_actions))}>
         <input
           phx-click="toggle-item-selection"
           type="checkbox"
@@ -36,7 +36,7 @@
       class={index_row_class(assigns, item, selected?(@selected_items, item), index)}
       phx-hook="BackpexStickyActions"
     >
-      <td :if={Enum.any?(index_item_actions(@item_actions))} class="relative">
+      <td :if={Enum.any?(item_actions_for(:index, @item_actions))} class="relative">
         <div :if={selected?(@selected_items, item)} class="bg-base-content absolute inset-y-0 left-0 w-0.5" />
         <input
           id={"select-input-#{LiveResource.primary_value(item, @live_resource)}"}
@@ -65,7 +65,7 @@
       ]}>
         <div class={["flex items-center justify-end space-x-2"]}>
           <button
-            :for={{key, action} <- row_item_actions(@item_actions)}
+            :for={{key, action} <- item_actions_for(:row, @item_actions)}
             :if={@live_resource.can?(assigns, key, item)}
             id={"item-action-#{key}-#{LiveResource.primary_value(item, @live_resource)}"}
             type="button"
diff --git a/lib/backpex/html/resource/resource_show.html.heex b/lib/backpex/html/resource/resource_show.html.heex
index 56f20457a..b50135b57 100644
--- a/lib/backpex/html/resource/resource_show.html.heex
+++ b/lib/backpex/html/resource/resource_show.html.heex
@@ -1,4 +1,26 @@
 <div>
+  <.modal
+    :if={@action_to_confirm}
+    id="action-confirm-modal"
+    title={@action_to_confirm.module.label(assigns, nil)}
+    on_cancel={JS.push("cancel-action-confirm")}
+    open={true}
+    close_label={Backpex.__("Close modal", @live_resource)}
+  >
+    <div class="px-5 py-3">
+      {@action_to_confirm.module.confirm(assigns)}
+    </div>
+    <div>
+      <.live_component
+        module={Backpex.FormComponent}
+        id={:item_action_modal}
+        live_resource={@live_resource}
+        action_type={:item}
+        {Map.drop(assigns, [:socket, :flash, :fields])}
+      />
+    </div>
+  </.modal>
+
   {@live_resource.render_resource_slot(assigns, :show, :before_page_title)}
   {@live_resource.render_resource_slot(assigns, :show, :page_title)}
 
diff --git a/lib/backpex/item_actions/delete.ex b/lib/backpex/item_actions/delete.ex
index 4c8c83545..23ef867c2 100644
--- a/lib/backpex/item_actions/delete.ex
+++ b/lib/backpex/item_actions/delete.ex
@@ -42,13 +42,15 @@ defmodule Backpex.ItemActions.Delete do
 
   @impl Backpex.ItemAction
   def handle(socket, items, _data) do
-    {:ok, deleted_items} = Resource.delete_all(items, socket.assigns.live_resource)
+    %{live_resource: live_resource, params: params} = socket.assigns
+    {:ok, deleted_items} = Resource.delete_all(items, live_resource)
 
-    Enum.each(deleted_items, fn deleted_item -> socket.assigns.live_resource.on_item_deleted(socket, deleted_item) end)
+    Enum.each(deleted_items, fn deleted_item -> live_resource.on_item_deleted(socket, deleted_item) end)
 
     socket
     |> clear_flash()
     |> put_flash(:info, success_message(socket.assigns, deleted_items))
+    |> assign(:return_to, Router.get_path(socket, live_resource, params, :index))
     |> ok()
   rescue
     error ->
diff --git a/lib/backpex/live_components/form_component.ex b/lib/backpex/live_components/form_component.ex
index 3fe455173..5a976b2fd 100644
--- a/lib/backpex/live_components/form_component.ex
+++ b/lib/backpex/live_components/form_component.ex
@@ -76,7 +76,7 @@ defmodule Backpex.FormComponent do
   end
 
   def handle_event("validate", %{"change" => change, "_target" => target}, %{assigns: %{action_type: :item}} = socket) do
-    %{assigns: %{item: item, fields: fields} = assigns} = socket
+    %{assigns: %{form_item: item, fields: fields} = assigns} = socket
 
     changeset_function = &assigns.action_to_confirm.module.changeset/3
 
@@ -107,7 +107,7 @@ defmodule Backpex.FormComponent do
   def handle_event("validate", %{"change" => change, "_target" => target}, socket) do
     %{
       live_resource: live_resource,
-      item: item,
+      form_item: item,
       fields: fields
     } = socket.assigns
 
@@ -159,7 +159,7 @@ defmodule Backpex.FormComponent do
       |> Map.get(:removed_uploads, [])
       |> Keyword.update(upload_key, [file_key], fn existing -> [file_key | existing] end)
 
-    files = Upload.existing_file_paths(field, socket.assigns.item, Keyword.get(removed_uploads, upload_key, []))
+    files = Upload.existing_file_paths(field, socket.assigns.form_item, Keyword.get(removed_uploads, upload_key, []))
     uploaded_files = Keyword.put(socket.assigns[:uploaded_files], upload_key, files)
 
     socket
@@ -207,7 +207,7 @@ defmodule Backpex.FormComponent do
   defp handle_save(socket, key, params, save_type \\ "save")
 
   defp handle_save(socket, :new, params, save_type) do
-    %{assigns: %{live_resource: live_resource, item: item, live_action: live_action} = assigns} = socket
+    %{assigns: %{live_resource: live_resource, form_item: item, live_action: live_action} = assigns} = socket
 
     opts = [
       assocs: Map.get(assigns, :assocs, []),
@@ -251,7 +251,7 @@ defmodule Backpex.FormComponent do
   defp handle_save(socket, :edit, params, save_type) do
     %{
       live_resource: live_resource,
-      item: item,
+      form_item: item,
       fields: fields,
       live_action: live_action
     } = socket.assigns
@@ -301,7 +301,7 @@ defmodule Backpex.FormComponent do
         %{
           live_resource: live_resource,
           resource_action: resource_action,
-          item: item,
+          form_item: item,
           return_to: return_to,
           fields: fields
         } = assigns
@@ -353,8 +353,7 @@ defmodule Backpex.FormComponent do
           live_resource: live_resource,
           selected_items: selected_items,
           action_to_confirm: action_to_confirm,
-          fields: fields,
-          return_to: return_to
+          fields: fields
         } = assigns
     } = socket
 
@@ -366,7 +365,7 @@ defmodule Backpex.FormComponent do
 
         metadata = Resource.build_changeset_metadata(assigns)
 
-        assigns.item
+        assigns.form_item
         |> changeset_function.(params, metadata)
         |> Map.put(:action, :insert)
         |> Ecto.Changeset.apply_action(:insert)
@@ -381,7 +380,7 @@ defmodule Backpex.FormComponent do
       |> assign(:show_form_errors, false)
       |> assign(:selected_items, [])
       |> assign(:select_all, false)
-      |> push_patch(to: return_to)
+      |> push_navigate(to: socket.assigns.return_to)
       |> noreply()
     else
       {:error, changeset} ->
@@ -450,7 +449,7 @@ defmodule Backpex.FormComponent do
         uploaded_entries = uploaded_entries(socket, upload_key)
         removed_entries = Keyword.get(socket.assigns.removed_uploads, upload_key, [])
 
-        change = put_upload_change.(socket, acc, socket.assigns.item, uploaded_entries, removed_entries, action)
+        change = put_upload_change.(socket, acc, socket.assigns.form_item, uploaded_entries, removed_entries, action)
 
         upload_used_input_data = Map.get(change, "#{to_string(name)}_used_input")
         used_input? = upload_used_input_data != "false"
diff --git a/lib/backpex/live_resource.ex b/lib/backpex/live_resource.ex
index 2d5e89a6f..40faef867 100644
--- a/lib/backpex/live_resource.ex
+++ b/lib/backpex/live_resource.ex
@@ -407,19 +407,21 @@ defmodule Backpex.LiveResource do
         ~H"""
         <.main_title class="flex items-center justify-between">
           {@page_title}
-          <.link
-            :if={@live_resource.can?(assigns, :edit, @item)}
-            id={"#{@live_resource.singular_name()}-edit-link"}
-            phx-hook="BackpexTooltip"
-            data-tooltip={Backpex.__("Edit", @live_resource)}
-            aria-label={Backpex.__("Edit", @live_resource)}
-            patch={Router.get_path(@socket, @live_resource, @params, :edit, @item)}
-          >
-            <Backpex.HTML.CoreComponents.icon
-              name="hero-pencil-square"
-              class="h-6 w-6 cursor-pointer transition duration-75 hover:text-primary hover:scale-110"
-            />
-          </.link>
+          <div class={["flex items-center justify-end space-x-2"]}>
+            <button
+              :for={{key, action} <- item_actions_for(:show, @item_actions)}
+              :if={@live_resource.can?(assigns, key, @item)}
+              id={"item-action-#{key}-#{LiveResource.primary_value(@item, @live_resource)}"}
+              type="button"
+              phx-click="item-action"
+              phx-value-action-key={key}
+              aria-label={action.module.label(assigns, @item)}
+              phx-hook="BackpexTooltip"
+              data-tooltip={action.module.label(assigns, @item)}
+            >
+              {action.module.icon(assigns, @item)}
+            </button>
+          </div>
         </.main_title>
         """
       end
diff --git a/lib/backpex/live_resource/index.ex b/lib/backpex/live_resource/index.ex
index 6e029b333..7777393fb 100644
--- a/lib/backpex/live_resource/index.ex
+++ b/lib/backpex/live_resource/index.ex
@@ -220,7 +220,7 @@ defmodule Backpex.LiveResource.Index do
 
   def handle_event("cancel-action-confirm", _params, socket) do
     socket
-    |> assign(:item, nil)
+    |> assign(:form_item, nil)
     |> assign(:changeset, nil)
     |> assign(:action_to_confirm, nil)
     |> noreply()
@@ -269,7 +269,7 @@ defmodule Backpex.LiveResource.Index do
       changeset = changeset_function.(base_schema, %{}, metadata)
 
       socket
-      |> assign(:item, base_schema)
+      |> assign(:form_item, base_schema)
       |> assign(:changeset, changeset)
     else
       assign(socket, :changeset, %{})
@@ -387,7 +387,7 @@ defmodule Backpex.LiveResource.Index do
     socket
     |> assign(:page_title, socket.assigns.live_resource.plural_name())
     |> apply_index()
-    |> assign(:item, nil)
+    |> assign(:form_item, nil)
   end
 
   defp apply_action(socket, :resource_action) do
@@ -409,7 +409,7 @@ defmodule Backpex.LiveResource.Index do
     |> assign(:page_title, ResourceAction.name(action, :title))
     |> assign(:resource_action, action)
     |> assign(:resource_action_id, id)
-    |> assign(:item, item)
+    |> assign(:form_item, item)
     |> apply_index()
     |> assign(:changeset_function, changeset_function)
     |> assign_changeset(changeset_function, item, action.module.fields(), :resource_action)
diff --git a/lib/backpex/live_resource/show.ex b/lib/backpex/live_resource/show.ex
index e9a089f9b..ed378e7bd 100644
--- a/lib/backpex/live_resource/show.ex
+++ b/lib/backpex/live_resource/show.ex
@@ -23,9 +23,12 @@ defmodule Backpex.LiveResource.Show do
     |> assign(:panels, live_resource.panels())
     |> assign(:fluid?, live_resource.config(:fluid?))
     |> assign(:page_title, live_resource.singular_name())
+    |> assign(:selected_items, [])
+    |> assign(:action_to_confirm, nil)
     |> assign(:params, params)
     |> assign_fields()
     |> assign_item()
+    |> assign_item_actions()
     |> ok()
   end
 
@@ -39,6 +42,27 @@ defmodule Backpex.LiveResource.Show do
     |> noreply()
   end
 
+  def handle_info(_event, socket) do
+    noreply(socket)
+  end
+
+  def handle_event("item-action", %{"action-key" => key}, socket) do
+    item = socket.assigns.item
+
+    socket
+    |> assign(selected_items: [item])
+    |> maybe_handle_item_action(key)
+    |> noreply()
+  end
+
+  def handle_event("cancel-action-confirm", _params, socket) do
+    socket
+    |> assign(:form_item, nil)
+    |> assign(:changeset, nil)
+    |> assign(:action_to_confirm, nil)
+    |> noreply()
+  end
+
   def handle_event(_event, _params, socket) do
     socket
     |> noreply()
@@ -70,4 +94,58 @@ defmodule Backpex.LiveResource.Show do
 
     assign(socket, :fields, fields)
   end
+
+  defp assign_item_actions(socket) do
+    item_actions = Backpex.ItemAction.default_actions() |> socket.assigns.live_resource.item_actions()
+    assign(socket, :item_actions, item_actions)
+  end
+
+  defp maybe_handle_item_action(socket, key) do
+    key = String.to_existing_atom(key)
+    action = socket.assigns.item_actions[key]
+    item = socket.assigns.item
+
+    if Backpex.ItemAction.has_confirm_modal?(action) do
+      open_action_confirm_modal(socket, action, key)
+    else
+      handle_item_action(socket, action, item)
+    end
+  end
+
+  defp open_action_confirm_modal(socket, action, key) do
+    if Backpex.ItemAction.has_form?(action) do
+      changeset_function = &action.module.changeset/3
+      base_schema = action.module.base_schema(socket.assigns)
+
+      metadata = Resource.build_changeset_metadata(socket.assigns)
+      changeset = changeset_function.(base_schema, %{}, metadata)
+
+      socket
+      |> assign(:form_item, base_schema)
+      |> assign(:changeset, changeset)
+    else
+      assign(socket, :changeset, %{})
+    end
+    |> assign(:action_to_confirm, Map.put(action, :key, key))
+  end
+
+  defp handle_item_action(socket, action, item) do
+    case action.module.handle(socket, [item], %{}) do
+      {:ok, socket} ->
+        socket
+        |> assign(action_to_confirm: nil)
+        |> assign(selected_items: [])
+        |> assign(select_all: false)
+
+      unexpected_return ->
+        raise ArgumentError, """
+        Invalid return value from #{inspect(action.module)}.handle/3.
+
+        Expected: {:ok, socket}
+        Got: #{inspect(unexpected_return)}
+
+        Item Actions with no form fields must return {:ok, socket}.
+        """
+    end
+  end
 end

From 116b96297ee16cf77ca2a9cf53a6b9762220aa4f Mon Sep 17 00:00:00 2001
From: Ricardo Marinovic <marinovic.ricardo@gmail.com>
Date: Tue, 17 Jun 2025 20:27:23 +0200
Subject: [PATCH 2/3] Update docs

---
 guides/actions/item-actions.md | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/guides/actions/item-actions.md b/guides/actions/item-actions.md
index dca426950..f71e6cebe 100644
--- a/guides/actions/item-actions.md
+++ b/guides/actions/item-actions.md
@@ -77,12 +77,13 @@ See `Backpex.ItemAction` for a list of all available callbacks.
 
 ## Placement of Item Actions
 
-Item actions can be placed in the resource table or at the top of it. You can specify the placement of the item action by using the `only` key.
+Item actions can be placed in the index view on resource table, on top of it or in the show viw. You can specify the placement of the item action by using the `only` key.
 
-The only key must provide a list and accepts the following options
+The `only` key must provide a list and accepts the following options
 
 * `:row` - display an icon for each element in the table that can trigger the Item Action for the corresponding element
 * `:index` - display a button at the top of the resource table, which triggers the Item Action for selected items
+* `:show` - display a icon at the top of the resource view, which triggers the Item Action for the corresponding element
 
 The following example shows how to place the `show` item action on the index table rows only.
 

From 40370a07b98c72bc85de7e21a72d98226de39bd6 Mon Sep 17 00:00:00 2001
From: Ricardo Marinovic <marinovic.ricardo@gmail.com>
Date: Tue, 17 Jun 2025 20:49:16 +0200
Subject: [PATCH 3/3] Displays error when the resource is not found.

In the example with soft delete of users, it would be best if the user handles the case to redirect to the index page after deleting, but if it is not done, we display an error message instead of raising an error.

Remove some redundant imports because the BakcpexWeb html already imports it.
---
 lib/backpex/live_resource/form.ex  |  2 --
 lib/backpex/live_resource/index.ex |  2 --
 lib/backpex/live_resource/show.ex  | 18 ++++++++++++------
 3 files changed, 12 insertions(+), 10 deletions(-)

diff --git a/lib/backpex/live_resource/form.ex b/lib/backpex/live_resource/form.ex
index 2c21bd672..a8f23f52d 100644
--- a/lib/backpex/live_resource/form.ex
+++ b/lib/backpex/live_resource/form.ex
@@ -2,8 +2,6 @@ defmodule Backpex.LiveResource.Form do
   @moduledoc false
   use BackpexWeb, :html
 
-  import Phoenix.Component
-
   alias Backpex.LiveResource
   alias Backpex.Resource
 
diff --git a/lib/backpex/live_resource/index.ex b/lib/backpex/live_resource/index.ex
index 7777393fb..e153978e8 100644
--- a/lib/backpex/live_resource/index.ex
+++ b/lib/backpex/live_resource/index.ex
@@ -2,8 +2,6 @@ defmodule Backpex.LiveResource.Index do
   @moduledoc false
   use BackpexWeb, :html
 
-  import Phoenix.Component
-
   alias Backpex.Adapters.Ecto, as: EctoAdapter
   alias Backpex.LiveResource
   alias Backpex.Resource
diff --git a/lib/backpex/live_resource/show.ex b/lib/backpex/live_resource/show.ex
index ed378e7bd..bd2204432 100644
--- a/lib/backpex/live_resource/show.ex
+++ b/lib/backpex/live_resource/show.ex
@@ -2,7 +2,7 @@ defmodule Backpex.LiveResource.Show do
   @moduledoc false
   use BackpexWeb, :html
 
-  import Phoenix.Component
+  import Phoenix.LiveView
 
   alias Backpex.LiveResource
   alias Backpex.Resource
@@ -78,13 +78,19 @@ defmodule Backpex.LiveResource.Show do
     backpex_id = Map.fetch!(params, "backpex_id")
     primary_value = URI.decode(backpex_id)
 
-    item = Resource.get!(primary_value, socket.assigns, live_resource)
+    case Resource.get(primary_value, socket.assigns, live_resource) do
+      {:ok, %{} = item} ->
+        if not live_resource.can?(socket.assigns, :show, item), do: raise(Backpex.ForbiddenError)
 
-    if not live_resource.can?(socket.assigns, :show, item), do: raise(Backpex.ForbiddenError)
+        socket
+        |> assign(:item, item)
+        |> assign(:return_to, Router.get_path(socket, live_resource, params, :show, item))
 
-    socket
-    |> assign(:item, item)
-    |> assign(:return_to, Router.get_path(socket, live_resource, params, :show, item))
+      _ ->
+        socket
+        |> put_flash(:error, "The resource does not exist.")
+        |> push_navigate(to: Router.get_path(socket, live_resource, params, :index))
+    end
   end
 
   defp assign_fields(socket) do