avo 4.0.15 → 4.0.16

data/app/assets/stylesheets/css/components/ui/card.css

@@ -35,6 +35,16 @@
   @apply w-full;
 }
 
+/* Padded variant. `.card__body` ships unpadded so wide content (tables,
+   scrollers) can sit flush to the card edge. Add `card--padded` to a card whose
+   body holds free-form content (custom tools, forms, prose) that should get the
+   standard card padding. Usage: `panel.with_card(class: "card--padded")`. */
+.card--padded {
+  .card__body {
+    @apply p-4;
+  }
+}
+
 /* Card wrapper */
 .card__wrapper {
   @apply flex flex-col items-start self-stretch w-full;

data/app/components/avo/u_i/card_component.html.erb

@@ -1,4 +1,4 @@
-<%= tag.div class: class_names("card", @class), data: {**@data, item_index: @index} do %>
+<%= tag.div class: class_names("card", @class, "card--padded" => @padded), data: {**@data, item_index: @index} do %>
   <%= tag.div class: class_names("card__wrapper", @wrapper_class) do %>
     <% if header? %>
       <div class="card__header"><%= header %></div>

data/app/components/avo/u_i/card_component.rb

@@ -5,7 +5,9 @@ class Avo::UI::CardComponent < Avo::BaseComponent
   prop :description
   prop :class
   prop :wrapper_class
-  prop :with_padding, default: -> { true }
+  # Opt-in body padding via the `.card--padded` modifier. Defaults to false so
+  # cards holding wide content (index tables, scrollers) stay flush to the edge.
+  prop :padded, default: -> { false }
   prop :variant, default: -> { :default }
   prop :options, kind: :**
   prop :data, default: -> { {}.freeze }
@@ -25,41 +27,4 @@ class Avo::UI::CardComponent < Avo::BaseComponent
       end
     end
   end
-
-  private
-
-  # def panel_classes
-  #   base_classes = "panel"
-  #   variant_classes = case variant
-  #                    when :compact
-  #                      "panel--compact"
-  #                    when :full_width
-  #                      "panel--full-width"
-  #                    else
-  #                      "panel--default"
-  #                    end
-  #   padding_classes = with_padding ? "panel--with-padding" : ""
-
-  #   [base_classes, variant_classes, padding_classes].compact.join(" ")
-  # end
-
-  # def compact?
-  #   variant == :compact
-  # end
-
-  # def full_width?
-  #   variant == :full_width
-  # end
-
-  # def with_padding?
-  #   with_padding
-  # end
-
-  # def header_classes
-  #   "panel__header"
-  # end
-
-  # def body_classes
-  #   "panel__body"
-  # end
 end

data/app/javascript/js/global_hotkeys.js

@@ -164,6 +164,16 @@ const TYPING_SELECTOR = 'input, textarea, select, [contenteditable]'
 // so it never reaches a document-level listener. Must be registered on each element.
 function hotkeyFireHandler(event) {
   const el = event.currentTarget
+
+  // Stale element left behind after a Turbo body replacement (e.g. clicking a
+  // broken resource renders Rails' error page). @github/hotkey keeps its
+  // reference, so without this guard pressing "p" would .click() the detached
+  // sidebar link and navigate. Cancel so hotkeys are inert on non-Avo pages.
+  if (!el.isConnected) {
+    event.preventDefault()
+    return
+  }
+
   const hotkey = el.getAttribute('data-hotkey')
 
   // Apply feedback to ALL elements sharing this hotkey (e.g. desktop + mobile sidebar).

data/lib/avo/resources/base.rb

@@ -726,7 +726,12 @@ module Avo
       end
 
       def get_external_link
-        return unless record&.persisted?
+        return if record.nil?
+        # Skip only on the "create" form views where there's no saved record to
+        # link to yet. Avoid `record.persisted?` here: non-ActiveRecord records
+        # (e.g. API-backed resources) report `persisted? => false` and would
+        # never render an external link.
+        return if view&.new? || view&.create?
 
         Avo::ExecutionContext.new(target: external_link, resource: self, record: record).handle
       end

data/lib/avo/version.rb

@@ -1,3 +1,3 @@
 module Avo
-  VERSION = "4.0.15" unless const_defined?(:VERSION)
+  VERSION = "4.0.16" unless const_defined?(:VERSION)
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: avo
 version: !ruby/object:Gem::Version
-  version: 4.0.15
+  version: 4.0.16
 platform: ruby
 authors:
 - Adrian Marin
@@ -1198,7 +1198,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.10
+rubygems_version: 3.6.9
 specification_version: 4
 summary: Admin panel framework and Content Management System for Ruby on Rails.
 test_files: []