Add dialog: true option to form block

app/helpers/trestle/url_helper.rb

@@ -14,7 +14,9 @@ def admin_link_to(content, instance=nil, options={}, &block)
     end
 
     def admin_url_for(instance, options={})
-      admin = options.key?(:admin) ? Trestle.lookup(options[:admin]) : admin_for(instance)
+      admin = Trestle.lookup(options[:admin]) if options.key?(:admin)
+      admin ||= admin_for(instance)
+
       admin.path(options[:action] || :show, id: admin.to_param(instance)) if admin
     end
 

app/views/trestle/resource/index.html.erb

@@ -1,7 +1,7 @@
 <% content_for(:title, t("admin.titles.index", default: "Listing %{pluralized_model_name}", model_name: admin.model_name.titleize, pluralized_model_name: admin.model_name.plural.titleize)) %>
 
 <% content_for(:primary_toolbar) do %>
-  <%= link_to admin.path(:new), class: "btn btn-default btn-lg" do %>
+  <%= link_to admin.path(:new), class: "btn btn-default btn-lg", data: ({ behavior: "dialog" } if admin.form.dialog?) do %>
     <%= icon("fa fa-plus") %>
     <span class="sr-only"><%= t("admin.buttons.new", default: "New %{model_name}", model_name: admin.model_name) %></span>
   <% end unless admin.readonly? %>

lib/trestle/admin/builder.rb

@@ -42,8 +42,8 @@ def table(options={}, &block)
         admin.table = Table::Builder.build(options.reverse_merge(admin: admin, sortable: true), &block)
       end
 
-      def form(&block)
-        admin.form = Form.new(&block)
+      def form(options={}, &block)
+        admin.form = Form.new(options, &block)
       end
 
       def admin(&block)

lib/trestle/form.rb

@@ -8,10 +8,14 @@ class Form
     autoload :Fields
     autoload :Renderer
 
-    attr_reader :block
+    attr_reader :options, :block
 
-    def initialize(&block)
-      @block = block
+    def initialize(options={}, &block)
+      @options, @block = options, block
+    end
+
+    def dialog?
+      options[:dialog] == true
     end
 
     def render(template, instance)

lib/trestle/form/automatic.rb

@@ -2,7 +2,7 @@ module Trestle
   class Form
     class Automatic < Form
       def initialize(admin)
-        @block = Proc.new do |instance|
+        super() do |instance|
           admin.default_form_attributes.each do |attribute|
             if attribute.array?
               if [:string, :text].include?(attribute.type)

lib/trestle/table/row.rb

@@ -21,9 +21,15 @@ def initialize(row, template)
 
         def options(instance)
           options = Trestle::Options.new
-          options.merge!(data: { url: admin_url_for(instance) }) if table.admin
+
+          if table.admin
+            options.merge!(data: { url: admin_url_for(instance) })
+            options.merge!(data: { behavior: "dialog" }) if table.admin.form.dialog?
+          end
+
           options.merge!(@row.options)
           options.merge!(@template.instance_exec(instance, &@row.block)) if @row.block
+
           options
         end
 

spec/trestle/admin/builder_spec.rb

@@ -118,10 +118,11 @@ module TestHelper; end
       b = Proc.new {}
 
       Trestle::Admin::Builder.build(:test) do
-        form &b
+        form custom: "option", &b
       end
 
       expect(::TestAdmin.form).to be_a(Trestle::Form)
+      expect(::TestAdmin.form.options).to eq(custom: "option")
       expect(::TestAdmin.form.block).to eq(b)
     end
   end

spec/trestle/form_spec.rb

@@ -0,0 +1,15 @@
+require 'spec_helper'
+
+describe Trestle::Form do
+  subject(:form) { Trestle::Form.new }
+
+  context "without options[:dialog]" do
+    it { is_expected.to_not be_dialog }
+  end
+
+  context "with options[:dialog]" do
+    subject(:form) { Trestle::Form.new(dialog: true) }
+
+    it { is_expected.to be_dialog }
+  end
+end