@innovastudio/contentbuilder 1.5.213 → 1.5.214

package/package.json

@@ -1,7 +1,7 @@
 {
   "type": "module",
   "name": "@innovastudio/contentbuilder",
-  "version": "1.5.213",
+  "version": "1.5.214",
   "description": "",
   "main": "public/contentbuilder/contentbuilder.esm.js",
   "types": "index.d.ts",

package/public/contentbuilder/contentbuilder.esm.js

@@ -8888,11 +8888,13 @@ class Dom {
   // source: https://stackoverflow.com/questions/2871081/jquery-setting-cursor-position-in-contenteditable-div
   moveCursorToElement(element) {
     var sel, range;
-    if (window.getSelection && this.builder.doc.createRange) {
+    if (this.builder.win.getSelection && this.builder.doc.createRange) {
       range = this.builder.doc.createRange();
       range.selectNodeContents(element);
       range.collapse(false);
-      sel = window.getSelection();
+      // The CONTENT window's selection — with the editor in an iframe the
+      // global one belongs to the host page, so the caret never moved.
+      sel = this.builder.win.getSelection();
       sel.removeAllRanges();
       sel.addRange(range);
     } else if (this.builder.doc.body.createTextRange) {
@@ -84958,20 +84960,6 @@ class ContentBuilder {
     // force (and the option itself) disappears at the final step.
     this.opts.engine = 'inscribe';
     this.engine = 'inscribe';
-    if (!this.opts.uploadFile && !this.opts.upload) {
-      this.opts.uploadFile = async e => {
-        const selectedFile = e.target.files[0];
-        const reader = new FileReader();
-        const result = await new Promise((resolve, reject) => {
-          reader.onload = () => resolve(reader.result);
-          reader.onerror = reject;
-          reader.readAsDataURL(selectedFile);
-        });
-        return {
-          url: result
-        };
-      };
-    }
     if (this.opts.uploadFile) {
       const uploadHandler = async e => {
         const data = await this.opts.uploadFile(e);
@@ -85172,6 +85160,26 @@ class ContentBuilder {
       this.opts.onLargerImageUpload = this.opts.onImageUpload;
     }
 
+    // Last-resort uploader for a builder configured without one (the demo
+    // and no-backend setups): embed the picked file as a data URL. Filled
+    // in per upload type, and only where nothing else provided a handler,
+    // so it can never overwrite `upload`, `uploadFile`, or a legacy
+    // per-type callback — overwriting those turned every upload into a
+    // silent base64 embed instead of calling the host app's uploader.
+    const embedAsDataUrl = async e => {
+      const selectedFile = e.target.files[0];
+      const reader = new FileReader();
+      const result = await new Promise((resolve, reject) => {
+        reader.onload = () => resolve(reader.result);
+        reader.onerror = reject;
+        reader.readAsDataURL(selectedFile);
+      });
+      this.returnUrl(result);
+    };
+    ['onImageUpload', 'onLargerImageUpload', 'onVideoUpload', 'onAudioUpload', 'onMediaUpload', 'onFileUpload'].forEach(name => {
+      if (!this.opts[name]) this.opts[name] = embedAsDataUrl;
+    });
+
     // useButtonPlugin
     if (this.opts.emailMode) {
       this.useButtonPlugin = true;
@@ -90674,7 +90682,11 @@ Please obtain a license at: https://innovastudio.com/contentbox`);
   handleCellKeydown(col, e) {
     if (e.which === 46 || e.which === 8) {
       //delete or backspace
-      if (this.activeIcon) {
+      // Only when the selected icon is still live AND belongs to THIS
+      // column: a stale activeIcon (already deleted, or left over from
+      // another column) would otherwise swallow the key — cancelling the
+      // delete without removing anything, or throwing on a detached node.
+      if (this.activeIcon && this.activeIcon.isConnected && col.contains(this.activeIcon)) {
         // Delete icon
         if (this.activeIcon.parentNode.tagName.toLowerCase() === 'a' && this.activeIcon.parentNode.innerText.trim() === '') {
           let link = this.activeIcon.parentNode;
@@ -90762,46 +90774,56 @@ Please obtain a license at: https://innovastudio.com/contentbox`);
         }
       }
     }
-    if (e.keyCode === 46) {
-      // console.log("delete");
-      let curr;
+
+    // LEGACY empty-block guards for Delete/Backspace (module/custom-code
+    // columns only). Inscribe owns block joining in its regions
+    // (_joinBack/_joinFwd + ensureBlocks), so running these there is both
+    // unnecessary and harmful: the Delete branch mutates the DOM behind
+    // Inscribe's back (bypassing its history) and the Backspace branch
+    // cancels the key outright.
+    //
+    // Both read the caret from `this.win` — the CONTENT window — not the
+    // global `window`, and only act when the caret really is inside this
+    // column. Reading the global selection was the cause of "Backspace
+    // stopped working": with the editor in an iframe (or after a click
+    // landed a caret in the editor UI), the global selection points at
+    // some unrelated node whose textContent is '' and which has no
+    // previousElementSibling, so EVERY Backspace was preventDefault()ed —
+    // typing kept working, and only a page reload cleared it.
+    const legacyDeleteGuard = !this.inscribeRegionAt(col, e.target);
+    if (legacyDeleteGuard && (e.keyCode === 46 || e.keyCode === 8)) {
+      let curr = null;
       try {
-        if (window.getSelection) {
-          curr = window.getSelection().getRangeAt(0).commonAncestorContainer;
-        } else if (document.selection) {
-          curr = document.selection.createRange();
-        }
-        if (curr.innerHTML === '<br>') {
-          let next = curr.nextElementSibling;
-          if (next) {
-            curr.parentNode.removeChild(curr); //without this, empty P, H1, H2 (contains only <br />) cannot be deleted (when there is another P below)
-            // Of course, we can backspace from the P below, but the formatting will change.
-            e.preventDefault();
+        const sel = this.win.getSelection ? this.win.getSelection() : null;
+        if (sel && sel.rangeCount) curr = sel.getRangeAt(0).commonAncestorContainer;
+      } catch (err) {
+        curr = null;
+      }
+      if (curr && col.contains(curr)) {
+        if (e.keyCode === 46) {
+          if (curr.innerHTML === '<br>') {
+            let next = curr.nextElementSibling;
+            if (next) {
+              curr.parentNode.removeChild(curr); //without this, empty P, H1, H2 (contains only <br />) cannot be deleted (when there is another P below)
+              // Of course, we can backspace from the P below, but the formatting will change.
+              e.preventDefault();
+            }
           }
         }
-      } catch (e) {
-        // Do Nothing
-      }
-    }
-    if (e.keyCode === 8) {
-      // delete on Mac
-      let curr;
-      try {
-        if (window.getSelection) {
-          curr = window.getSelection().getRangeAt(0).commonAncestorContainer;
-        } else if (document.selection) {
-          curr = document.selection.createRange();
-        }
-        if (curr.textContent === '') {
-          let prev = curr.previousElementSibling;
-          if (!prev) {
-            e.preventDefault(); //Without this, empty P, H1, H2 (that doesn't have prev element) will be lost => can make empty column.
+        if (e.keyCode === 8) {
+          // delete on Mac
+          // Element nodes only: a text node has no previousElementSibling,
+          // so an empty text node under the caret would always cancel the key.
+          if (curr.nodeType === 1 && curr.textContent === '') {
+            let prev = curr.previousElementSibling;
+            if (!prev) {
+              e.preventDefault(); //Without this, empty P, H1, H2 (that doesn't have prev element) will be lost => can make empty column.
+            }
           }
         }
-      } catch (e) {
-        // Do Nothing
       }
     }
+
     if (e.which === 9 && !e.shiftKey) {
       // tab key pressed
       const activeCol = this.activeCol;
@@ -90812,7 +90834,7 @@ Please obtain a license at: https://innovastudio.com/contentbox`);
           if (target) target.click();
           // if(target.firstElementChild) target.firstElementChild.click();
 
-          let selection = window.getSelection();
+          let selection = this.win.getSelection();
           let range = this.createRange(target, {
             count: 0
           });
@@ -90836,7 +90858,7 @@ Please obtain a license at: https://innovastudio.com/contentbox`);
               if (firstCol) firstCol.click();
               // if(firstCol.firstElementChild) firstCol.firstElementChild.click();
 
-              let selection = window.getSelection();
+              let selection = this.win.getSelection();
               let range = this.createRange(firstCol, {
                 count: 0
               });
@@ -90865,7 +90887,7 @@ Please obtain a license at: https://innovastudio.com/contentbox`);
           const target = activeCol.previousElementSibling;
           if (target) target.click();
           if (target.firstElementChild) target.firstElementChild.click();
-          let selection = window.getSelection();
+          let selection = this.win.getSelection();
           let range = this.createRange(target, {
             count: 0
           });
@@ -90888,7 +90910,7 @@ Please obtain a license at: https://innovastudio.com/contentbox`);
               const target = divs[divs.length - 1];
               if (target) target.click();
               if (target.firstElementChild) target.firstElementChild.click();
-              let selection = window.getSelection();
+              let selection = this.win.getSelection();
               let range = this.createRange(target, {
                 count: 0
               });
@@ -90957,7 +90979,7 @@ Please obtain a license at: https://innovastudio.com/contentbox`);
   }
   createRange(node, chars, range) {
     if (!range) {
-      range = document.createRange();
+      range = this.doc.createRange();
       range.selectNode(node);
       range.setStart(node, 0);
     }