v13.0.0-65 -- distribution files

package-lock.json

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

package.json

@@ -1,6 +1,6 @@
 {
   "name": "yjs",
-  "version": "13.0.0-62",
+  "version": "13.0.0-65",
   "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,7 +140,6 @@ 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
@@ -159,7 +158,6 @@ export default class DomBinding extends Binding {
           if (node !== baseNode || offset !== baseOffset) {
             baseNode = node
             baseOffset = offset
-            shouldUpdate = true
           }
         }
       }
@@ -171,18 +169,15 @@ export default class DomBinding extends Binding {
           if (node !== extentNode || offset !== extentOffset) {
             extentNode = node
             extentOffset = offset
-            shouldUpdate = true
           }
         }
       }
-      if (shouldUpdate) {
-        browserSelection.setBaseAndExtent(
-          baseNode,
-          baseOffset,
-          extentNode,
-          extentOffset
-        )
-      }
+      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._start`.
+     * which to insert to. Otherwise it is `parent._map`.
      * @type {String}
      */
     this._parentSub = null
@@ -125,8 +125,17 @@ export default class Item {
       return this._redone
     }
     let struct = this._copy()
-    let left = this._left
-    let right = this
+    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 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-62
+ * @version v13.0.0-65
  * @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._start`.
+     * which to insert to. Otherwise it is `parent._map`.
      * @type {String}
      */
     this._parentSub = null;
@@ -1682,8 +1682,17 @@ class Item {
       return this._redone
     }
     let struct = this._copy();
-    let left = this._left;
-    let right = this;
+    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 parent = this._parent;
     // make sure that parent is redone
     if (parent._deleted === true && parent._redone === null) {
@@ -5636,7 +5645,6 @@ 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
@@ -5655,7 +5663,6 @@ class DomBinding extends Binding {
           if (node !== baseNode || offset !== baseOffset) {
             baseNode = node;
             baseOffset = offset;
-            shouldUpdate = true;
           }
         }
       }
@@ -5667,18 +5674,15 @@ class DomBinding extends Binding {
           if (node !== extentNode || offset !== extentOffset) {
             extentNode = node;
             extentOffset = offset;
-            shouldUpdate = true;
           }
         }
       }
-      if (shouldUpdate) {
-        browserSelection.setBaseAndExtent(
-          baseNode,
-          baseOffset,
-          extentNode,
-          extentOffset
-        );
-      }
+      browserSelection.setBaseAndExtent(
+        baseNode,
+        baseOffset,
+        extentNode,
+        extentOffset
+      );
     }
   }