@atlaskit/editor-plugin-block-controls 3.3.15 → 3.3.16

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # @atlaskit/editor-plugin-block-controls
 
+## 3.3.16
+
+### Patch Changes
+
+- [#127305](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/pull-requests/127305)
+  [`d73df5795b977`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/d73df5795b977) -
+  Fix range error
+
 ## 3.3.15
 
 ### Patch Changes

package/dist/cjs/editor-commands/move-node.js

@@ -275,9 +275,15 @@ var moveNodeViaShortcut = exports.moveNodeViaShortcut = function moveNodeViaShor
         }
       }
       var nodeType = (_state$doc$nodeAt = state.doc.nodeAt(currentNodePos)) === null || _state$doc$nodeAt === void 0 ? void 0 : _state$doc$nodeAt.type.name;
-
-      // only move the node if the destination is at the same depth, not support moving a nested node to a parent node
-      var shouldMoveNode = (shouldEnableNestedDndA11y ? moveToPos > -1 && $currentNodePos.depth === state.doc.resolve(moveToPos).depth || nodeType === 'layoutColumn' : moveToPos > -1) || nodeType === 'layoutColumn' && (0, _platformFeatureFlags.fg)('platform_editor_advanced_layouts_accessibility');
+      var shouldMoveNode = false;
+      if (moveToPos > -1 && (0, _platformFeatureFlags.fg)('platform_editor_elements_dnd_multi_select_patch_2')) {
+        var isDestDepthSameAsSource = $currentNodePos.depth === state.doc.resolve(moveToPos).depth;
+        var isSourceLayoutColumn = nodeType === 'layoutColumn';
+        shouldMoveNode = (shouldEnableNestedDndA11y ? isDestDepthSameAsSource || isSourceLayoutColumn : true) || isSourceLayoutColumn && (0, _platformFeatureFlags.fg)('platform_editor_advanced_layouts_accessibility');
+      } else {
+        // only move the node if the destination is at the same depth, not support moving a nested node to a parent node
+        shouldMoveNode = (shouldEnableNestedDndA11y ? moveToPos > -1 && $currentNodePos.depth === state.doc.resolve(moveToPos).depth || nodeType === 'layoutColumn' : moveToPos > -1) || nodeType === 'layoutColumn' && (0, _platformFeatureFlags.fg)('platform_editor_advanced_layouts_accessibility');
+      }
       var _expandSelectionBound = (0, _selection.expandSelectionBounds)($currentNodePos, selection.$to),
         $newAnchor = _expandSelectionBound.$anchor,
         $newHead = _expandSelectionBound.$head;
