plutonium 0.52.0 → 0.53.0

data/lib/plutonium/ui/avatar.rb

@@ -0,0 +1,182 @@
+# frozen_string_literal: true
+
+require "digest"
+require "cgi"
+
+module Plutonium
+  module UI
+    # Renders a profile/avatar image for a subject.
+    #
+    #   Avatar(user)                     # Navii fallback seeded from the record
+    #   Avatar(user, src: :photo)        # user.photo if present, else Navii fallback
+    #   Avatar(user, src: user.photo)    # pass the attachment/uploader/URL directly
+    #   Avatar("acme-team")              # a String subject is a deterministic seed
+    #   Avatar("https://.../p.png")      # a URL-shaped subject is shown as the image
+    #   Avatar(src: "https://.../p.png") # a bare image, no subject/fallback
+    #
+    # The positional +subject+ is the identity the fallback is derived from: a
+    # record or a String, hashed to an opaque, PII-free seed. As a convenience, a
+    # URL-shaped String subject is treated as +src+ (the image) instead.
+    # +src+ is the image to show and may be:
+    # - a Symbol naming a method on the subject (e.g. +:avatar+ -> +subject.avatar+).
+    #   This is a contract: the subject must respond to it (raises NoMethodError
+    #   otherwise), so only use a Symbol +src+ with a record subject.
+    # - an ActiveStorage attachment, active_shrine/Shrine uploader, or URL String
+    #
+    # Resolution order: the resolved +src+, then a Navii avatar seeded from the
+    # subject, then a generic user icon when there is nothing to show.
+    class Avatar < Plutonium::UI::Component::Base
+      # Pixel dimensions per semantic size, plus the matching Tailwind width/height
+      # utilities (needed because the preflight resets `img { height: auto }`, so
+      # width/height attributes alone don't pin the rendered size).
+      SIZES = {xs: 24, sm: 32, md: 40, lg: 48, xl: 64}.freeze
+      SIZE_CLASSES = {xs: "w-6 h-6", sm: "w-8 h-8", md: "w-10 h-10", lg: "w-12 h-12", xl: "w-16 h-16"}.freeze
+
+      # Resolve an image value to a URL string. Supports:
+      # - ActiveStorage attachments -> helpers.url_for (they aren't routable via #url)
+      # - active_shrine / other ActiveStorage-style wrappers -> value.url
+      # - Bare Shrine::UploadedFile, CarrierWave, etc. (respond to :url) -> value.url
+      # - Plain URL strings ("https://..." or "/uploads/...")
+      #
+      # Exposed as a module method so collaborators (e.g. Grid::Card) can reuse
+      # the resolution without instantiating the component.
+      def self.resolve_image_src(value, helpers = nil)
+        return nil if value.nil?
+
+        # ActiveStorage is the only supported source that must go through Rails
+        # routing rather than its own #url. It has to be matched *before* the
+        # generic attached?/url checks, because ActiveStorage-compatible wrappers
+        # (e.g. active_shrine) respond to BOTH attached? and url, and those should
+        # resolve via their own #url instead.
+        if active_storage_attachment?(value)
+          return value.attached? ? helpers&.url_for(value) : nil
+        end
+
+        if value.respond_to?(:attached?)  # active_shrine & other AS-style wrappers
+          value.attached? ? value.url : nil
+        elsif value.respond_to?(:url)     # bare Shrine::UploadedFile, CarrierWave, ...
+          value.url
+        elsif value.is_a?(String) && value.start_with?("http", "/")
+          value
+        end
+      rescue ArgumentError, URI::InvalidURIError
+        nil
+      end
+
+      def self.active_storage_attachment?(value)
+        defined?(ActiveStorage::Attached) && value.is_a?(ActiveStorage::Attached)
+      end
+      private_class_method :active_storage_attachment?
+
+      def initialize(subject = nil, src: nil, size: :md, alt: nil, **attributes)
+        # A URL-shaped positional subject is really an image, not an identity:
+        # route it to src so Avatar("https://…/p.png") shows the image rather
+        # than hashing the URL into a seed.
+        if src.nil? && subject.is_a?(String) && subject.start_with?("http", "/")
+          src = subject
+          subject = nil
+        end
+
+        @subject = subject
+        @src = src
+        @size = size
+        @alt = alt
+        @attributes = attributes
+      end
+
+      def view_template
+        url = resolved_src || navii_url
+
+        if url
+          img(
+            src: url, alt: alt_text.to_s, width: pixel_size, height: pixel_size, loading: "lazy",
+            **sized_attributes("rounded-full object-cover bg-[var(--pu-surface-alt)] shrink-0")
+          )
+        else
+          div(**sized_attributes("rounded-full bg-[var(--pu-surface-alt)] text-[var(--pu-text-muted)] flex items-center justify-center shrink-0")) do
+            render Phlex::TablerIcons::User.new(class: "w-2/3 h-2/3")
+          end
+        end
+      end
+
+      private
+
+      # Merge the component's base classes, the size class, and the caller's class;
+      # add an inline dimension style for raw-pixel (Integer) sizes.
+      def sized_attributes(base)
+        attrs = @attributes.dup
+        attrs[:class] = tokens(base, size_class, attrs.delete(:class))
+        attrs[:style] = [size_style, attrs[:style]].compact.join("; ") if size_style
+        attrs
+      end
+
+      def pixel_size
+        @size.is_a?(Symbol) ? SIZES.fetch(@size) : @size
+      end
+
+      def size_class
+        @size.is_a?(Symbol) ? SIZE_CLASSES.fetch(@size) : nil
+      end
+
+      def size_style
+        "width: #{@size}px; height: #{@size}px" unless @size.is_a?(Symbol)
+      end
+
+      def resolved_src
+        value = image_src_value
+        return nil if value.nil?
+
+        # Only reach for the Rails helper proxy when we have an attachment-style
+        # source (ActiveStorage needs helpers.url_for; the resolver ignores it
+        # for active_shrine and other #url-bearing sources).
+        resolver_helpers = value.respond_to?(:attached?) ? helpers : nil
+        self.class.resolve_image_src(value, resolver_helpers)
+      end
+
+      # A Symbol src names a method on the subject (e.g. :avatar -> subject.avatar);
+      # anything else is the attachment/uploader/URL itself.
+      def image_src_value
+        @src.is_a?(Symbol) ? @subject&.public_send(@src) : @src
+      end
+
+      def navii_url
+        seed = navii_seed
+        return nil unless seed
+
+        host = Plutonium.configuration.navii_host_url
+        "#{host}/avatar/#{CGI.escape(seed)}?size=#{pixel_size}"
+      end
+
+      # The value sent to Navii is ALWAYS a hash of the subject's identity, so no
+      # plaintext (model names, ids, emails, or caller-provided seed strings) ever
+      # reaches the external service. Determinism is preserved: same identity ->
+      # same hash -> same avatar.
+      def navii_seed
+        identity = subject_identity
+        return nil unless identity
+
+        Digest::SHA256.hexdigest(identity)[0, 16]
+      end
+
+      # Stable identity string for the subject: a String subject verbatim, or
+      # "Class:id" for a record. Hashed by #navii_seed before it leaves the app.
+      def subject_identity
+        case @subject
+        when nil then nil
+        when String then @subject
+        else "#{@subject.class.name}:#{@subject.id}" if @subject.respond_to?(:id) && @subject.id.present?
+        end
+      end
+
+      def alt_text
+        return @alt if @alt
+
+        case @subject
+        when nil then nil
+        when String then @subject
+        else helpers&.display_name_of(@subject)
+        end
+      end
+    end
+  end
+end

