jodit-pro 4.13.30 → 4.13.32

package/es5/jodit.js

@@ -1,7 +1,7 @@
 /*!
  * jodit-pro - PRO Version of Jodit Editor
  * Author: Chupurnov Valerii <chupurnov@gmail.com>
- * Version: v4.13.30
+ * Version: v4.13.32
  * Url: https://xdsoft.net/jodit/pro/
  * License(s): SEE LICENSE IN LICENSE.md
  */
@@ -2299,7 +2299,7 @@ function _unsupported_iterable_to_array(o, minLen) {
     if (n === "Map" || n === "Set") return Array.from(n);
     if (n === "Arguments" || /^(?:Ui|I)nt(?:8|16|32)(?:Clamped)?Array$/.test(n)) return _array_like_to_array(o, minLen);
 }
-var APP_VERSION = "4.13.30";
+var APP_VERSION = "4.13.32";
 // prettier-ignore
 var ES = "es5";
 var IS_ES_MODERN = false;
@@ -11150,7 +11150,12 @@ var map = {};
     while(el && el !== root && el.parentNode){
         el = el.parentNode;
         rect = el.getBoundingClientRect();
-        if (!(top <= rect.bottom)) {
+        // The element's bottom is clipped by the container's bottom edge: a
+        // caret sitting on that line is invisible, so the element is not in
+        // view even though its top still fits. The `top > rect.top` guard keeps
+        // elements taller than the container (which can never fully fit) from
+        // being treated as always out of view. See #1300
+        if (top + height > rect.bottom && top > rect.top) {
             return false;
         }
         // Check if the element is out of view due to a container scrolling
@@ -11158,19 +11163,24 @@ var map = {};
             return false;
         }
     }
-    // Check it's within the document viewport: the element must not be below
-    // the viewport bottom, and (its bottom) must not be above the viewport top —
-    // the latter guard was missing, so an element scrolled above the top was
-    // wrongly reported as visible and never scrolled to. See #1279
+    // Check it's within the document viewport: the element must fit above the
+    // viewport bottom (same clipped-bottom rule as above), and its bottom must
+    // not be above the viewport top — the latter guard was missing, so an
+    // element scrolled above the top was wrongly reported as visible and never
+    // scrolled to. See #1279
     var clientHeight = doc.documentElement && doc.documentElement.clientHeight || 0;
-    return top <= clientHeight && top + height >= 0;
+    return (top + height <= clientHeight || top <= 0) && top + height >= 0;
 }
 /**
  * Scroll element into view if it is not in view
  */ function scrollIntoViewIfNeeded(elm, root, doc) {
     if (_dom_dom_js__WEBPACK_IMPORTED_MODULE_0__.Dom.isHTMLElement(elm) && !inView(elm, root, doc)) {
         if (root.clientHeight !== root.scrollHeight) {
-            root.scrollTop = elm.offsetTop;
+            // Scroll the minimal distance: when the element is below the
+            // visible area align it to the bottom (natural typing flow),
+            // otherwise align it to the top. See #1300
+            var alignBottom = elm.offsetTop + elm.offsetHeight - root.clientHeight;
+            root.scrollTop = root.scrollTop < alignBottom ? alignBottom : elm.offsetTop;
         }
         if (!inView(elm, root, doc)) {
             elm.scrollIntoView();
@@ -42574,7 +42584,7 @@ function readImagesWithReader(fileList, imagesExtensions, promises, uploader, ha
                 return "continue";
             }
             var reader = new FileReader();
-            promises.push(uploader.j.async.promise(function(resolve, reject) {
+            var readerPromise = uploader.j.async.promise(function(resolve, reject) {
                 reader.onerror = reject;
                 reader.onloadend = function() {
                     var resp = {
@@ -42591,12 +42601,24 @@ function readImagesWithReader(fileList, imagesExtensions, promises, uploader, ha
                     resolve(resp);
                 };
                 reader.readAsDataURL(file);
-            }));
+            });
+            readerPromises.push(readerPromise);
+            promises.push(readerPromise);
             fileList[i] = null;
         }
     };
+    var readerPromises = [];
     var file, i;
     for(i = 0; i < fileList.length; i += 1)_loop();
+    // Images inserted as base64 never reach the server branch in `sendFiles`,
+    // which is the only other place that fires `filesWereUploaded`
+    if (readerPromises.length) {
+        Promise.all(readerPromises).then(function() {
+            uploader.j.events && uploader.j.e.fire('filesWereUploaded');
+        }).catch(function() {
+        // Read errors are propagated to the caller via `promises`
+        });
+    }
 }
 
 
@@ -51010,8 +51032,17 @@ function brBefore(start) {
         }
         return;
     }
-    var newP = sel.splitSelection(block, fake);
-    (0,_core_helpers_utils_scroll_into_view_js__WEBPACK_IMPORTED_MODULE_1__.scrollIntoViewIfNeeded)(newP, jodit.editor, jodit.ed);
+    // Inserting the left half above the caret can trigger the browser's
+    // scroll anchoring and shift the visible area — keep it stable. See #1300
+    var scrollTop = jodit.editor.scrollTop;
+    sel.splitSelection(block, fake);
+    if (jodit.editor.scrollTop !== scrollTop) {
+        jodit.editor.scrollTop = scrollTop;
+    }
+    // After the split the caret stays inside `block` (the right half) —
+    // `splitSelection` returns the left half, which is above the caret and
+    // usually already visible. Follow the caret, not the left half. See #1300
+    (0,_core_helpers_utils_scroll_into_view_js__WEBPACK_IMPORTED_MODULE_1__.scrollIntoViewIfNeeded)(block, jodit.editor, jodit.ed);
 }
 
 

package/es5/jodit.min.css

@@ -1,14 +1,14 @@
 /*!
  * jodit-pro - PRO Version of Jodit Editor
  * Author: Chupurnov Valerii <chupurnov@gmail.com>
- * Version: v4.13.30
+ * Version: v4.13.32
  * Url: https://xdsoft.net/jodit/pro/
  * License(s): SEE LICENSE IN LICENSE.md
  */
 	/*!
  * jodit-pro - PRO Version of Jodit Editor
  * Author: Chupurnov Valerii <chupurnov@gmail.com>
- * Version: v4.13.30
+ * Version: v4.13.32
  * Url: https://xdsoft.net/jodit/pro/
  * License(s): SEE LICENSE IN LICENSE.md
  */