ppff01

ppff01

Hi there,

I’ve been testing Phoenix 1.8 a lot lately and noticed the modal component from 1.7 had disappeared. Since we now have DaisyUI which comes with a Modal component, I figured I’d try to use it. It’s not that easy because if you want to do it the cleanest way, you need to use the browser’s showModal() function. I won’t get into too much detail but after several hours of tests here is what I came up with. I hope it can be useful to some of you here!

The component:

  @doc """
  Modal dialog.
  """
  attr :id, :string, required: true
  attr :on_cancel, JS, default: %JS{}
  slot :inner_block, required: true

  def modal(assigns) do
    ~H"""
    <dialog
      id={@id}
      phx-hook="Modal"
      phx-remove={
        JS.remove_attribute("open")
        |> JS.transition({"ease-out duration-200", "opacity-100", "opacity-0"}, time: 0)
      }
      data-cancel={JS.exec(@on_cancel, "phx-remove")}
      class="modal"
    >
      <.focus_wrap
        id={"#{@id}-container"}
        phx-window-keydown={JS.exec("data-cancel", to: "##{@id}")}
        phx-key="escape"
        phx-click-away={JS.exec("data-cancel", to: "##{@id}")}
        class="modal-box"
      >
        <.button phx-click={JS.exec("data-cancel", to: "##{@id}")} class="btn btn-sm btn-circle btn-ghost absolute right-2 top-2 tx-lg">✕</.button>
        <%= render_slot(@inner_block) %>
      </.focus_wrap>
    </dialog>
    """
  end

Add a new hook which will trigger showModal():

Hooks.Modal = {
    mounted() {
        this.el.showModal();
    },
}

In your liveview, just add the modal:

  <.modal
    id="delete_modal"
    :if={@live_action in [:delete]}
    on_cancel={JS.patch(~p"/brands")}
  >
    <h3 class="text-xl text-bold">Warning</h3>
    <p class="text-md mt-3">Are you sure you want to delete brand {@brand.name}?</p>
    <div class="modal-action">
      <.button phx-click="delete" phx-value-id={@brand.id}>Delete</.button>
    </div>
  </.modal>

It’s basically a hot-swap solution for Phoenix modal 1.7. Please note that the modal should be opened only with a specific live_action which is defined in the router:

live "/brands/:id/delete", BrandsLive, :delete

This is necessary because we don’t have an easy way to trigger the close javascript command and retrieve data while letting the browser-operated fade out operate properly (or at least I didn’t find one). But it’s also very useful because it allows us to pass parameters to the modal easily using the url.

Anyway, just my two cents! Feel free to suggest improvements or ask questions.

Showing Posts 1 to 10

ppff01

ppff01 OP

Update: the issue with this implementation is that if you use it with a form for example, the liveview updates will rewrite the html without the open tag on the dialog HTML element, therefore closing the modal.

So here is my final solution: ditch the hook and just include the open tag from the start.

    <dialog
      id={@id}
      phx-remove={
        JS.remove_attribute("open")
        |> JS.transition({"ease-out duration-200", "opacity-100", "opacity-0"}, time: 0)
      }
      data-cancel={JS.exec(@on_cancel, "phx-remove")}
      class="modal"
      open
    >
joshua-bouv

joshua-bouv

Only available in LV 1.1+, but alternatively I believe you can use phx-mounted={JS.ignore_attributes([“open”]}).

It’s a shame the modal got removed from the core_components. I wonder if its because of this and they wanted to keep the components simple?

LostKobrakai

LostKobrakai

core_components only include what phoenix generators use. Modals in generated code eventually got replaced with separate pages, so there was no use for the modal anymore.

ppff01

ppff01 OP

Very interesting! Thank you. I checked the doc and the example is actually given on a dialog :slight_smile:

garrison

garrison

Which is a fantastic change, the modal abuse in the templates was a bit much :slight_smile:

It seems like no matter what anyone says people expect the core components to be a full component system. Hopefully some of the various actual component systems from the community become mature enough to be strongly recommended to new users. I believe Ash has one of those component libraries in their generators now (don’t remember which).

xir

xir

Yesterday I was also testing Phoenix 1.8 + the DaisyUI modal and actually gave up on showModal() and such, being confused by conflicts between what I made and the flash component, which I had also earlier modified using JS hooks to make it disappear after a certain period of time.

So, I went with this purely LiveView solution – good or not. Please criticize.

  @doc """
  Renders a modal dialog.

  The `on_confirm` and `on_cancel` attributes specify the events to trigger
  when the user clicks the respective buttons or dismisses the modal (e.g.,
  pressing Escape or clicking outside).

  ## Examples
      <.link phx-click={JS.push("open", value: %{id: item.id, name: item.name})}>
        Delete
      </.link>

      <.modal :if={@modal} on_confirm="confirm" on_cancel="cancel">
        <div>Are you sure you want to delete {@modal.name}?</div>
      </.modal>
  """
  attr :on_confirm, :string, required: true, doc: "the event to trigger on confirmation"
  attr :on_cancel, :string, required: true, doc: "the event to trigger on cancellation"

  slot :inner_block, required: true, doc: "the content of the modal dialog"

  def modal(assigns) do
    ~H"""
    <div class="fixed inset-0 bg-black/50 flex items-center justify-center z-50">
      <div
        class="bg-white p-6 rounded-xl shadow-xl w-full max-w-md"
        phx-click-away={@on_cancel}
        phx-window-keydown={@on_cancel}
        phx-key="escape"
      >
        {render_slot(@inner_block)}
        <div class="flex justify-end gap-2 mt-8">
          <.button phx-click={@on_cancel}>Cancel</.button>
          <.button phx-click={@on_confirm} variant="primary">Confirm</.button>
        </div>
      </div>
    </div>
    """
  end