@@ -340,7 +346,7 @@ var moveNode = exports.moveNode = function moveNode(api) {
     return function (_ref7) {
       var _api$blockControls$co;
       var tr = _ref7.tr;
-      if (!api) {
+      if (!api || (start < 0 || to < 0) && (0, _platformFeatureFlags.fg)('platform_editor_elements_dnd_multi_select_patch_2')) {
         return tr;
       }
       var handleNode = tr.doc.nodeAt(start);

package/dist/es2019/editor-commands/move-node.js

@@ -275,9 +275,15 @@ export const moveNodeViaShortcut = (api, direction, formatMessage) => {
         }
       }
       const nodeType = (_state$doc$nodeAt = state.doc.nodeAt(currentNodePos)) === null || _state$doc$nodeAt === void 0 ? void 0 : _state$doc$nodeAt.type.name;
-
-      // only move the node if the destination is at the same depth, not support moving a nested node to a parent node
-      const shouldMoveNode = (shouldEnableNestedDndA11y ? moveToPos > -1 && $currentNodePos.depth === state.doc.resolve(moveToPos).depth || nodeType === 'layoutColumn' : moveToPos > -1) || nodeType === 'layoutColumn' && fg('platform_editor_advanced_layouts_accessibility');
+      let shouldMoveNode = false;
+      if (moveToPos > -1 && fg('platform_editor_elements_dnd_multi_select_patch_2')) {
+        const isDestDepthSameAsSource = $currentNodePos.depth === state.doc.resolve(moveToPos).depth;
+        const isSourceLayoutColumn = nodeType === 'layoutColumn';
+        shouldMoveNode = (shouldEnableNestedDndA11y ? isDestDepthSameAsSource || isSourceLayoutColumn : true) || isSourceLayoutColumn && fg('platform_editor_advanced_layouts_accessibility');
+      } else {
+        // only move the node if the destination is at the same depth, not support moving a nested node to a parent node
+        shouldMoveNode = (shouldEnableNestedDndA11y ? moveToPos > -1 && $currentNodePos.depth === state.doc.resolve(moveToPos).depth || nodeType === 'layoutColumn' : moveToPos > -1) || nodeType === 'layoutColumn' && fg('platform_editor_advanced_layouts_accessibility');
+      }
       const {
         $anchor: $newAnchor,
         $head: $newHead
@@ -341,7 +347,7 @@ export const moveNode = api => (start, to, inputMethod = INPUT_METHOD.DRAG_AND_D
   tr
 }) => {
   var _api$blockControls$co2;
-  if (!api) {
+  if (!api || (start < 0 || to < 0) && fg('platform_editor_elements_dnd_multi_select_patch_2')) {
     return tr;
   }
   const handleNode = tr.doc.nodeAt(start);

package/dist/esm/editor-commands/move-node.js

@@ -269,9 +269,15 @@ export var moveNodeViaShortcut = function moveNodeViaShortcut(api, direction, fo
         }
       }
       var nodeType = (_state$doc$nodeAt = state.doc.nodeAt(currentNodePos)) === null || _state$doc$nodeAt === void 0 ? void 0 : _state$doc$nodeAt.type.name;
-
-      // only move the node if the destination is at the same depth, not support moving a nested node to a parent node
-      var shouldMoveNode = (shouldEnableNestedDndA11y ? moveToPos > -1 && $currentNodePos.depth === state.doc.resolve(moveToPos).depth || nodeType === 'layoutColumn' : moveToPos > -1) || nodeType === 'layoutColumn' && fg('platform_editor_advanced_layouts_accessibility');
+      var shouldMoveNode = false;
+      if (moveToPos > -1 && fg('platform_editor_elements_dnd_multi_select_patch_2')) {
+        var isDestDepthSameAsSource = $currentNodePos.depth === state.doc.resolve(moveToPos).depth;
+        var isSourceLayoutColumn = nodeType === 'layoutColumn';
+        shouldMoveNode = (shouldEnableNestedDndA11y ? isDestDepthSameAsSource || isSourceLayoutColumn : true) || isSourceLayoutColumn && fg('platform_editor_advanced_layouts_accessibility');
+      } else {
+        // only move the node if the destination is at the same depth, not support moving a nested node to a parent node
+        shouldMoveNode = (shouldEnableNestedDndA11y ? moveToPos > -1 && $currentNodePos.depth === state.doc.resolve(moveToPos).depth || nodeType === 'layoutColumn' : moveToPos > -1) || nodeType === 'layoutColumn' && fg('platform_editor_advanced_layouts_accessibility');
+      }
       var _expandSelectionBound = expandSelectionBounds($currentNodePos, selection.$to),
         $newAnchor = _expandSelectionBound.$anchor,
         $newHead = _expandSelectionBound.$head;
@@ -334,7 +340,7 @@ export var moveNode = function moveNode(api) {
     return function (_ref7) {
       var _api$blockControls$co;
       var tr = _ref7.tr;
-      if (!api) {
+      if (!api || (start < 0 || to < 0) && fg('platform_editor_elements_dnd_multi_select_patch_2')) {
         return tr;
       }
       var handleNode = tr.doc.nodeAt(start);

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@atlaskit/editor-plugin-block-controls",
-	"version": "3.3.15",
+	"version": "3.3.16",
 	"description": "Block controls plugin for @atlaskit/editor-core",
 	"author": "Atlassian Pty Ltd",
 	"license": "Apache-2.0",
@@ -33,7 +33,7 @@
 	},
 	"dependencies": {
 		"@atlaskit/adf-schema": "^47.6.0",
-		"@atlaskit/editor-common": "^102.9.0",
+		"@atlaskit/editor-common": "^102.10.0",
 		"@atlaskit/editor-plugin-accessibility-utils": "^2.0.0",
 		"@atlaskit/editor-plugin-analytics": "^2.2.0",
 		"@atlaskit/editor-plugin-editor-disabled": "^2.0.0",
@@ -169,6 +169,9 @@
 		},
 		"platform_editor_drag_layout_column_into_nodes": {
 			"type": "boolean"
+		},
+		"platform_editor_elements_dnd_multi_select_patch_2": {
+			"type": "boolean"
 		}
 	}
 }