Set default target turbo_frame within admin_link_to depending on moda…

…l_request?

app/helpers/trestle/url_helper.rb

@@ -50,7 +50,7 @@ def admin_link_to(content, instance_or_url=nil, options={}, &block)
           if MODAL_ACTIONS.include?(action) && admin.respond_to?(:form) && admin.form.modal?
             options[:data][:controller] ||= "modal-trigger"
           else
-            options[:data][:turbo_frame] = "_top"
+            options[:data][:turbo_frame] ||= (modal_request? ? "modal" : "_top")
           end
 
           link_to(content, path, options)

lib/trestle/resource/toolbar.rb

@@ -20,7 +20,7 @@ def save(label: t("buttons.save", default: "Save %{model_name}"), **attrs)
         def delete(label: t("buttons.delete", default: "Delete %{model_name}"), **attrs)
           return unless action?(:destroy)
 
-          defaults = Trestle::Options.new(action: :destroy, style: :danger, icon: "fa fa-trash", data: { turbo_method: "delete", controller: "confirm-delete", confirm_delete_placement_value: "bottom" })
+          defaults = Trestle::Options.new(action: :destroy, style: :danger, icon: "fa fa-trash", data: { turbo_method: "delete", turbo_frame: "_top", controller: "confirm-delete", confirm_delete_placement_value: "bottom" })
           link(label, instance, defaults.merge(attrs))
         end
 

spec/support/contexts/template.rb

@@ -7,5 +7,6 @@
     template.extend(Trestle::UrlHelper)
 
     allow(template).to receive(:admin).and_return(admin)
+    allow(template).to receive(:modal_request?).and_return(false)
   end
 end

spec/trestle/helpers/url_helper_spec.rb

@@ -7,6 +7,7 @@
 
   let(:form) { double(modal?: false) }
   let(:admin) { double(form: form) }
+  let(:modal_request?) { false }
 
   before(:each) do
     allow(Trestle).to receive(:lookup).and_return(nil)

spec/trestle/resource/toolbar_spec.rb

@@ -69,15 +69,15 @@
       end
 
       it "returns a delete link" do
-        expect(builder.delete).to eq(Trestle::Toolbar::Link.new(template, "Delete Resource", instance, action: :destroy, style: :danger, icon: "fa fa-trash", data: { turbo_method: "delete", controller: "confirm-delete", confirm_delete_placement_value: "bottom" }))
+        expect(builder.delete).to eq(Trestle::Toolbar::Link.new(template, "Delete Resource", instance, action: :destroy, style: :danger, icon: "fa fa-trash", data: { turbo_method: "delete", turbo_frame: "_top", controller: "confirm-delete", confirm_delete_placement_value: "bottom" }))
       end
 
       it "accepts a custom label" do
-        expect(builder.delete(label: "Delete Me")).to eq(Trestle::Toolbar::Link.new(template, "Delete Me", instance, action: :destroy, style: :danger, icon: "fa fa-trash", data: { turbo_method: "delete", controller: "confirm-delete", confirm_delete_placement_value: "bottom" }))
+        expect(builder.delete(label: "Delete Me")).to eq(Trestle::Toolbar::Link.new(template, "Delete Me", instance, action: :destroy, style: :danger, icon: "fa fa-trash", data: { turbo_method: "delete", turbo_frame: "_top", controller: "confirm-delete", confirm_delete_placement_value: "bottom" }))
       end
 
       it "accepts custom options" do
-        expect(builder.delete(style: :warning, icon: "fas fa-bomb")).to eq(Trestle::Toolbar::Link.new(template, "Delete Resource", instance, action: :destroy, style: :warning, icon: "fas fa-bomb", data: { turbo_method: "delete", controller: "confirm-delete", confirm_delete_placement_value: "bottom" }))
+        expect(builder.delete(style: :warning, icon: "fas fa-bomb")).to eq(Trestle::Toolbar::Link.new(template, "Delete Resource", instance, action: :destroy, style: :warning, icon: "fas fa-bomb", data: { turbo_method: "delete", turbo_frame: "_top", controller: "confirm-delete", confirm_delete_placement_value: "bottom" }))
       end
     end