v13.0.0-66 -- distribution files

package-lock.json

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

package.json

@@ -1,6 +1,6 @@
 {
   "name": "yjs",
-  "version": "13.0.0-62",
+  "version": "13.0.0-66",
   "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/Bindings/DomBinding/selection.js

@@ -19,17 +19,17 @@ function _getCurrentRelativeSelection (domBinding) {
 
 export const getCurrentRelativeSelection = typeof getSelection !== 'undefined' ? _getCurrentRelativeSelection : () => null
 
-export function beforeTransactionSelectionFixer (domBinding, remote) {
-  if (remote) {
-    relativeSelection = getCurrentRelativeSelection(domBinding)
-  }
+export function beforeTransactionSelectionFixer (domBinding) {
+  relativeSelection = getCurrentRelativeSelection(domBinding)
 }
 
 /**
+ * Reset the browser range after every transaction.
+ * This prevents any collapsing issues with the local selection.
  * @private
  */
-export function afterTransactionSelectionFixer (domBinding, remote) {
-  if (relativeSelection !== null && remote) {
+export function afterTransactionSelectionFixer (domBinding) {
+  if (relativeSelection !== null) {
     domBinding.restoreSelection(relativeSelection)
   }
 }

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) {