data/lib/plutonium/ui/component/kit.rb

@@ -50,6 +50,8 @@ module Plutonium
           self.class.method_defined?(build_method) || super
         end
 
+        def BuildAvatar(...) = Plutonium::UI::Avatar.new(...)
+
         def BuildBreadcrumbs(...) = Plutonium::UI::Breadcrumbs.new(...)
 
         def BuildSkeletonTable(...) = Plutonium::UI::SkeletonTable.new(...)

data/lib/plutonium/ui/form/base.rb

@@ -149,8 +149,22 @@ module Plutonium
         def initialize_attributes
           super
 
-          attributes[:id] ||= "resource-form"
-          attributes["data-controller"] = "form"
+          # Only fall back to :resource_form when the caller didn't already
+          # name the form. Phlexi moves an explicit `attributes[:id]` onto
+          # `@dom_id` before this runs, so a blind `||=` here would clobber
+          # things like the filter slideover's `id: "filter-form"` —
+          # producing two `<form id="resource-form">` on the page and
+          # silently breaking the modal pre_submit re-render (Turbo's
+          # `getElementById` finds the filter form first).
+          attributes[:id] ||= "resource-form" if @dom_id.nil?
+          attributes["data-controller"] = form_data_controller
+        end
+
+        # `dirty-form-guard` is attached unconditionally — it self-disables
+        # outside a <dialog>. Branching on `in_modal?` here would fail:
+        # Phlex forbids view-context access before rendering begins.
+        def form_data_controller
+          "form dirty-form-guard"
         end
 
         # Scope the form id to the current turbo frame at render time (we

