prosemirror-view 1.21.0 → 1.22.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "prosemirror-view",
-  "version": "1.21.0",
+  "version": "1.22.0",
   "description": "ProseMirror's view component",
   "main": "dist/index.js",
   "module": "dist/index.es.js",

package/src/decoration.js

@@ -25,6 +25,10 @@ class WidgetType {
        (this.spec.key && this.spec.key == other.spec.key ||
         this.toDOM == other.toDOM && compareObjs(this.spec, other.spec)))
   }
+
+  destroy(node) {
+    if (this.spec.destroy) this.spec.destroy(node)
+  }
 }
 
 class InlineType {
@@ -152,6 +156,10 @@ export class Decoration {
   //     key are interchangeable—if widgets differ in, for example,
   //     the behavior of some event handler, they should get
   //     different keys.
+  //    
+  //     destroy:: ?(node: dom.Node)
+  //     Called when the widget decoration is removed as a result of
+  //     mapping
   static widget(pos, toDOM, spec) {
     return new Decoration(pos, pos, new WidgetType(toDOM, spec))
   }

package/src/index.js

@@ -642,7 +642,7 @@ function checkStateComponent(plugin) {
 //   view](#state.PluginSpec.view) and
 //   [props](#state.PluginSpec.props). Passing plugins with a state
 //   component (a [state field](#state.PluginSpec.state) field or a
-//   [transaction)[#state.PluginSpec.filterTransaction] filter or
+//   [transaction](#state.PluginSpec.filterTransaction) filter or
 //   appender) will result in an error, since such plugins must be
 //   present in the state to work.
 //

package/src/viewdesc.js

@@ -526,6 +526,11 @@ class WidgetViewDesc extends ViewDesc {
     return mutation.type != "selection" || this.widget.spec.ignoreSelection
   }
 
+  destroy() {
+    this.widget.type.destroy(this.dom)
+    super.destroy()
+  }
+
   get domAtom() { return true }
 }
 
@@ -1038,6 +1043,8 @@ function patchAttributes(dom, prev, cur) {
       dom.classList.remove(prevList[i])
     for (let i = 0; i < curList.length; i++) if (prevList.indexOf(curList[i]) == -1)
       dom.classList.add(curList[i])
+    if (dom.classList.length == 0)
+      dom.removeAttribute("class")
   }
   if (prev.style != cur.style) {
     if (prev.style) {
@@ -1147,9 +1154,11 @@ class ViewTreeUpdater {
   findNodeMatch(node, outerDeco, innerDeco, index) {
     let children = this.top.children, found = -1
     if (index >= this.preMatch.index) {
-      for (let i = this.index; i < children.length; i++) if (children[i].matchesNode(node, outerDeco, innerDeco)) {
-        found = i
-        break
+      for (let i = this.index; i < children.length; i++) {
+        if (this.preMatch.matched.get(children[i]) == index) {
+          if (children[i].matchesNode(node, outerDeco, innerDeco)) found = i
+          break
+        }
       }
     } else {
       for (let i = this.index, e = Math.min(children.length, i + 1); i < e; i++) {