v13.0.0-62 -- distribution files

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "yjs",
-  "version": "13.0.0-65",
+  "version": "13.0.0-62",
   "lockfileVersion": 1,
   "requires": true,
   "dependencies": {

package.json

@@ -1,6 +1,6 @@
 {
   "name": "yjs",
-  "version": "13.0.0-65",
+  "version": "13.0.0-62",
   "description": "A framework for real-time p2p shared editing on any data",
   "main": "./y.node.js",
   "browser": "./y.js",

src/Bindings/DomBinding/DomBinding.js

@@ -140,6 +140,7 @@ export default class DomBinding extends Binding {
   restoreSelection (selection) {
     if (selection !== null) {
       const { to, from } = selection
+      let shouldUpdate = false
       /**
        * There is little information on the difference between anchor/focus and base/extent.
        * MDN doesn't even mention base/extent anymore.. though you still have to call
@@ -158,6 +159,7 @@ export default class DomBinding extends Binding {
           if (node !== baseNode || offset !== baseOffset) {
             baseNode = node
             baseOffset = offset
+            shouldUpdate = true
           }
         }
       }
@@ -169,15 +171,18 @@ export default class DomBinding extends Binding {
           if (node !== extentNode || offset !== extentOffset) {
             extentNode = node
             extentOffset = offset
+            shouldUpdate = true
           }
         }
       }
-      browserSelection.setBaseAndExtent(
-        baseNode,
-        baseOffset,
-        extentNode,
-        extentOffset
-      )
+      if (shouldUpdate) {
+        browserSelection.setBaseAndExtent(
+          baseNode,
+          baseOffset,
+          extentNode,
+          extentOffset
+        )
+      }
     }
   }
 

src/Struct/Item.js

@@ -87,7 +87,7 @@ export default class Item {
      * If the parent refers to this item with some kind of key (e.g. YMap, the
      * key is specified here. The key is then used to refer to the list in which
      * to insert this item. If `parentSub = null` type._start is the list in
-     * which to insert to. Otherwise it is `parent._map`.
+     * which to insert to. Otherwise it is `parent._start`.
      * @type {String}
      */
     this._parentSub = null
@@ -125,17 +125,8 @@ export default class Item {
       return this._redone
     }
     let struct = this._copy()
-    let left, right
-    if (this._parentSub === null) {
-      // Is an array item. Insert at the old position
-      left = this._left
-      right = this
-    } else {
-      // Is a map item. Insert at the start
-      left = null
-      right = this._parent._map.get(this._parentSub)
-      right._delete(y)
-    }
+    let left = this._left
+    let right = this
     let parent = this._parent
     // make sure that parent is redone
     if (parent._deleted === true && parent._redone === null) {

y.node.js

@@ -1,7 +1,7 @@
 
 /**
  * yjs - A framework for real-time p2p shared editing on any data
- * @version v13.0.0-65
+ * @version v13.0.0-62
  * @license MIT
  */
 
@@ -1644,7 +1644,7 @@ class Item {
      * If the parent refers to this item with some kind of key (e.g. YMap, the
      * key is specified here. The key is then used to refer to the list in which
      * to insert this item. If `parentSub = null` type._start is the list in
-     * which to insert to. Otherwise it is `parent._map`.
+     * which to insert to. Otherwise it is `parent._start`.
      * @type {String}
      */
     this._parentSub = null;
@@ -1682,17 +1682,8 @@ class Item {
       return this._redone
     }
     let struct = this._copy();
-    let left, right;
-    if (this._parentSub === null) {
-      // Is an array item. Insert at the old position
-      left = this._left;
-      right = this;
-    } else {
-      // Is a map item. Insert at the start
-      left = null;
-      right = this._parent._map.get(this._parentSub);
-      right._delete(y);
-    }
+    let left = this._left;
+    let right = this;
     let parent = this._parent;
     // make sure that parent is redone
     if (parent._deleted === true && parent._redone === null) {
@@ -5645,6 +5636,7 @@ class DomBinding extends Binding {
   restoreSelection (selection) {
     if (selection !== null) {
       const { to, from } = selection;
+      let shouldUpdate = false;
       /**
        * There is little information on the difference between anchor/focus and base/extent.
        * MDN doesn't even mention base/extent anymore.. though you still have to call
@@ -5663,6 +5655,7 @@ class DomBinding extends Binding {
           if (node !== baseNode || offset !== baseOffset) {
             baseNode = node;
             baseOffset = offset;
+            shouldUpdate = true;
           }
         }
       }
@@ -5674,15 +5667,18 @@ class DomBinding extends Binding {
           if (node !== extentNode || offset !== extentOffset) {
             extentNode = node;
             extentOffset = offset;
+            shouldUpdate = true;
           }
         }
       }
-      browserSelection.setBaseAndExtent(
-        baseNode,
-        baseOffset,
-        extentNode,
-        extentOffset
-      );
+      if (shouldUpdate) {
+        browserSelection.setBaseAndExtent(
+          baseNode,
+          baseOffset,
+          extentNode,
+          extentOffset
+        );
+      }
     }
   }