more utility around Y.UndoManager

package-lock.json

@@ -2210,9 +2210,9 @@
       }
     },
     "node_modules/lib0": {
-      "version": "0.2.46",
+      "version": "0.2.47",
-      "resolved": "https://registry.npmjs.org/lib0/-/lib0-0.2.46.tgz",
+      "resolved": "https://registry.npmjs.org/lib0/-/lib0-0.2.47.tgz",
-      "integrity": "sha512-U8V4Nc91EUSXLewvlJb0uSm9JI5rbiZKXBKHqWwPHX4g5cHx09dWAeD//UEZplm96xCrZ1BKDh5QaFHUIsnGIg==",
+      "integrity": "sha512-RXprIyaflw7OmFNMpb8HmvDhuRVUFXYCXrmynQN8OGbGevgMx9u6tjQG/yB0dOoDcuB1XXgqFn8Oy3RlKF/Qhg==",
       "dependencies": {
         "isomorphic.js": "^0.2.4"
       },
@@ -5591,9 +5591,9 @@
       }
     },
     "lib0": {
-      "version": "0.2.46",
+      "version": "0.2.47",
-      "resolved": "https://registry.npmjs.org/lib0/-/lib0-0.2.46.tgz",
+      "resolved": "https://registry.npmjs.org/lib0/-/lib0-0.2.47.tgz",
-      "integrity": "sha512-U8V4Nc91EUSXLewvlJb0uSm9JI5rbiZKXBKHqWwPHX4g5cHx09dWAeD//UEZplm96xCrZ1BKDh5QaFHUIsnGIg==",
+      "integrity": "sha512-RXprIyaflw7OmFNMpb8HmvDhuRVUFXYCXrmynQN8OGbGevgMx9u6tjQG/yB0dOoDcuB1XXgqFn8Oy3RlKF/Qhg==",
       "requires": {
         "isomorphic.js": "^0.2.4"
       }

src/utils/UndoManager.js

@@ -14,6 +14,7 @@ import {
 } from '../internals.js'
 
 import * as time from 'lib0/time'
+import * as array from 'lib0/array'
 import { Observable } from 'lib0/observable'
 
 class StackItem {
@@ -30,6 +31,18 @@ class StackItem {
     this.meta = new Map()
   }
 }
+/**
+ * @param {Transaction} tr
+ * @param {UndoManager} um
+ * @param {StackItem} stackItem
+ */
+const clearUndoManagerStackItem = (tr, um, stackItem) => {
+  iterateDeletedStructs(tr, stackItem.deletions, item => {
+    if (item instanceof Item && um.scope.some(type => isParentOf(type, item))) {
+      keepItem(item, false)
+    }
+  })
+}
 
 /**
  * @param {UndoManager} undoManager
@@ -133,7 +146,7 @@ const popStackItem = (undoManager, stack, eventType) => {
  * Fires 'stack-item-popped' event when a stack item was popped from either the
  * undo- or the redo-stack. You may restore the saved stack information from `event.stackItem.meta`.
  *
- * @extends {Observable<'stack-item-added'|'stack-item-popped'>}
+ * @extends {Observable<'stack-item-added'|'stack-item-popped'|'stack-cleared'>}
  */
 export class UndoManager extends Observable {
   /**
@@ -142,7 +155,11 @@ export class UndoManager extends Observable {
    */
   constructor (typeScope, { captureTimeout = 500, deleteFilter = () => true, trackedOrigins = new Set([null]) } = {}) {
     super()
-    this.scope = typeScope instanceof Array ? typeScope : [typeScope]
+    /**
+     * @type {Array<AbstractType<any>>}
+     */
+    this.scope = []
+    this.addToScope(typeScope)
     this.deleteFilter = deleteFilter
     trackedOrigins.add(this)
     this.trackedOrigins = trackedOrigins
@@ -175,7 +192,7 @@ export class UndoManager extends Observable {
         this.stopCapturing() // next undo should not be appended to last stack item
       } else if (!redoing) {
         // neither undoing nor redoing: delete redoStack
-        this.redoStack = []
+        this.clear(false, true)
       }
       const insertions = new DeleteSet()
       transaction.afterState.forEach((endClock, client) => {
@@ -210,25 +227,37 @@ export class UndoManager extends Observable {
         this.emit('stack-item-added', [{ stackItem: stack[stack.length - 1], origin: transaction.origin, type: undoing ? 'redo' : 'undo', changedParentTypes: transaction.changedParentTypes }, this])
       }
     })
+    this.doc.on('destroy', () => {
+      this.destroy()
+    })
   }
 
-  clear () {
+  /**
-    this.doc.transact(transaction => {
+   * @param {Array<AbstractType<any>> | AbstractType<any>} ytypes
-      /**
+   */
-       * @param {StackItem} stackItem
+  addToScope (ytypes) {
-       */
+    ytypes = array.isArray(ytypes) ? ytypes : [ytypes]
-      const clearItem = stackItem => {
+    ytypes.forEach(ytype => {
-        iterateDeletedStructs(transaction, stackItem.deletions, item => {
+      if (this.scope.every(yt => yt !== ytype)) {
-          if (item instanceof Item && this.scope.some(type => isParentOf(type, item))) {
+        this.scope.push(ytype)
-            keepItem(item, false)
-          }
-        })
       }
-      this.undoStack.forEach(clearItem)
-      this.redoStack.forEach(clearItem)
     })
-    this.undoStack = []
+  }
-    this.redoStack = []
+
+  clear (clearUndoStack = true, clearRedoStack = true) {
+    if ((clearUndoStack && this.canUndo()) || (clearRedoStack && this.canRedo())) {
+      this.doc.transact(tr => {
+        if (clearUndoStack) {
+          this.undoStack.forEach(item => clearUndoManagerStackItem(tr, this, item))
+          this.undoStack = []
+        }
+        if (clearRedoStack) {
+          this.redoStack.forEach(item => clearUndoManagerStackItem(tr, this, item))
+          this.redoStack = []
+        }
+        this.emit('stack-cleared', [{ undoStackCleared: clearUndoStack, redoStackCleared: clearRedoStack }])
+      })
+    }
   }
 
   /**