data/lib/plutonium/ui/form/components/secure_association.rb

@@ -70,12 +70,13 @@ module Plutonium
             end
 
             return unless registered_resources.include?(klass)
-            action = resource_definition(klass).defined_actions[:new]
+            target_definition = resource_definition(klass)
+            action = target_definition.defined_actions[:new]
             return unless action
             return unless @skip_authorization || action.permitted_by?(policy_for(record: klass))
 
             url = route_options_to_url(action.route_options, klass)
-            [with_return_to(url), action.turbo_frame]
+            [with_return_to(url), action.turbo_frame(target_definition)]
           end
 
           def with_return_to(url)

data/lib/plutonium/ui/form/resource.rb

@@ -31,12 +31,70 @@ module Plutonium
           render_actions
         end
 
+        # Mirrors Phlexi::Form::Base#view_template (phlexi-form ~> 0.14)
+        # — keep these in sync if upgrading. We override so the guard
+        # dialog renders inside the <form> tag (where the JS controller
+        # looks for it via `dirty-form-guard-target`) even when a
+        # subclass overrides `form_template`. Without this, the
+        # controller silently falls back to `window.confirm`.
+        def view_template(&block)
+          captured_body = capture { form_template(&block) }
+          captured_guard = capture { render_dirty_form_guard_dialog if in_modal? }
+          form_tag do
+            form_errors
+            raw(safe(captured_body))
+            raw(safe(captured_guard))
+          end
+        end
+
         def form_class
           in_modal? ? "flex-1 flex flex-col min-h-0" : super
         end
 
         private
 