and

  @impl true
  def mount(_params, _session, socket) do
    {:ok,
     socket
     |> assign(...)
     |> assign(:modal, nil)
     |> ...}
  end

  @impl true
  def handle_event("open", %{"id" => id, "name" => name}, socket) do
    {:noreply, assign(socket, :modal, %{:id => id, :name => name})}
  end

  @impl true
  def handle_event("confirm", _, socket) do
    # skipped ...
    
    {:noreply,
     socket
     |> assign(...)
     |> assign(:modal, nil)
     |> ...
  end

  @impl true
  def handle_event("cancel", _, socket) do
    {:noreply, assign(socket, :modal, nil)}
  end
end
xir

xir

Yesterday, I’m afraid I wasn’t clear enough. I wanted to ask if it’s really that bad to go with a purely LiveView solution without using any JavaScript, like what I did above.

garrison

garrison

It’s fine. If you want you can combine events with local JS commands to get lower latency for the open/close. But there are legitimate circumstances in which you actually want the modal to be server-rendered. As an example, I have a piece of UI which supports an arbitrary number of stateful modals, some of which can even fetch web content and hold it in-process. This is obviously not something that can be implemented with local JS commands (or even React, interestingly).

If the latency is bothering you then you can simply render the modal into the page and toggle it with JS. There are no rules here, it’s just preference.

xir

xir

Thank you, @garrison !

RodolfoSilva

RodolfoSilva

This is my implementation:

  @doc """
  Renders a modal.

  ## Examples

      <.modal id="confirm-modal">
        This is a modal.
      </.modal>

  JS commands may be passed to the `:on_cancel` to configure
  the closing/cancel event, for example:

      <.modal id="confirm" on_cancel={JS.navigate(~p"/posts")}>
        This is another modal.
      </.modal>

  """
  attr :id, :string, required: true
  attr :on_cancel, JS, default: nil
  slot :inner_block, required: true

  def modal(assigns) do
    ~H"""
    <.portal id={@id} target="body">
      <dialog
        open
        closedby={if @on_cancel, do: "any", else: "none"}
        phx-mounted={
          JS.ignore_attributes("open")
          |> JS.transition({"ease-in duration-200", "opacity-0", "opacity-100"}, time: 0)
        }
        phx-remove={
          JS.remove_attribute("open")
          |> JS.transition({"ease-out duration-200", "opacity-100", "opacity-0"}, time: 0)
        }
        class="modal"
      >
        <.focus_wrap
          class="modal-box w-11/12 max-w-2xl"
          id={"#{@id}-container"}
          tabindex="0"
          phx-key="escape"
          phx-window-keydown={@on_cancel}
        >
          <form :if={@on_cancel} method="dialog">
            <button
              phx-click={@on_cancel}
              class="btn btn-sm btn-circle btn-ghost absolute top-2 right-2"
            >
              ✕
            </button>
          </form>
          {render_slot(@inner_block)}
        </.focus_wrap>
        <form :if={@on_cancel} method="dialog" class="modal-backdrop">
          <button phx-click={@on_cancel}>close</button>
        </form>
      </dialog>
    </.portal>
    """
  end

Requires the LiveView > v1.1.5.

Where Next? Top

Trending in Discussions Top

cblavier
Hey there, It’s been more than a year since we started using LiveView as our main UI library and building a whole library of UI componen...
New
wmnnd
Hey folks, I wanted to share with you a project I’ve been working on for a while: It’s an Open Source (AGPLv3) newsletter tool. Think Ma...
New
axelson
Hi there! :wave: @frigidcode and I (but mostly him) have been running an Elixir Book club, we’re almost done with Designing Elixir Syste...
New
alexslade
Fly’s CEO posted this recently - Turn And Face The Strange · The Fly Blog It says that Fly is going all-in on sprites, which is a worry ...
New
sntran
In my third year of college, around 2010, I found Erlang. A few years later, I found Elixir. I liked the ideas at once: many small proces...
New
Ganna
Hi everyone, I have 8 years of production experience with Erlang/OTP and I’m just starting to get to know Elixir. What would you learn f...
New
gtcode
I’ve been tinkering with Lynx (by José) via hold_ledger. I used AI to write the code, a separate specification, the tests, and the Lean p...
New

Other Trending Topics Top

dominicletz
Hey there, I haven’t seen a list of useful Elixir specific skills for the popular AI tools yet so I’m starting one here with my first. ...
New
mudasobwa
I fully migrated to my own harness from Anthropic/Gemini and I think it’s time to share it. Welcome DSH, the DeepSeek Harness, fully writ...
New
KristerV
Hey. Is there anyone here who creates agents in their apps? Not talking about using agents, but creating them. I’m finding it pretty diff...
New
OndrejValenta
(I just needed to vent somewhere and LinkedIn is full of hope, or hype, I’m not sure which exactly) AI dream has many faces, but general...
New
webofbits
With AI doing more of the implementation work, I’ve been wondering how much coding I should deliberately keep doing myself. My main conc...
#ai
New
kip
In 2021 I started a new library called Tempo with the objective of modelling time as a set of intervals - not as instants. In 2022 I gave...
New

We're in Beta

About us Mission Statement

Options

Thread Display Mode




Thread Preview

Skip Thread Previews