@atlaskit/editor-plugin-block-controls 3.3.5 → 3.3.6

package/CHANGELOG.md

@@ -1,5 +1,14 @@
 # @atlaskit/editor-plugin-block-controls
 
+## 3.3.6
+
+### Patch Changes
+
+- [#121718](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/pull-requests/121718)
+  [`ea6771bd5b498`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/ea6771bd5b498) -
+  [ux] ED-26814 fix copy and paste non bodied macro in list item
+- Updated dependencies
+
 ## 3.3.5
 
 ### Patch Changes

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

@@ -379,7 +379,7 @@ var moveNode = exports.moveNode = function moveNode(api) {
         var destParent = $to.node($to.depth);
         var sourceNode = $handlePos.nodeAfter;
 
-        //TODO: Does this need to be updated with new selection logic above? ^
+        //TODO: ED-26959 - Does this need to be updated with new selection logic above? ^
         // Move a layout column to top level
         if (sourceNode && isDragLayoutColumnToTopLevel($handlePos, $to)) {
           // need update after we support single column layout.

package/dist/cjs/editor-commands/move-to-layout.js

@@ -207,7 +207,7 @@ var canMoveToLayout = function canMoveToLayout(api, from, to, tr, moveNodeAtCurs
       sourceTo = Math.max(anchor, head);
       sourceContent = tr.doc.slice(sourceFrom, sourceTo).content;
 
-      // TODO: this might become expensive for large content, consider removing it if check has been done beforehand
+      // TODO: ED-26959 - this might become expensive for large content, consider removing it if check has been done beforehand
       if ((0, _checkFragment.containsNodeOfType)(sourceContent, 'layoutSection')) {
         return;
       }
@@ -286,7 +286,7 @@ var getBreakoutMode = function getBreakoutMode(content, breakout) {
   }
 };
 
-// TODO: As part of platform_editor_element_drag_and_drop_multiselect clean up,
+// TODO: ED-26959 - As part of platform_editor_element_drag_and_drop_multiselect clean up,
 // source content variable that has type of `PMNode | Fragment` should be updated to `Fragment` only
 var moveToLayout = exports.moveToLayout = function moveToLayout(api) {
   return function (from, to, options) {

package/dist/cjs/pm-plugins/handle-mouse-over.js

@@ -67,7 +67,7 @@ var handleMouseOver = exports.handleMouseOver = function handleMouseOver(view, e
     }
 
     // We want to exlude handles from showing for wrapped nodes
-    // TODO We should be able remove these check if we decided to
+    // TODO: ED-26959 - We should be able remove these check if we decided to
     // go we not decoration for wrapped image solution.
     if (['wrap-right', 'wrap-left'].includes(rootElement.getAttribute('layout') || '')) {
       return false;

package/dist/cjs/pm-plugins/main.js

@@ -633,7 +633,7 @@ var createPlugin = exports.createPlugin = function createPlugin(api, getIntl, no
             return false;
           }
 
-          // TODO: Review usage of posAtDOM here
+          // TODO: ED-26959 - Review usage of posAtDOM here
           var domPos = Math.max(view.posAtDOM(nodeElement, 0) - 1, 0);
           var nodeTarget = state.doc.nodeAt(domPos);
           var isSameNode = !!(nodeTarget && draggable !== null && draggable !== void 0 && draggable.eq(nodeTarget));

package/dist/cjs/pm-plugins/utils/active-anchor-tracker.js

@@ -59,7 +59,7 @@ var ActiveAnchorTracker = exports.ActiveAnchorTracker = /*#__PURE__*/function ()
       }
     }
   }]);
-}(); // TODO We should use a scoped ActiveAnchorTracker rather than the global static object.
+}(); // TODO: ED-26959 - We should use a scoped ActiveAnchorTracker rather than the global static object.
 // Move this into the plugin scope once the newApply functions becomes default apply.
 var defaultActiveAnchorTracker = exports.defaultActiveAnchorTracker = new ActiveAnchorTracker();
 var useActiveAnchorTracker = exports.useActiveAnchorTracker = function useActiveAnchorTracker(anchorName) {

package/dist/cjs/pm-plugins/utils/getSelection.js

@@ -59,7 +59,8 @@ var getSelection = exports.getSelection = function getSelection(tr, start) {
   var nodeName = node === null || node === void 0 ? void 0 : node.type.name;
   var isBlockQuoteWithMedia = nodeName === 'blockquote' && isNodeWithMedia(tr, start, nodeSize);
   var isBlockQuoteWithMediaOrExtension = nodeName === 'blockquote' && isNodeWithMediaOrExtension(tr, start, nodeSize);
-  if (isNodeSelection && nodeName !== 'blockquote' || ((0, _platformFeatureFlags.fg)('platform_editor_non_macros_copy_and_paste_fix') ? isBlockQuoteWithMediaOrExtension : isBlockQuoteWithMedia) ||
+  var isListWithMediaOrExtension = nodeName === 'bulletList' && isNodeWithMediaOrExtension(tr, start, nodeSize) || nodeName === 'orderedList' && isNodeWithMediaOrExtension(tr, start, nodeSize);
+  if (isNodeSelection && nodeName !== 'blockquote' || isListWithMediaOrExtension && (0, _platformFeatureFlags.fg)('platform_editor_non_macros_copy_and_paste_fix') || ((0, _platformFeatureFlags.fg)('platform_editor_non_macros_copy_and_paste_fix') ? isBlockQuoteWithMediaOrExtension : isBlockQuoteWithMedia) ||
   // decisionList/layoutColumn node is not selectable, but we want to select the whole node not just text
   ['decisionList', 'layoutColumn'].includes(nodeName || '')) {
     return new _state.NodeSelection($startPos);

package/dist/cjs/ui/drag-handle.js

@@ -236,7 +236,7 @@ var DragHandle = exports.DragHandle = function DragHandle(_ref) {
     view.focus();
   }, [isMultiSelect, api === null || api === void 0 || (_api$core2 = api.core) === null || _api$core2 === void 0 ? void 0 : _api$core2.actions, api === null || api === void 0 ? void 0 : api.blockControls.sharedState, api === null || api === void 0 || (_api$blockControls3 = api.blockControls) === null || _api$blockControls3 === void 0 ? void 0 : _api$blockControls3.commands, api === null || api === void 0 || (_api$analytics2 = api.analytics) === null || _api$analytics2 === void 0 ? void 0 : _api$analytics2.actions, view, dragHandleSelected, getPos, isTopLevelNode, nodeType, anchorName]);
 
-  // TODO - This needs to be investigated further. Drag preview generation is not always working
+  // TODO: ED-26959 - This needs to be investigated further. Drag preview generation is not always working
   // as expected with a node selection. This workaround sets the selection to the node on mouseDown,
   // but ensures the preview is generated correctly.
   var handleMouseDown = (0, _react.useCallback)(function () {

package/dist/cjs/ui/quick-insert-button.js

@@ -50,7 +50,7 @@ var containerStaticStyles = (0, _primitives.xcss)({
   zIndex: 'card'
 });
 
-// TODO: Share prop types between DragHandle - generic enough to create a type for block control decoration
+// TODO: ED-26959 - Share prop types between DragHandle - generic enough to create a type for block control decoration
 
 var isSelectionInNode = function isSelectionInNode(start, view) {
   var node = view.state.doc.nodeAt(start);

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

@@ -381,7 +381,7 @@ export const moveNode = api => (start, to, inputMethod = INPUT_METHOD.DRAG_AND_D
     const destParent = $to.node($to.depth);
     const sourceNode = $handlePos.nodeAfter;
 
-    //TODO: Does this need to be updated with new selection logic above? ^
+    //TODO: ED-26959 - Does this need to be updated with new selection logic above? ^
     // Move a layout column to top level
     if (sourceNode && isDragLayoutColumnToTopLevel($handlePos, $to)) {
       // need update after we support single column layout.

package/dist/es2019/editor-commands/move-to-layout.js

@@ -207,7 +207,7 @@ const canMoveToLayout = (api, from, to, tr, moveNodeAtCursorPos) => {
       sourceTo = Math.max(anchor, head);
       sourceContent = tr.doc.slice(sourceFrom, sourceTo).content;
 
-      // TODO: this might become expensive for large content, consider removing it if check has been done beforehand
+      // TODO: ED-26959 - this might become expensive for large content, consider removing it if check has been done beforehand
       if (containsNodeOfType(sourceContent, 'layoutSection')) {
         return;
       }
@@ -277,7 +277,7 @@ const getBreakoutMode = (content, breakout) => {
   }
 };
 
-// TODO: As part of platform_editor_element_drag_and_drop_multiselect clean up,
+// TODO: ED-26959 - As part of platform_editor_element_drag_and_drop_multiselect clean up,
 // source content variable that has type of `PMNode | Fragment` should be updated to `Fragment` only
 export const moveToLayout = api => (from, to, options) => ({
   tr

package/dist/es2019/pm-plugins/handle-mouse-over.js

@@ -62,7 +62,7 @@ export const handleMouseOver = (view, event, api) => {
     }
 
     // We want to exlude handles from showing for wrapped nodes
-    // TODO We should be able remove these check if we decided to
+    // TODO: ED-26959 - We should be able remove these check if we decided to
     // go we not decoration for wrapped image solution.
     if (['wrap-right', 'wrap-left'].includes(rootElement.getAttribute('layout') || '')) {
       return false;

package/dist/es2019/pm-plugins/main.js

@@ -615,7 +615,7 @@ export const createPlugin = (api, getIntl, nodeViewPortalProviderAPI) => {
             return false;
           }
 
-          // TODO: Review usage of posAtDOM here
+          // TODO: ED-26959 - Review usage of posAtDOM here
           const domPos = Math.max(view.posAtDOM(nodeElement, 0) - 1, 0);
           const nodeTarget = state.doc.nodeAt(domPos);
           const isSameNode = !!(nodeTarget && draggable !== null && draggable !== void 0 && draggable.eq(nodeTarget));

package/dist/es2019/pm-plugins/utils/active-anchor-tracker.js

@@ -39,7 +39,7 @@ export class ActiveAnchorTracker {
   }
 }
 
-// TODO We should use a scoped ActiveAnchorTracker rather than the global static object.
+// TODO: ED-26959 - We should use a scoped ActiveAnchorTracker rather than the global static object.
 // Move this into the plugin scope once the newApply functions becomes default apply.
 export const defaultActiveAnchorTracker = new ActiveAnchorTracker();
 export const useActiveAnchorTracker = (anchorName, activeAnchorTracker = defaultActiveAnchorTracker) => {

package/dist/es2019/pm-plugins/utils/getSelection.js

@@ -53,7 +53,8 @@ export const getSelection = (tr, start) => {
   const nodeName = node === null || node === void 0 ? void 0 : node.type.name;
   const isBlockQuoteWithMedia = nodeName === 'blockquote' && isNodeWithMedia(tr, start, nodeSize);
   const isBlockQuoteWithMediaOrExtension = nodeName === 'blockquote' && isNodeWithMediaOrExtension(tr, start, nodeSize);
-  if (isNodeSelection && nodeName !== 'blockquote' || (fg('platform_editor_non_macros_copy_and_paste_fix') ? isBlockQuoteWithMediaOrExtension : isBlockQuoteWithMedia) ||
+  const isListWithMediaOrExtension = nodeName === 'bulletList' && isNodeWithMediaOrExtension(tr, start, nodeSize) || nodeName === 'orderedList' && isNodeWithMediaOrExtension(tr, start, nodeSize);
+  if (isNodeSelection && nodeName !== 'blockquote' || isListWithMediaOrExtension && fg('platform_editor_non_macros_copy_and_paste_fix') || (fg('platform_editor_non_macros_copy_and_paste_fix') ? isBlockQuoteWithMediaOrExtension : isBlockQuoteWithMedia) ||
   // decisionList/layoutColumn node is not selectable, but we want to select the whole node not just text
   ['decisionList', 'layoutColumn'].includes(nodeName || '')) {
     return new NodeSelection($startPos);

package/dist/es2019/ui/drag-handle.js

@@ -217,7 +217,7 @@ export const DragHandle = ({
     view.focus();
   }, [isMultiSelect, api === null || api === void 0 ? void 0 : (_api$core2 = api.core) === null || _api$core2 === void 0 ? void 0 : _api$core2.actions, api === null || api === void 0 ? void 0 : api.blockControls.sharedState, api === null || api === void 0 ? void 0 : (_api$blockControls3 = api.blockControls) === null || _api$blockControls3 === void 0 ? void 0 : _api$blockControls3.commands, api === null || api === void 0 ? void 0 : (_api$analytics2 = api.analytics) === null || _api$analytics2 === void 0 ? void 0 : _api$analytics2.actions, view, dragHandleSelected, getPos, isTopLevelNode, nodeType, anchorName]);
 
-  // TODO - This needs to be investigated further. Drag preview generation is not always working
+  // TODO: ED-26959 - This needs to be investigated further. Drag preview generation is not always working
   // as expected with a node selection. This workaround sets the selection to the node on mouseDown,
   // but ensures the preview is generated correctly.
   const handleMouseDown = useCallback(() => {

package/dist/es2019/ui/global-styles.js

@@ -36,7 +36,7 @@ const extendedHoverZone = () => css({
       zIndex: -1
     }
   },
-  // TODO - ED-23995 this style override needs to be moved to the Rule styles after FF cleanup - packages/editor/editor-common/src/styles/shared/rule.ts
+  // TODO: ED-23995 - this style override needs to be moved to the Rule styles after FF cleanup - packages/editor/editor-common/src/styles/shared/rule.ts
   // eslint-disable-next-line @atlaskit/ui-styling-standard/no-nested-selectors
   'hr[data-drag-handler-anchor-name]': {
     overflow: 'visible'
@@ -109,7 +109,7 @@ const extendedHoverZoneNested = () => css({
       zIndex: 1
     }
   },
-  // TODO - ED-23995 this style override needs to be moved to the Rule styles after FF cleanup - packages/editor/editor-common/src/styles/shared/rule.ts
+  // TODO: ED-23995 - this style override needs to be moved to the Rule styles after FF cleanup packages/editor/editor-common/src/styles/shared/rule.ts
   // eslint-disable-next-line @atlaskit/ui-styling-standard/no-nested-selectors
   'hr[data-drag-handler-anchor-name]': {
     overflow: 'visible'

package/dist/es2019/ui/quick-insert-button.js

@@ -39,7 +39,7 @@ const containerStaticStyles = xcss({
   zIndex: 'card'
 });
 
-// TODO: Share prop types between DragHandle - generic enough to create a type for block control decoration
+// TODO: ED-26959 - Share prop types between DragHandle - generic enough to create a type for block control decoration
 
 const isSelectionInNode = (start, view) => {
   const node = view.state.doc.nodeAt(start);

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

@@ -373,7 +373,7 @@ export var moveNode = function moveNode(api) {
         var destParent = $to.node($to.depth);
         var sourceNode = $handlePos.nodeAfter;
 
-        //TODO: Does this need to be updated with new selection logic above? ^
+        //TODO: ED-26959 - Does this need to be updated with new selection logic above? ^
         // Move a layout column to top level
         if (sourceNode && isDragLayoutColumnToTopLevel($handlePos, $to)) {
           // need update after we support single column layout.

package/dist/esm/editor-commands/move-to-layout.js

@@ -201,7 +201,7 @@ var canMoveToLayout = function canMoveToLayout(api, from, to, tr, moveNodeAtCurs
       sourceTo = Math.max(anchor, head);
       sourceContent = tr.doc.slice(sourceFrom, sourceTo).content;
 
-      // TODO: this might become expensive for large content, consider removing it if check has been done beforehand
+      // TODO: ED-26959 - this might become expensive for large content, consider removing it if check has been done beforehand
       if (containsNodeOfType(sourceContent, 'layoutSection')) {
         return;
       }
@@ -280,7 +280,7 @@ var getBreakoutMode = function getBreakoutMode(content, breakout) {
   }
 };
 
-// TODO: As part of platform_editor_element_drag_and_drop_multiselect clean up,
+// TODO: ED-26959 - As part of platform_editor_element_drag_and_drop_multiselect clean up,
 // source content variable that has type of `PMNode | Fragment` should be updated to `Fragment` only
 export var moveToLayout = function moveToLayout(api) {
   return function (from, to, options) {

package/dist/esm/pm-plugins/handle-mouse-over.js

@@ -61,7 +61,7 @@ export var handleMouseOver = function handleMouseOver(view, event, api) {
     }
 
     // We want to exlude handles from showing for wrapped nodes
-    // TODO We should be able remove these check if we decided to
+    // TODO: ED-26959 - We should be able remove these check if we decided to
     // go we not decoration for wrapped image solution.
     if (['wrap-right', 'wrap-left'].includes(rootElement.getAttribute('layout') || '')) {
       return false;

package/dist/esm/pm-plugins/main.js

@@ -626,7 +626,7 @@ export var createPlugin = function createPlugin(api, getIntl, nodeViewPortalProv
             return false;
           }
 
-          // TODO: Review usage of posAtDOM here
+          // TODO: ED-26959 - Review usage of posAtDOM here
           var domPos = Math.max(view.posAtDOM(nodeElement, 0) - 1, 0);
           var nodeTarget = state.doc.nodeAt(domPos);
           var isSameNode = !!(nodeTarget && draggable !== null && draggable !== void 0 && draggable.eq(nodeTarget));

package/dist/esm/pm-plugins/utils/active-anchor-tracker.js

@@ -54,7 +54,7 @@ export var ActiveAnchorTracker = /*#__PURE__*/function () {
   }]);
 }();
 
-// TODO We should use a scoped ActiveAnchorTracker rather than the global static object.
+// TODO: ED-26959 - We should use a scoped ActiveAnchorTracker rather than the global static object.
 // Move this into the plugin scope once the newApply functions becomes default apply.
 export var defaultActiveAnchorTracker = new ActiveAnchorTracker();
 export var useActiveAnchorTracker = function useActiveAnchorTracker(anchorName) {

package/dist/esm/pm-plugins/utils/getSelection.js

@@ -53,7 +53,8 @@ export var getSelection = function getSelection(tr, start) {
   var nodeName = node === null || node === void 0 ? void 0 : node.type.name;
   var isBlockQuoteWithMedia = nodeName === 'blockquote' && isNodeWithMedia(tr, start, nodeSize);
   var isBlockQuoteWithMediaOrExtension = nodeName === 'blockquote' && isNodeWithMediaOrExtension(tr, start, nodeSize);
-  if (isNodeSelection && nodeName !== 'blockquote' || (fg('platform_editor_non_macros_copy_and_paste_fix') ? isBlockQuoteWithMediaOrExtension : isBlockQuoteWithMedia) ||
+  var isListWithMediaOrExtension = nodeName === 'bulletList' && isNodeWithMediaOrExtension(tr, start, nodeSize) || nodeName === 'orderedList' && isNodeWithMediaOrExtension(tr, start, nodeSize);
+  if (isNodeSelection && nodeName !== 'blockquote' || isListWithMediaOrExtension && fg('platform_editor_non_macros_copy_and_paste_fix') || (fg('platform_editor_non_macros_copy_and_paste_fix') ? isBlockQuoteWithMediaOrExtension : isBlockQuoteWithMedia) ||
   // decisionList/layoutColumn node is not selectable, but we want to select the whole node not just text
   ['decisionList', 'layoutColumn'].includes(nodeName || '')) {
     return new NodeSelection($startPos);

package/dist/esm/ui/drag-handle.js

@@ -231,7 +231,7 @@ export var DragHandle = function DragHandle(_ref) {
     view.focus();
   }, [isMultiSelect, api === null || api === void 0 || (_api$core2 = api.core) === null || _api$core2 === void 0 ? void 0 : _api$core2.actions, api === null || api === void 0 ? void 0 : api.blockControls.sharedState, api === null || api === void 0 || (_api$blockControls3 = api.blockControls) === null || _api$blockControls3 === void 0 ? void 0 : _api$blockControls3.commands, api === null || api === void 0 || (_api$analytics2 = api.analytics) === null || _api$analytics2 === void 0 ? void 0 : _api$analytics2.actions, view, dragHandleSelected, getPos, isTopLevelNode, nodeType, anchorName]);
 
-  // TODO - This needs to be investigated further. Drag preview generation is not always working
+  // TODO: ED-26959 - This needs to be investigated further. Drag preview generation is not always working
   // as expected with a node selection. This workaround sets the selection to the node on mouseDown,
   // but ensures the preview is generated correctly.
   var handleMouseDown = useCallback(function () {

package/dist/esm/ui/quick-insert-button.js

@@ -40,7 +40,7 @@ var containerStaticStyles = xcss({
   zIndex: 'card'
 });
 
-// TODO: Share prop types between DragHandle - generic enough to create a type for block control decoration
+// TODO: ED-26959 - Share prop types between DragHandle - generic enough to create a type for block control decoration
 
 var isSelectionInNode = function isSelectionInNode(start, view) {
   var node = view.state.doc.nodeAt(start);

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@atlaskit/editor-plugin-block-controls",
-	"version": "3.3.5",
+	"version": "3.3.6",
 	"description": "Block controls plugin for @atlaskit/editor-core",
 	"author": "Atlassian Pty Ltd",
 	"license": "Apache-2.0",
@@ -33,14 +33,14 @@
 	},
 	"dependencies": {
 		"@atlaskit/adf-schema": "^47.6.0",
-		"@atlaskit/editor-common": "^102.0.0",
+		"@atlaskit/editor-common": "^102.3.0",
 		"@atlaskit/editor-plugin-accessibility-utils": "^2.0.0",
 		"@atlaskit/editor-plugin-analytics": "^2.1.0",
 		"@atlaskit/editor-plugin-editor-disabled": "^2.0.0",
 		"@atlaskit/editor-plugin-feature-flags": "^1.3.0",
 		"@atlaskit/editor-plugin-metrics": "^3.4.0",
-		"@atlaskit/editor-plugin-quick-insert": "^2.0.0",
-		"@atlaskit/editor-plugin-selection": "^2.0.0",
+		"@atlaskit/editor-plugin-quick-insert": "^2.1.0",
+		"@atlaskit/editor-plugin-selection": "^2.1.0",
 		"@atlaskit/editor-plugin-width": "^3.0.0",
 		"@atlaskit/editor-prosemirror": "7.0.0",
 		"@atlaskit/editor-shared-styles": "^3.4.0",
@@ -49,10 +49,10 @@
 		"@atlaskit/platform-feature-flags": "^1.1.0",
 		"@atlaskit/pragmatic-drag-and-drop": "^1.5.0",
 		"@atlaskit/pragmatic-drag-and-drop-auto-scroll": "^2.1.0",
-		"@atlaskit/pragmatic-drag-and-drop-react-drop-indicator": "^2.1.0",
+		"@atlaskit/pragmatic-drag-and-drop-react-drop-indicator": "^2.2.0",
 		"@atlaskit/primitives": "^14.1.0",
 		"@atlaskit/theme": "^18.0.0",
-		"@atlaskit/tmp-editor-statsig": "^3.4.0",
+		"@atlaskit/tmp-editor-statsig": "^3.5.0",
 		"@atlaskit/tokens": "^4.3.0",
 		"@atlaskit/tooltip": "^20.0.0",
 		"@babel/runtime": "^7.0.0",