+        # Nested inside the form so showModal() stacks it in the browser's
+        # top layer above the surrounding slideover/centered modal — no
+        # z-index juggling required.
+        def render_dirty_form_guard_dialog
+          dialog(
+            class:
+              "pu-dialog " \
+              "top-1/2 -translate-y-1/2 left-1/2 -translate-x-1/2 " \
+              "w-full max-w-md p-0 " \
+              "open:flex flex-col " \
+              "opacity-0 scale-95 data-[open]:opacity-100 data-[open]:scale-100 " \
+              "transition-[opacity,transform] duration-200 ease-out",
+            data: {"dirty-form-guard-target": "confirmDialog"},
+            # Modern Chrome refuses user-agent close requests (Esc, backdrop);
+            # older browsers fall back to the JS controller's interception.
+            closedby: "none",
+            "aria-labelledby": "pu-dirty-guard-title",
+            "aria-describedby": "pu-dirty-guard-desc"
+          ) do
+            div(class: "px-6 pt-5 pb-4 border-b border-[var(--pu-border)]") do
+              h2(id: "pu-dirty-guard-title", class: "text-lg font-semibold text-[var(--pu-text)]") do
+                "Discard changes?"
+              end
+              p(id: "pu-dirty-guard-desc", class: "mt-1 text-sm text-[var(--pu-text-muted)]") do
+                "You have unsaved changes. Closing this form now will lose them."
+              end
+            end
+            div(class: "flex items-center justify-end gap-2 px-6 py-4") do
+              button(
+                type: "button",
+                class: "pu-btn pu-btn-md pu-btn-outline",
+                data: {action: "dirty-form-guard#keepEditing"}
+              ) { "Keep editing" }
+              button(
+                type: "button",
+                class: "pu-btn pu-btn-md pu-btn-danger",
+                data: {action: "dirty-form-guard#discard"}
+              ) { "Discard changes" }
+            end
+          end
+        end
+
         def render_fields
           fields_wrapper {
             resource_fields.each { |name|

data/lib/plutonium/ui/form/theme.rb

@@ -30,11 +30,15 @@ module Plutonium
             valid_input: "pu-input pu-input-valid",
             neutral_input: "",
 
-            # Checkbox
+            # Checkbox / Boolean
             checkbox: "pu-checkbox",
+            boolean: "pu-checkbox",
+            valid_boolean: "pu-checkbox",
+            invalid_boolean: "pu-checkbox pu-input-invalid",
 
             # Radio buttons
-            radio_button: "pu-checkbox",
+            radio_button: "pu-radio",
+            collection_radio_buttons: "flex flex-col gap-2",
 
             # Color
             color: "pu-color-input appearance-none bg-transparent border-none cursor-pointer w-12 h-12 rounded-lg",
@@ -46,7 +50,7 @@ module Plutonium
             file: "pu-input py-2 [&::file-selector-button]:mr-4 [&::file-selector-button]:px-4 [&::file-selector-button]:py-2 [&::file-selector-button]:bg-[var(--pu-surface-alt)] [&::file-selector-button]:border-0 [&::file-selector-button]:rounded-md [&::file-selector-button]:text-sm [&::file-selector-button]:font-semibold [&::file-selector-button]:text-[var(--pu-text-muted)] [&::file-selector-button]:hover:bg-[var(--pu-border)] [&::file-selector-button]:cursor-pointer [&::file-selector-button]:transition-colors",
 
             # Hint themes
-            hint: "pu-hint whitespace-pre",
+            hint: "pu-hint whitespace-pre-wrap",
 
             # Error themes
             error: "text-xs text-danger-600 mt-1",

data/lib/plutonium/ui/grid/card.rb

@@ -75,20 +75,21 @@ module Plutonium
 
         def render_image_slot(size:)
           value = field_value(slots[:image])
-          return unless value
-          src = image_src_for(value)
-          return unless src
 
           if size == :cover
+            # Cover is a full-width banner, not an avatar: only render when an
+            # actual image resolves (no deterministic fallback).
+            src = Plutonium::UI::Avatar.resolve_image_src(value, helpers)
+            return unless src
+
             div(class: "w-full aspect-video bg-[var(--pu-surface-alt)] overflow-hidden") do
               img(src: src, alt: header_text.to_s, class: "w-full h-full object-cover")
             end
           else
-            img(
-              src: src,
-              alt: header_text.to_s,
-              class: "w-12 h-12 rounded-full object-cover bg-[var(--pu-surface-alt)] shrink-0"
-            )
+            # Small avatar slot: render the resolved image, or Avatar's generic
+            # icon fallback. No subject is passed, so image-less cards fall back
+            # to the local icon rather than a per-card request to Navii.
+            Avatar(src: value, size: :lg, alt: header_text.to_s)
           end
         end
 
@@ -163,7 +164,7 @@ module Plutonium
           url = route_options_to_url(show.route_options, record)
           a(
             href: url,
-            data: {row_click_target: "show", turbo_frame: show.turbo_frame},
+            data: {row_click_target: "show", turbo_frame: show.turbo_frame(resource_definition)},
             class: "sr-only",
             tabindex: "-1",
             "aria-label": "Open #{header_text}"
@@ -192,23 +193,6 @@ module Plutonium
           record.public_send(name)
         end
 
-        # Resolves a slot value to an image URL. Supports:
-        # - ActiveStorage attachments (`record.avatar` -> Attached::One/Many)
-        # - Shrine uploaders (`record.avatar` -> UploadedFile, responds to :url)
-        # - Plain URL strings ("https://..." or "/uploads/...")
-        def image_src_for(value)
-          return nil if value.nil?
-          if value.respond_to?(:attached?)
-            value.attached? ? helpers.url_for(value) : nil
-          elsif value.respond_to?(:url)
-            value.url
-          elsif value.is_a?(String) && value.start_with?("http", "/")
-            value
-          end
-        rescue ArgumentError, URI::InvalidURIError
-          nil
-        end
-
         def row_actions
           @row_actions ||= resource_definition.defined_actions.values.select { |a|
             a.collection_record_action? && a.permitted_by?(record_policy)

data/lib/plutonium/ui/modal/base.rb

@@ -9,9 +9,26 @@ module Plutonium
         slot :close
         slot :footer
 
-        def initialize(title: nil, description: nil)
+        # Sizes that all modal subclasses must implement entries for in
+        # their SIZE_CLASSES table. `:auto` is content-driven (`w-fit`
+        # with a viewport cap and a sensible floor) and is the only way
+        # to avoid clipping forms whose natural width exceeds the
+        # default. Sizes intentionally mirror Tailwind's max-w-* scale
+        # so a definition that says `size: :xl` reads predictably.
+        VALID_SIZES = [:sm, :md, :lg, :xl, :auto, :full].freeze
+
+        # Resolves the concrete modal class for a definition's `modal_mode`
+        # symbol. Unknown / `false` modes fall back to `Slideover` so call
+        # sites can stay branchless.
+        def self.class_for_mode(mode)
+          (mode == :centered) ? Plutonium::UI::Modal::Centered : Plutonium::UI::Modal::Slideover
+        end
+
+        def initialize(title: nil, description: nil, size: :md)
           @title = title
           @description = description
+          @size = size
+          validate_size!
         end
 
         def view_template(&block)
@@ -54,9 +71,27 @@ module Plutonium
         end
 
         def dialog_classes
+          "#{base_dialog_classes} #{size_classes}"
+        end
+
+        # Positioning, backdrop, transitions — everything that does
+        # not vary with `size`. Width/height tokens live in
+        # `size_classes` so size keys can fully replace them
+        # (notably `:auto`, which needs `w-fit` instead of `w-full`).
+        def base_dialog_classes
           raise NotImplementedError
         end
 
+        def size_classes
+          self.class::SIZE_CLASSES.fetch(@size)
+        end
+
+        def validate_size!
+          return if VALID_SIZES.include?(@size)
+          raise ArgumentError,
+            "modal size must be one of #{VALID_SIZES.inspect}, got #{@size.inspect}"
+        end
+
         def inner_classes
           "flex flex-col h-full max-h-[inherit] min-h-0"
         end

data/lib/plutonium/ui/modal/centered.rb

@@ -4,16 +4,34 @@ module Plutonium
   module UI
     module Modal
       class Centered < Plutonium::UI::Modal::Base
+        # Width tokens for each VALID_SIZES key. `:md` reproduces the
+        # historical default (`w-full max-w-xl`); `:auto` drops `w-full`
+        # so the dialog hugs its content, with a floor that keeps tiny
+        # confirm dialogs from collapsing and a cap to stay on-screen.
+        SIZE_CLASSES = {
+          sm: "w-full max-w-md",
+          md: "w-full max-w-xl",
+          lg: "w-full max-w-2xl",
+          xl: "w-full max-w-4xl",
+          auto: "w-fit max-w-[90vw] min-w-[400px]",
+          full: "w-full max-w-[95vw]"
+        }.freeze
+
         protected
 
-        def dialog_classes
-          "rounded-[var(--pu-radius-lg)] w-full max-w-xl " \
-            "bg-[var(--pu-surface)] border border-[var(--pu-border)] " \
-            "backdrop:bg-black/60 backdrop:backdrop-blur-sm " \
+        # Surface (bg, border, radius, backdrop) lives in `.pu-dialog` so
+        # the centered modal, dirty-form-guard prompt, and Turbo confirm
+        # can't drift on design tokens. The remaining utilities are
+        # positioning, sizing, and the open/close transform animation —
+        # driven by [data-open] (set on the frame after showModal() by
+        # remote_modal_controller); avoids the @starting-style spec dance.
+        def base_dialog_classes
+          "pu-dialog " \
             "top-1/2 -translate-y-1/2 left-1/2 -translate-x-1/2 " \
             "max-h-[80vh] " \
-            "hidden open:flex flex-col p-0 " \
-            "opacity-0 open:opacity-100 transition-opacity duration-200 ease-in-out"
+            "open:flex flex-col p-0 " \
+            "opacity-0 scale-95 data-[open]:opacity-100 data-[open]:scale-100 " \
+            "transition-[opacity,transform] duration-200 ease-out"
         end
       end
     end

data/lib/plutonium/ui/modal/slideover.rb

@@ -4,21 +4,36 @@ module Plutonium
   module UI
     module Modal
       class Slideover < Plutonium::UI::Modal::Base
+        # Width tokens for each VALID_SIZES key. Mobile always takes
+        # full width (the slideover is pinned to the right edge, so
+        # anything narrower than the viewport looks awkward on phones);
+        # the `sm:` token controls the desktop width. `:md` reproduces
+        # the historical default. `:auto` switches to `sm:w-auto` with
+        # a viewport cap so the panel grows to fit the form.
+        SIZE_CLASSES = {
+          sm: "w-full sm:w-[400px]",
+          md: "w-full sm:w-[480px]",
+          lg: "w-full sm:w-[640px]",
+          xl: "w-full sm:w-[800px]",
+          auto: "w-full sm:w-auto sm:max-w-[90vw] sm:min-w-[480px]",
+          full: "w-full sm:w-[95vw]"
+        }.freeze
+
         protected
 
-        def dialog_classes
-          "fixed top-0 right-0 bottom-0 left-auto m-0 h-screen w-full sm:w-[480px] max-w-full max-h-screen " \
+        # Animation is driven by `data-open`, toggled by the remote-modal
+        # controller on the frame after showModal(). Mirrors the filter
+        # slideover's pattern — see Centered for the same rationale.
+        def base_dialog_classes
+          "fixed top-0 right-0 bottom-0 left-auto m-0 h-screen max-w-full max-h-screen " \
             "bg-[var(--pu-surface)] border-l border-[var(--pu-border)] " \
-            "backdrop:bg-black/60 backdrop:backdrop-blur-sm " \
+            "backdrop:bg-transparent data-[open]:backdrop:bg-black/60 " \
+            "data-[open]:backdrop:backdrop-blur-sm " \
+            "backdrop:transition-[background-color] backdrop:duration-300 backdrop:ease-out " \
             "rounded-none p-0 " \
-            "hidden open:flex flex-col " \
-            "translate-x-full open:translate-x-0 " \
-            "transition-[transform,display,overlay] duration-300 ease-out " \
-            "[transition-behavior:allow-discrete] " \
-            "starting:open:translate-x-full " \
-            "backdrop:transition-[display,overlay,background-color] backdrop:duration-300 " \
-            "backdrop:[transition-behavior:allow-discrete] " \
-            "starting:open:backdrop:bg-transparent"
+            "open:flex flex-col " \
+            "translate-x-full data-[open]:translate-x-0 " \
+            "transition-transform duration-300 ease-out"
         end
       end
     end

data/lib/plutonium/ui/nav_user.rb

@@ -52,10 +52,11 @@ module Plutonium
 
       slot :section, Section, collection: true
 
-      def initialize(email:, name: nil, avatar_url: nil)
+      def initialize(email:, name: nil, avatar_url: nil, record: nil)
         @email = email
         @name = name
         @avatar_url = avatar_url
+        @record = record
       end
 
       def view_template
@@ -68,14 +69,6 @@ module Plutonium
       private
 
       def render_trigger_button
-        if @avatar_url.present?
-          render_avatar_button
-        else
-          render_default_button
-        end
-      end
-
-      def render_avatar_button
         button(
           type: "button",
           class: "flex mx-3 text-sm rounded-full md:mr-0 focus:ring-2 focus:ring-[var(--pu-border)] focus:ring-offset-2 transition-shadow",
@@ -84,20 +77,7 @@ module Plutonium
           data: {resource_drop_down_target: "trigger"}
         ) do
           span(class: "sr-only") { "Open user menu" }
-          img(class: "w-8 h-8 rounded-full ring-2 ring-[var(--pu-border)]", src: @avatar_url, alt: "avatar")
-        end
-      end
-
-      def render_default_button
-        button(
-          type: "button",
-          class: "flex mx-3 p-1 text-sm border border-[var(--pu-border)] text-[var(--pu-text-muted)] hover:text-[var(--pu-text)] hover:bg-[var(--pu-surface-alt)] rounded-full md:mr-0 focus:ring-2 focus:ring-[var(--pu-border)] focus:ring-offset-2 transition-colors",
-          aria: {expanded: "false"},
-          id: "user-nav-dropdown-toggle",
-          data: {resource_drop_down_target: "trigger"}
-        ) do
-          span(class: "sr-only") { "Open user menu" }
-          render Phlex::TablerIcons::User.new(class: "w-6 h-6")
+          Avatar(@record, src: @avatar_url, size: :sm, alt: "avatar", class: "ring-2 ring-[var(--pu-border)]")
         end
       end
 

data/lib/plutonium/ui/page/edit.rb

@@ -23,9 +23,12 @@ module Plutonium
         end
 
         def render_modal_form
-          modal_class = (current_definition.modal == :centered) ?
-            Plutonium::UI::Modal::Centered : Plutonium::UI::Modal::Slideover
-          render modal_class.new(title: page_title, description: page_description) do
+          modal_class = Plutonium::UI::Modal::Base.class_for_mode(current_definition.modal_mode)
+          render modal_class.new(
+            title: page_title,
+            description: page_description,
+            size: current_definition.modal_size
+          ) do
             render partial("resource_form")
           end
         end

data/lib/plutonium/ui/page/interactive_action.rb

@@ -18,12 +18,14 @@ module Plutonium
 
         def render_default_content
           if in_modal?
-            modal_class = (current_interactive_action.modal == :slideover) ?
-              Plutonium::UI::Modal::Slideover : Plutonium::UI::Modal::Centered
+            modal_class = Plutonium::UI::Modal::Base.class_for_mode(
+              current_interactive_action.modal_mode(current_definition)
+            )
 
             render modal_class.new(
               title: page_title,
-              description: page_description
+              description: page_description,
+              size: current_interactive_action.modal_size(current_definition)
             ) do
               render partial("interactive_action_form")
             end

data/lib/plutonium/ui/page/new.rb

@@ -23,9 +23,12 @@ module Plutonium
         end
 
         def render_modal_form
-          modal_class = (current_definition.modal == :centered) ?
-            Plutonium::UI::Modal::Centered : Plutonium::UI::Modal::Slideover
-          render modal_class.new(title: page_title, description: page_description) do
+          modal_class = Plutonium::UI::Modal::Base.class_for_mode(current_definition.modal_mode)
+          render modal_class.new(
+            title: page_title,
+            description: page_description,
+            size: current_definition.modal_size
+          ) do
             render partial("resource_form")
           end
         end

data/lib/plutonium/ui/table/components/bulk_actions_toolbar.rb

@@ -68,7 +68,7 @@ module Plutonium
                 bulk_actions_target: "actionButton",
                 bulk_action_name: action.name,
                 bulk_action_url: url,
-                turbo_frame: action.turbo_frame
+                turbo_frame: action.turbo_frame(current_definition)
               }
             ) do
               if action.icon

data/lib/plutonium/version.rb

@@ -1,5 +1,5 @@
 module Plutonium
-  VERSION = "0.52.0"
+  VERSION = "0.53.0"
   NEXT_MAJOR_VERSION = VERSION.split(".").tap { |v|
     v[1] = v[1].to_i + 1
     v[2] = 0