prosemirror-view 1.23.6 → 1.23.9

package/CHANGELOG.md

@@ -1,3 +1,23 @@
+## 1.23.9 (2022-03-11)
+
+### Bug fixes
+
+Make sure screen readers don't read the `<img>` nodes added as kludge for cursor behavior.
+
+## 1.23.8 (2022-03-10)
+
+### Bug fixes
+
+Fix an issue where the editor needlessly interrupted composition with IME systems that keep the cursor at the start of the composition, such as some Pinyin input methods.
+
+## 1.23.7 (2022-02-25)
+
+### Bug fixes
+
+Fix a crash when a view was being destroyed during input reading.
+
+Fix a crash on Firefox when replacing text in some specific types of document structure.
+
 ## 1.23.6 (2022-01-13)
 
 ### Bug fixes

package/dist/index.es.js

@@ -64,7 +64,8 @@ function scanFor(node, off, targetNode, targetOff, dir) {
     if (node == targetNode && off == targetOff) { return true }
     if (off == (dir < 0 ? 0 : nodeSize(node))) {
       var parent = node.parentNode;
-      if (parent.nodeType != 1 || hasBlockDesc(node) || atomElements.test(node.nodeName) || node.contentEditable == "false")
+      if (!parent || parent.nodeType != 1 || hasBlockDesc(node) || atomElements.test(node.nodeName) ||
+          node.contentEditable == "false")
         { return false }
       off = domIndex(node) + (dir < 0 ? 0 : 1);
       node = parent;
@@ -1923,7 +1924,10 @@ ViewTreeUpdater.prototype.addHackNode = function addHackNode (nodeName) {
     this.index++;
   } else {
     var dom = document.createElement(nodeName);
-    if (nodeName == "IMG") { dom.className = "ProseMirror-separator"; }
+    if (nodeName == "IMG") {
+      dom.className = "ProseMirror-separator";
+      dom.alt = "";
+    }
     if (nodeName == "BR") { dom.className = "ProseMirror-trailingBreak"; }
     this.top.children.splice(this.index++, 0, new TrailingHackViewDesc(this.top, nothing, dom, null));
     this.changed = true;
@@ -2081,10 +2085,13 @@ function findTextInFragment(frag, text, from, to) {
       if (!next.isText) { break }
       str += next.text;
     }
-    if (pos >= from && childStart < to) {
-      var found = str.lastIndexOf(text, to - childStart - 1);
+    if (pos >= from) {
+      var found = childStart < to ? str.lastIndexOf(text, to - childStart - 1) : -1;
       if (found >= 0 && found + text.length + childStart >= from)
         { return childStart + found }
+      if (from == to && str.length >= (to + text.length) - childStart &&
+          str.slice(to - childStart, to - childStart + text.length) == text)
+        { return to }
     }
   }
   return -1
@@ -2681,6 +2688,7 @@ function readDOMChange(view, from, to, typeOver, addedNodes) {
 
   var sel = view.state.selection;
   var parse = parseBetween(view, from, to);
+
   // Chrome sometimes leaves the cursor before the inserted text when
   // composing after a cursor wrapper. This moves it forward.
   if (result.chrome && view.cursorWrapper && parse.sel && parse.sel.anchor == view.cursorWrapper.deco.from) {
@@ -2726,9 +2734,11 @@ function readDOMChange(view, from, to, typeOver, addedNodes) {
   if (view.state.selection.from < view.state.selection.to &&
       change.start == change.endB &&
       view.state.selection instanceof TextSelection) {
-    if (change.start > view.state.selection.from && change.start <= view.state.selection.from + 2) {
+    if (change.start > view.state.selection.from && change.start <= view.state.selection.from + 2 &&
+        view.state.selection.from >= parse.from) {
       change.start = view.state.selection.from;
-    } else if (change.endA < view.state.selection.to && change.endA >= view.state.selection.to - 2) {
+    } else if (change.endA < view.state.selection.to && change.endA >= view.state.selection.to - 2 &&
+               view.state.selection.to <= parse.to) {
       change.endB += (view.state.selection.to - change.endA);
       change.endA = view.state.selection.to;
     }
@@ -3358,7 +3368,7 @@ DOMObserver.prototype.flush = function flush () {
       checkCSS(this.view);
     }
     this.handleDOMChange(from, to, typeOver, added);
-    if (this.view.docView.dirty) { this.view.updateState(this.view.state); }
+    if (this.view.docView && this.view.docView.dirty) { this.view.updateState(this.view.state); }
     else if (!this.currentSelection.eq(sel)) { selectionToDOM(this.view); }
     this.currentSelection.set(sel);
   }
@@ -3909,7 +3919,7 @@ function endComposition(view, forceUpdate) {
   if (result.android && view.domObserver.flushingSoon >= 0) { return }
   view.domObserver.forceFlush();
   clearComposition(view);
-  if (forceUpdate || view.docView.dirty) {
+  if (forceUpdate || view.docView && view.docView.dirty) {
     var sel = selectionFromDOM(view);
     if (sel && !sel.eq(view.state.selection)) { view.dispatch(view.state.tr.setSelection(sel)); }
     else { view.updateState(view.state); }
@@ -5277,6 +5287,7 @@ function updateCursorWrapper(view) {
     var dom = document.createElement("img");
     dom.className = "ProseMirror-separator";
     dom.setAttribute("mark-placeholder", "true");
+    dom.setAttribute("alt", "");
     view.cursorWrapper = {dom: dom, deco: Decoration.widget(view.state.selection.head, dom, {raw: true, marks: view.markCursor})};
   } else {
     view.cursorWrapper = null;
@@ -5451,7 +5462,10 @@ function checkStateComponent(plugin) {
 //   The DOM serializer to use when putting content onto the
 //   clipboard. If not given, the result of
 //   [`DOMSerializer.fromSchema`](#model.DOMSerializer^fromSchema)
-//   will be used.
+//   will be used. This object will only have its
+//   [`serializeFragment`](#model.DOMSerializer.serializeFragment)
+//   method called, and you may provide an alternative object type
+//   implementing a compatible method.
 //
 //   clipboardTextSerializer:: ?(Slice) → string
 //   A function that will be called to get the text for the current