fix form submission

_config/pure-modal.yml

@@ -4,3 +4,4 @@ Name: pure-modal
 LeKoala\PureModal\PureModal:
   move_modal_to_body: true
   overlay_triggers_close: true
+  compute_iframe_height: false

src/PureModal.php

@@ -12,7 +12,7 @@
  * A simple pure css modal for usage in the cms
  *
  * If your content contains a form, it should be loaded through an iframe
- * because you cannot nest forms
+ * because you cannot nest forms (warning, this gets tricky with "move_modal_to_body")
  *
  * @author lekoala
  */
@@ -106,7 +106,6 @@ public static function renderDialog($controller, $customFields = null)
     public function setContent($content)
     {
         $this->content = $content;
-
         return $this;
     }
 
@@ -160,7 +159,6 @@ public function getTitle()
     public function setTitle($title)
     {
         $this->title = $title;
-
         return $this;
     }
 
@@ -181,7 +179,6 @@ public function getIframe()
     public function setIframe($iframe)
     {
         $this->iframe = $iframe;
-
         return $this;
     }
 

src/PureModalAction.php

@@ -72,11 +72,20 @@ public function getOverlayTriggersClose()
         return PureModal::getOverlayTriggersCloseConfig();
     }
 
+    public static function getMoveModalScript()
+    {
+        if (!PureModal::config()->move_modal_to_body) {
+            return '';
+        }
+        return "document.getElementById('Form_ItemEditForm').appendChild(this.parentElement.querySelector('.pure-modal'));this.onclick=null;";
+    }
+
     public function getAttributes()
     {
         $attrs = [];
-        // Move modal to body to avoid nesting issues
-        $attrs['onclick'] = PureModal::getMoveModalScript();
+        // Move modal to form to avoid nesting issues
+        // We cannot append to body because this breaks form submission
+        $attrs['onclick'] = self::getMoveModalScript();
         return $attrs;
     }
 

templates/LeKoala/PureModal/PureModalAction.ss

@@ -22,7 +22,7 @@
                 </div>
                 <% if $ShowDialogButton %>
                     <div class="toolbar toolbar--south">
-                        <button type="submit" name="action_$Name" class="action custom-action<% if $extraClass %> $extraClass<% end_if %>">
+                        <button type="submit" name="action_$Name" onclick="$SubmitOnClickScript" class="action custom-action<% if $extraClass %> $extraClass<% end_if %>">
                             {$DialogButtonTitle.RAW}
                         </button>
                     </div>