prosemirror-view 1.18.7 → 1.18.8

package/CHANGELOG.md

@@ -1,3 +1,15 @@
+## 1.18.8 (2021-06-23)
+
+### Bug fixes
+
+Work around a Safari bug where it draws the cursor at the start of the line when it is after an uneditable node at the end of the line.
+
+Fix an issue where the DOM could get out of sync when editing decorated text.
+
+Work around an issue where Firefox draws the cursor on the wrong line when after a newline.
+
+Fix a bug where double-clicking with the left mouse button and then pressing another mouse button was treated as a triple click.
+
 ## 1.18.7 (2021-05-20)
 
 ### Bug fixes

package/dist/index.es.js

@@ -694,7 +694,7 @@ var prototypeAccessors = { beforePosition: { configurable: true },size: { config
 ViewDesc.prototype.matchesWidget = function matchesWidget () { return false };
 ViewDesc.prototype.matchesMark = function matchesMark () { return false };
 ViewDesc.prototype.matchesNode = function matchesNode () { return false };
-ViewDesc.prototype.matchesHack = function matchesHack () { return false };
+ViewDesc.prototype.matchesHack = function matchesHack (_nodeName) { return false };
 
 prototypeAccessors.beforePosition.get = function () { return false };
 
@@ -960,9 +960,17 @@ ViewDesc.prototype.setSelection = function setSelection (anchor, head, root, for
     if (node.nodeType == 3) {
       brKludge = offset$1 && node.nodeValue[offset$1 - 1] == "\n";
       // Issue #1128
-      if (brKludge && offset$1 == node.nodeValue.length &&
-          node.nextSibling && node.nextSibling.nodeName == "BR")
-        { anchorDOM = headDOM = {node: node.parentNode, offset: domIndex(node) + 1}; }
+      if (brKludge && offset$1 == node.nodeValue.length) {
+        for (var scan = node, after = (void 0); scan; scan = scan.parentNode) {
+          if (after = scan.nextSibling) {
+            if (after.nodeName == "BR")
+              { anchorDOM = headDOM = {node: after.parentNode, offset: domIndex(after) + 1}; }
+            break
+          }
+          var desc = scan.pmViewDesc;
+          if (desc && desc.node && desc.node.isBlock) { break }
+        }
+      }
     } else {
       var prev = node.childNodes[offset$1 - 1];
       brKludge = prev && (prev.nodeName == "BR" || prev.contentEditable == "false");
@@ -971,8 +979,8 @@ ViewDesc.prototype.setSelection = function setSelection (anchor, head, root, for
   // Firefox can act strangely when the selection is in front of an
   // uneditable node. See #1163 and https://bugzilla.mozilla.org/show_bug.cgi?id=1709536
   if (result.gecko && domSel.focusNode && domSel.focusNode != headDOM.node && domSel.focusNode.nodeType == 1) {
-    var after = domSel.focusNode.childNodes[domSel.focusOffset];
-    if (after && after.contentEditable == "false") { force = true; }
+    var after$1 = domSel.focusNode.childNodes[domSel.focusOffset];
+    if (after$1 && after$1.contentEditable == "false") { force = true; }
   }
 
   if (!(force || brKludge && result.safari) &&
@@ -1482,6 +1490,12 @@ var TextViewDesc = /*@__PURE__*/(function (NodeViewDesc) {
     return new TextViewDesc(this.parent, node, this.outerDeco, this.innerDeco, dom, dom, view)
   };
 
+  TextViewDesc.prototype.markDirty = function markDirty (from, to) {
+    NodeViewDesc.prototype.markDirty.call(this, from, to);
+    if (this.dom != this.nodeDOM && (from == 0 || to == this.nodeDOM.nodeValue.length))
+      { this.dirty = NODE_DIRTY; }
+  };
+
   prototypeAccessors$4.domAtom.get = function () { return false };
 
   Object.defineProperties( TextViewDesc.prototype, prototypeAccessors$4 );
@@ -1503,7 +1517,7 @@ var TrailingHackViewDesc = /*@__PURE__*/(function (ViewDesc) {
   var prototypeAccessors$5 = { domAtom: { configurable: true } };
 
   TrailingHackViewDesc.prototype.parseRule = function parseRule () { return {ignore: true} };
-  TrailingHackViewDesc.prototype.matchesHack = function matchesHack () { return this.dirty == NOT_DIRTY };
+  TrailingHackViewDesc.prototype.matchesHack = function matchesHack (nodeName) { return this.dirty == NOT_DIRTY && this.dom.nodeName == nodeName };
   prototypeAccessors$5.domAtom.get = function () { return true };
 
   Object.defineProperties( TrailingHackViewDesc.prototype, prototypeAccessors$5 );
@@ -1847,13 +1861,20 @@ ViewTreeUpdater.prototype.addTextblockHacks = function addTextblockHacks () {
   if (!lastChild || // Empty textblock
       !(lastChild instanceof TextViewDesc) ||
       /\n$/.test(lastChild.node.text)) {
-    if (this.index < this.top.children.length && this.top.children[this.index].matchesHack()) {
-      this.index++;
-    } else {
-      var dom = document.createElement("br");
-      this.top.children.splice(this.index++, 0, new TrailingHackViewDesc(this.top, nothing, dom, null));
-      this.changed = true;
-    }
+    // Avoid a bug in Safari's cursor drawing (#1165)
+    if (result.safari && lastChild && lastChild.dom.contentEditable == "false")
+      { this.addHackNode("IMG"); }
+    this.addHackNode("BR");
+  }
+};
+
+ViewTreeUpdater.prototype.addHackNode = function addHackNode (nodeName) {
+  if (this.index < this.top.children.length && this.top.children[this.index].matchesHack(nodeName)) {
+    this.index++;
+  } else {
+    var dom = document.createElement(nodeName);
+    this.top.children.splice(this.index++, 0, new TrailingHackViewDesc(this.top, nothing, dom, null));
+    this.changed = true;
   }
 };
 
@@ -3364,7 +3385,7 @@ function dispatchEvent(view, event) {
 editHandlers.keydown = function (view, event) {
   view.shiftKey = event.keyCode == 16 || event.shiftKey;
   if (inOrNearComposition(view, event)) { return }
-  view.domObserver.forceFlush();
+  if (event.keyCode != 229) { view.domObserver.forceFlush(); }
   view.lastKeyCode = event.keyCode;
   view.lastKeyCodeTime = Date.now();
   // On iOS, if we preventDefault enter key presses, the virtual
@@ -3490,10 +3511,11 @@ function handleDoubleClick(view, pos, inside, event) {
 function handleTripleClick(view, pos, inside, event) {
   return runHandlerOnContext(view, "handleTripleClickOn", pos, inside, event) ||
     view.someProp("handleTripleClick", function (f) { return f(view, pos, event); }) ||
-    defaultTripleClick(view, inside)
+    defaultTripleClick(view, inside, event)
 }
 
-function defaultTripleClick(view, inside) {
+function defaultTripleClick(view, inside, event) {
+  if (event.button != 0) { return false }
   var doc = view.state.doc;
   if (inside == -1) {
     if (doc.inlineContent) {
@@ -3555,7 +3577,7 @@ var MouseDown = function MouseDown(view, pos, event, flushed) {
   this.event = event;
   this.flushed = flushed;
   this.selectNode = event[selectNodeModifier];
-  this.allowDefault = event.shiftKey;
+  this.allowDefault = event.shiftKey || event.button != 0;
 
   var targetNode, targetPos;
   if (pos.inside > -1) {
@@ -3575,7 +3597,8 @@ var MouseDown = function MouseDown(view, pos, event, flushed) {
 
   var ref = view.state;
   var selection = ref.selection;
-  if (targetNode.type.spec.draggable && targetNode.type.spec.selectable !== false ||
+  if (event.button == 0 &&
+      targetNode.type.spec.draggable && targetNode.type.spec.selectable !== false ||
       selection instanceof NodeSelection && selection.from <= targetPos && selection.to > targetPos)
     { this.mightDrag = {node: targetNode,
                       pos: targetPos,