phoenix_live_view 1.0.13 → 1.0.14

package/assets/js/phoenix_live_view/view.js

@@ -1464,7 +1464,10 @@ export default class View {
     const phxTarget = newForm.getAttribute(this.binding("target")) || newForm
     const phxEvent = newForm.getAttribute(this.binding(PHX_AUTO_RECOVER)) || newForm.getAttribute(this.binding("change"))
     const inputs = Array.from(oldForm.elements).filter(el => DOM.isFormInput(el) && el.name && !el.hasAttribute(phxChange))
-    if(inputs.length === 0){ return }
+    if(inputs.length === 0){
+      callback()
+      return
+    }
 
     // we must clear tracked uploads before recovery as they no longer have valid refs
     inputs.forEach(input => input.hasAttribute(PHX_UPLOAD_REF) && LiveUploader.clearFiles(input))
@@ -1529,7 +1532,19 @@ export default class View {
       .filter(form => form.id)
       .filter(form => form.elements.length > 0)
       .filter(form => form.getAttribute(this.binding(PHX_AUTO_RECOVER)) !== "ignore")
-      .map(form => form.cloneNode(true))
+      .map(form => {
+        // we perform a shallow clone and manually copy all elementsAdd commentMore actions
+        const clonedForm = form.cloneNode(false)
+        // we need to copy the private data as it contains
+        // the information about touched fields
+        DOM.copyPrivates(clonedForm, form)
+        Array.from(form.elements).forEach((el) => {
+          const clonedEl = el.cloneNode(false)
+          DOM.copyPrivates(clonedEl, el)
+          clonedForm.appendChild(clonedEl)
+        })
+        return clonedForm
+      })
       .reduce((acc, form) => {
         acc[form.id] = form
         return acc

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "phoenix_live_view",
-  "version": "1.0.13",
+  "version": "1.0.14",
   "description": "The Phoenix LiveView JavaScript client.",
   "license": "MIT",
   "module": "./priv/static/phoenix_live_view.esm.js",

package/priv/static/phoenix_live_view.cjs.js

@@ -4691,6 +4691,7 @@ var View = class _View {
     const phxEvent = newForm.getAttribute(this.binding(PHX_AUTO_RECOVER)) || newForm.getAttribute(this.binding("change"));
     const inputs = Array.from(oldForm.elements).filter((el) => dom_default.isFormInput(el) && el.name && !el.hasAttribute(phxChange));
     if (inputs.length === 0) {
+      callback();
       return;
     }
     inputs.forEach((input2) => input2.hasAttribute(PHX_UPLOAD_REF) && LiveUploader.clearFiles(input2));
@@ -4742,7 +4743,16 @@ var View = class _View {
       return {};
     }
     let phxChange = this.binding("change");
-    return dom_default.all(this.el, `form[${phxChange}]`).filter((form) => form.id).filter((form) => form.elements.length > 0).filter((form) => form.getAttribute(this.binding(PHX_AUTO_RECOVER)) !== "ignore").map((form) => form.cloneNode(true)).reduce((acc, form) => {
+    return dom_default.all(this.el, `form[${phxChange}]`).filter((form) => form.id).filter((form) => form.elements.length > 0).filter((form) => form.getAttribute(this.binding(PHX_AUTO_RECOVER)) !== "ignore").map((form) => {
+      const clonedForm = form.cloneNode(false);
+      dom_default.copyPrivates(clonedForm, form);
+      Array.from(form.elements).forEach((el) => {
+        const clonedEl = el.cloneNode(false);
+        dom_default.copyPrivates(clonedEl, el);
+        clonedForm.appendChild(clonedEl);
+      });
+      return clonedForm;
+    }).reduce((acc, form) => {
       acc[form.id] = form;
       return acc;
     }, {});
@@ -4854,7 +4864,7 @@ var LiveSocket = class {
   }
   // public
   version() {
-    return "1.0.13";
+    return "1.0.14";
   }
   isProfileEnabled() {
     return this.sessionStorage.getItem(PHX_LV_PROFILE) === "true";