@atlaskit/editor-core 182.1.2 → 183.0.0

package/dist/es2019/plugins/fragment/pm-plugins/fragment-consistency.js

@@ -10,16 +10,8 @@
 import { SafePlugin } from '@atlaskit/editor-common/safe-plugin';
 import { PluginKey } from 'prosemirror-state';
 import { uuid } from '@atlaskit/adf-schema';
-import { stepAddsOneOf } from '../../../utils/step';
+import { getChangedNodes } from '../../../utils/document';
 const pluginKey = new PluginKey('fragmentMarkConsistencyPlugin');
-
-/**
- * Traverses a transaction's steps to see if we're inserting a node which supports a fragment mark
- */
-const checkIsAddingSupportedNode = (schema, transaction) => {
-  const nodesSupportingFragmentMark = getNodesSupportingFragmentMark(schema);
-  return transaction.steps.some(step => nodesSupportingFragmentMark.some(nodeType => stepAddsOneOf(step, new Set([nodeType]))));
-};
 const getNodesSupportingFragmentMark = schema => {
   const {
     table,
@@ -29,48 +21,6 @@ const getNodesSupportingFragmentMark = schema => {
   } = schema.nodes;
   return [table, extension, bodiedExtension, inlineExtension];
 };
-const regenerateFragmentIdIfNeeded = ({
-  transaction,
-  schema,
-  doc,
-  shouldRegenerateLocalId
-}) => {
-  const {
-    fragment
-  } = schema.marks;
-  const nodesSupportingFragmentMark = getNodesSupportingFragmentMark(schema);
-  let transactionChanged = false;
-  doc.descendants((node, pos) => {
-    const isSupportedNode = nodesSupportingFragmentMark.some(supportedNode => node.type === supportedNode);
-    if (!isSupportedNode) {
-      // continue traversing
-      return true;
-    }
-    const existingFragmentMark = node.marks.find(mark => mark.type === fragment);
-    if (!existingFragmentMark) {
-      // continue traversing
-      return true;
-    }
-    if (shouldRegenerateLocalId(existingFragmentMark.attrs.localId)) {
-      transactionChanged = true;
-      transaction.setNodeMarkup(pos, undefined, node.attrs, node.marks.map(mark => {
-        if (mark.type === fragment) {
-          mark.attrs.localId = uuid.generate();
-          mark.attrs.name = null;
-        }
-        return mark;
-      }));
-    }
-
-    /**
-     * Continue traversing, as we can encounter inline extension nodes pretty much anywhere
-     */
-    return true;
-  });
-  return {
-    transactionChanged
-  };
-};
 
 /**
  * Ensures presence of `fragment` mark on certain node types and the uniqueness of their `localId` attributes
@@ -80,8 +30,14 @@ export const createPlugin = dispatch => new SafePlugin({
   appendTransaction: (transactions, _oldState, newState) => {
     let modified = false;
     const tr = newState.tr;
+    const {
+      fragment
+    } = newState.schema.marks;
+    const supportedNodeTypes = getNodesSupportingFragmentMark(newState.schema);
+    const addedSupportedNodes = new Set();
+    const addedSupportedNodesPos = new Map();
+    const localIds = new Set();
     transactions.forEach(transaction => {
-      const fragmentLocalIdsObserved = new Set();
       if (!transaction.docChanged) {
         return;
       }
@@ -92,30 +48,59 @@ export const createPlugin = dispatch => new SafePlugin({
       if (uiEvent === 'cut') {
         return;
       }
+      const changedNodes = getChangedNodes(transaction);
+      for (const {
+        node
+      } of changedNodes) {
+        if (!supportedNodeTypes.includes(node.type)) {
+          continue;
+        }
+        addedSupportedNodes.add(node);
+      }
+    });
+    if (!addedSupportedNodes.size) {
+      return;
+    }
 
-      /** Check if we're actually inserting a supported node, otherwise we can ignore this tr */
-      const isAddingSupportedNode = checkIsAddingSupportedNode(newState.schema, transaction);
-      if (!isAddingSupportedNode) {
-        return;
+    // Get existing fragment marks localIds on the page
+    newState.doc.descendants((node, pos) => {
+      if (addedSupportedNodes.has(node)) {
+        addedSupportedNodesPos.set(node, pos);
+        return true;
+      }
+      if (!supportedNodeTypes.includes(node.type)) {
+        return true;
+      }
+      const existingFragmentMark = node.marks.find(mark => mark.type === fragment);
+      if (!existingFragmentMark) {
+        // continue traversing
+        return true;
+      }
+      localIds.add(existingFragmentMark.attrs.localId);
+      return true;
+    });
+
+    // If an added node has localId that collides with existing node, generate new localId
+    for (const node of addedSupportedNodes) {
+      const pos = addedSupportedNodesPos.get(node);
+      if (pos === undefined) {
+        continue;
+      }
+      const existingFragmentMark = node.marks.find(mark => mark.type === fragment);
+      if (!existingFragmentMark) {
+        continue;
       }
-      const {
-        transactionChanged
-      } = regenerateFragmentIdIfNeeded({
-        doc: newState.doc,
-        schema: newState.schema,
-        transaction: tr,
-        shouldRegenerateLocalId: localId => {
-          if (fragmentLocalIdsObserved.has(localId)) {
-            return true;
+      if (localIds.has(existingFragmentMark.attrs.localId)) {
+        tr.setNodeMarkup(pos, undefined, node.attrs, node.marks.map(mark => {
+          if (mark.type === fragment) {
+            mark.attrs.localId = uuid.generate();
+            mark.attrs.name = null;
           }
-          fragmentLocalIdsObserved.add(localId);
-          return false;
-        }
-      });
-      if (transactionChanged) {
+          return mark;
+        }));
         modified = true;
       }
-    });
+    }
     if (modified) {
       return tr;
     }

package/dist/es2019/plugins/history/actions.js

@@ -1,4 +1,4 @@
-export let HistoryActionTypes;
-(function (HistoryActionTypes) {
+export let HistoryActionTypes = /*#__PURE__*/function (HistoryActionTypes) {
   HistoryActionTypes["UPDATE"] = "UPDATE";
-})(HistoryActionTypes || (HistoryActionTypes = {}));
+  return HistoryActionTypes;
+}({});

package/dist/es2019/plugins/hyperlink/pm-plugins/input-rule.js

@@ -66,6 +66,9 @@ export function createInputRulePlugin(schema, skipAnalytics = false, featureFlag
       href: url
     });
     const tr = state.tr.replaceWith(start + prefix.length, end, schema.text((linkText || '').trim(), [markType]));
+    addLinkMetadata(state.selection, tr, {
+      inputMethod: INPUT_METHOD.FORMATTING
+    });
     if (skipAnalytics) {
       return tr;
     }

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

@@ -2,20 +2,20 @@ import { SafePlugin } from '@atlaskit/editor-common/safe-plugin';
 import { PluginKey, TextSelection } from 'prosemirror-state';
 import uuid from 'uuid';
 import { shallowEqual } from '../../../utils';
-export let LinkAction;
-(function (LinkAction) {
+export let LinkAction = /*#__PURE__*/function (LinkAction) {
   LinkAction["SHOW_INSERT_TOOLBAR"] = "SHOW_INSERT_TOOLBAR";
   LinkAction["HIDE_TOOLBAR"] = "HIDE_TOOLBAR";
   LinkAction["SELECTION_CHANGE"] = "SELECTION_CHANGE";
   LinkAction["INSERT_LINK_TOOLBAR"] = "INSERT";
   LinkAction["EDIT_INSERTED_TOOLBAR"] = "EDIT_INSERTED_TOOLBAR";
-})(LinkAction || (LinkAction = {}));
-export let InsertStatus;
-(function (InsertStatus) {
+  return LinkAction;
+}({});
+export let InsertStatus = /*#__PURE__*/function (InsertStatus) {
   InsertStatus["EDIT_LINK_TOOLBAR"] = "EDIT";
   InsertStatus["INSERT_LINK_TOOLBAR"] = "INSERT";
   InsertStatus["EDIT_INSERTED_TOOLBAR"] = "EDIT_INSERTED";
-})(InsertStatus || (InsertStatus = {}));
+  return InsertStatus;
+}({});
 export const canLinkBeCreatedInRange = (from, to) => state => {
   if (!state.doc.rangeHasMark(from, to, state.schema.marks.link)) {
     const $from = state.doc.resolve(from);

package/dist/es2019/plugins/hyperlink/ui/HyperlinkAddToolbar/HyperlinkAddToolbar.js

@@ -114,10 +114,9 @@ const mapSearchProviderResultToLinkSearchItemData = ({
   prefetch: false
 });
 export class HyperlinkLinkAddToolbar extends PureComponent {
-  /* To prevent double submit */
-
   constructor(props) {
     super(props);
+    /* To prevent double submit */
     _defineProperty(this, "submitted", false);
     _defineProperty(this, "urlInputContainer", null);
     _defineProperty(this, "displayTextInputContainer", null);

package/dist/es2019/plugins/hyperlink/utils.js

@@ -1,3 +1,6 @@
+// File has been copied to packages/editor/editor-plugin-ai/src/provider/markdown-transformer/utils/hyperlink.ts
+// If changes are made to this file, please make the same update in the linked file.
+
 import { mapSlice } from '../../utils/slice';
 import { isSafeUrl, linkify, normalizeUrl as normaliseLinkHref } from '@atlaskit/adf-schema';
 

package/dist/es2019/plugins/list/utils/node.js

@@ -21,11 +21,11 @@ function isListNodeValidContent(node) {
   const listFragment = Fragment.from(bulletList.createAndFill());
   return !isListItemNode(node) && node.type.validContent(listFragment);
 }
-export let JoinDirection;
-(function (JoinDirection) {
+export let JoinDirection = /*#__PURE__*/function (JoinDirection) {
   JoinDirection[JoinDirection["LEFT"] = 1] = "LEFT";
   JoinDirection[JoinDirection["RIGHT"] = -1] = "RIGHT";
-})(JoinDirection || (JoinDirection = {}));
+  return JoinDirection;
+}({});
 export const joinSiblingLists = ({
   tr,
   direction,

package/dist/es2019/plugins/media/nodeviews/mediaNodeUpdater.js

@@ -6,6 +6,8 @@ import { ACTION, ACTION_SUBJECT, EVENT_TYPE } from '../../analytics';
 import { replaceExternalMedia, updateAllMediaSingleNodesAttrs, updateCurrentMediaNodeAttrs, updateMediaSingleNodeAttrs } from '../commands/helpers';
 export class MediaNodeUpdater {
   constructor(props) {
+    // Updates the node with contextId if it doesn't have one already
+    // TODO [MS-2258]: remove updateContextId in order to only use updateMediaSingleFileAttrs
     _defineProperty(this, "updateContextId", async () => {
       const attrs = this.getAttrs();
       if (!attrs || attrs.type !== 'file') {
@@ -238,6 +240,7 @@ export class MediaNodeUpdater {
         __fileSize: size
       })(this.props.view.state, this.props.view.dispatch);
     });
+    // Copies the pasted node into the current collection using a getPos handler
     _defineProperty(this, "copyNodeFromPos", async (getPos, traceContext) => {
       const attrs = this.getAttrs();
       if ((attrs === null || attrs === void 0 ? void 0 : attrs.type) !== 'file') {
@@ -252,6 +255,7 @@ export class MediaNodeUpdater {
         getPos
       })(this.props.view.state, this.props.view.dispatch);
     });
+    // Copies the pasted node into the current collection
     _defineProperty(this, "copyNode", async traceContext => {
       const attrs = this.getAttrs();
       const {
@@ -303,9 +307,6 @@ export class MediaNodeUpdater {
     const attrs = this.getAttrs();
     return !!(attrs && attrs.type === 'external' && isMediaBlobUrl(attrs.url));
   }
-
-  // Updates the node with contextId if it doesn't have one already
-  // TODO [MS-2258]: remove updateContextId in order to only use updateMediaSingleFileAttrs
   async getRemoteDimensions() {
     const mediaProvider = await this.props.mediaProvider;
     const {
@@ -369,12 +370,7 @@ export class MediaNodeUpdater {
       await this.uploadExternalMedia(getPos);
     }
   }
-
-  // Copies the pasted node into the current collection using a getPos handler
-
-  // Copies the pasted node into the current collection
 }
-
 const hasPrivateAttrsChanged = (currentAttrs, newAttrs) => {
   return currentAttrs.__fileName !== newAttrs.__fileName || currentAttrs.__fileMimeType !== newAttrs.__fileMimeType || currentAttrs.__fileSize !== newAttrs.__fileSize || currentAttrs.__contextId !== newAttrs.__contextId;
 };

package/dist/es2019/plugins/media/nodeviews/mediaSingle.js

@@ -146,6 +146,8 @@ export default class MediaSingleNode extends Component {
       tr.setSelection(NodeSelection.create(tr.doc, pos));
       return dispatch(tr);
     });
+    // Workaound for iOS 16 Caption selection issue
+    // @see https://product-fabric.atlassian.net/browse/MEX-2012
     _defineProperty(this, "onMediaSingleClicked", event => {
       var _this$captionPlaceHol;
       if (!browser.ios) {
@@ -204,9 +206,6 @@ export default class MediaSingleNode extends Component {
       contextIdentifierProvider: await contextIdentifierProvider
     });
   }
-
-  // Workaound for iOS 16 Caption selection issue
-  // @see https://product-fabric.atlassian.net/browse/MEX-2012
   render() {
     const {
       selected,

package/dist/es2019/plugins/media/pm-plugins/linking/actions.js

@@ -1,7 +1,7 @@
-export let MediaLinkingActionsTypes;
-(function (MediaLinkingActionsTypes) {
+export let MediaLinkingActionsTypes = /*#__PURE__*/function (MediaLinkingActionsTypes) {
   MediaLinkingActionsTypes["showToolbar"] = "MEDIA_SHOW_TOOLBAR";
   MediaLinkingActionsTypes["hideToolbar"] = "MEDIA_HIDE_TOOLBAR";
   MediaLinkingActionsTypes["setUrl"] = "MEDIA_SET_LINK_TO";
   MediaLinkingActionsTypes["unlink"] = "MEDIA_LINKING_UNLINK";
-})(MediaLinkingActionsTypes || (MediaLinkingActionsTypes = {}));
+  return MediaLinkingActionsTypes;
+}({});

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

@@ -43,8 +43,6 @@ const createDropPlaceholder = (intl, allowDropLine) => {
 };
 const MEDIA_RESOLVED_STATES = ['ready', 'error', 'cancelled'];
 export class MediaPluginStateImplementation {
-  // @ts-ignore: private is OK
-
   constructor(_state, options, mediaOptions, newInsertionBehaviour, _dispatch) {
     _defineProperty(this, "allowsUploads", false);
     _defineProperty(this, "ignoreLinks", false);
@@ -143,6 +141,11 @@ export class MediaPluginStateImplementation {
       }
       return type === mediaSingle;
     });
+    /**
+     * we insert a new file by inserting a initial state for that file.
+     *
+     * called when we insert a new file via the picker (connected via pickerfacade)
+     */
     _defineProperty(this, "insertFile", (mediaState, onMediaStateChanged, pickerType) => {
       var _mediaState$collectio, _this$mediaOptions2;
       const {
@@ -219,7 +222,17 @@ export class MediaPluginStateImplementation {
     _defineProperty(this, "onPopupToggle", onPopupToggleCallback => {
       this.onPopupToggleCallback = onPopupToggleCallback;
     });
+    /**
+     * Returns a promise that is resolved after all pending operations have been finished.
+     * An optional timeout will cause the promise to reject if the operation takes too long
+     *
+     * NOTE: The promise will resolve even if some of the media have failed to process.
+     */
     _defineProperty(this, "waitForPendingTasks", this.taskManager.waitForPendingTasks);
+    /**
+     * Called from React UI Component when user clicks on "Delete" icon
+     * inside of it
+     */
     _defineProperty(this, "handleMediaNodeRemoval", (node, getPos) => {
       let getNode = node;
       if (!getNode) {
@@ -245,6 +258,9 @@ export class MediaPluginStateImplementation {
       }
       this.nodeCount.set(id, count - 1);
     });
+    /**
+     * Called from React UI Component on componentDidMount
+     */
     _defineProperty(this, "handleMediaNodeMount", (node, getPos) => {
       this.trackMediaNodeAddition(node);
       this.mediaNodes.unshift({
@@ -252,6 +268,10 @@ export class MediaPluginStateImplementation {
         getPos
       });
     });
+    /**
+     * Called from React UI Component on componentWillUnmount and UNSAFE_componentWillReceiveProps
+     * when React component's underlying node property is replaced with a new node
+     */
     _defineProperty(this, "handleMediaNodeUnmount", oldNode => {
       this.trackMediaNodeRemoval(oldNode);
       this.mediaNodes = this.mediaNodes.filter(({
@@ -399,38 +419,9 @@ export class MediaPluginStateImplementation {
     }
     return;
   }
-
-  /**
-   * we insert a new file by inserting a initial state for that file.
-   *
-   * called when we insert a new file via the picker (connected via pickerfacade)
-   */
-
-  /**
-   * Returns a promise that is resolved after all pending operations have been finished.
-   * An optional timeout will cause the promise to reject if the operation takes too long
-   *
-   * NOTE: The promise will resolve even if some of the media have failed to process.
-   */
-
   setView(view) {
     this.view = view;
   }
-
-  /**
-   * Called from React UI Component when user clicks on "Delete" icon
-   * inside of it
-   */
-
-  /**
-   * Called from React UI Component on componentDidMount
-   */
-
-  /**
-   * Called from React UI Component on componentWillUnmount and UNSAFE_componentWillReceiveProps
-   * when React component's underlying node property is replaced with a new node
-   */
-
   destroy() {
     if (this.destroyed) {
       return;

package/dist/es2019/plugins/mobile-dimensions/actions.js

@@ -1,7 +1,7 @@
-export let MobileDimensionsActionTypes;
-(function (MobileDimensionsActionTypes) {
+export let MobileDimensionsActionTypes = /*#__PURE__*/function (MobileDimensionsActionTypes) {
   MobileDimensionsActionTypes["SET_KEYBOARD_HEIGHT"] = "SET_KEYBOARD_HEIGHT";
   MobileDimensionsActionTypes["SET_WINDOW_HEIGHT"] = "SET_WINDOW_HEIGHT";
   MobileDimensionsActionTypes["SET_MOBILE_PADDING_TOP"] = "SET_MOBILE_PADDING_TOP";
   MobileDimensionsActionTypes["SET_IS_EXPANDED"] = "SET_IS_EXPANDED";
-})(MobileDimensionsActionTypes || (MobileDimensionsActionTypes = {}));
+  return MobileDimensionsActionTypes;
+}({});

package/dist/es2019/plugins/paste/actions.js

@@ -1,5 +1,5 @@
-export let PastePluginActionTypes;
-(function (PastePluginActionTypes) {
+export let PastePluginActionTypes = /*#__PURE__*/function (PastePluginActionTypes) {
   PastePluginActionTypes["START_TRACKING_PASTED_MACRO_POSITIONS"] = "START_TRACKING_PASTED_MACRO_POSITIONS";
   PastePluginActionTypes["STOP_TRACKING_PASTED_MACRO_POSITIONS"] = "STOP_TRACKING_PASTED_MACRO_POSITIONS";
-})(PastePluginActionTypes || (PastePluginActionTypes = {}));
+  return PastePluginActionTypes;
+}({});

package/dist/es2019/plugins/paste/linkify-md-plugin.js

@@ -1,3 +1,6 @@
+// File has been copied to packages/editor/editor-plugin-ai/src/provider/markdown-transformer/md/linkify-md-plugin.ts
+// If changes are made to this file, please make the same update in the linked file.
+
 import { linkifyMatch } from '@atlaskit/adf-schema';
 import LinkifyIt from 'linkify-it';
 import { findFilepaths, isLinkInMatches } from '../hyperlink/utils';

package/dist/es2019/plugins/paste/md.js

@@ -1,3 +1,6 @@
+// File has been copied to packages/editor/editor-plugin-ai/src/provider/markdown-transformer/md/index.ts
+// If changes are made to this file, please make the same update in the linked file.
+
 import MarkdownIt from 'markdown-it';
 import linkify from './linkify-md-plugin';
 import newline from './newline-md-plugin';

package/dist/es2019/plugins/paste/newline-md-plugin.js

@@ -1,3 +1,6 @@
+// File has been copied to packages/editor/editor-plugin-ai/src/provider/markdown-transformer/md/newline-md-plugin.ts
+// If changes are made to this file, please make the same update in the linked file.
+
 // ED-15363: modified version of the original newline plugin
 // https://github.com/markdown-it/markdown-it/blob/master/lib/rules_inline/newline.js
 const newline = (state, silent) => {

package/dist/es2019/plugins/paste/paragraph-md-plugin.js

@@ -1,3 +1,6 @@
+// File has been copied to packages/editor/editor-plugin-ai/src/provider/markdown-transformer/md/paragraph-md-plugin.ts
+// If changes are made to this file, please make the same update in the linked file.
+
 // ED-15363: modified version of the original paragraph plugin
 // https://github.com/markdown-it/markdown-it/blob/master/lib/rules_block/paragraph.js
 

package/dist/es2019/plugins/selection/actions.js

@@ -1,5 +1,5 @@
-export let SelectionActionTypes;
-(function (SelectionActionTypes) {
+export let SelectionActionTypes = /*#__PURE__*/function (SelectionActionTypes) {
   SelectionActionTypes["SET_DECORATIONS"] = "SET_DECORATIONS";
   SelectionActionTypes["SET_RELATIVE_SELECTION"] = "SET_RELATIVE_SELECTION";
-})(SelectionActionTypes || (SelectionActionTypes = {}));
+  return SelectionActionTypes;
+}({});

package/dist/es2019/plugins/selection/gap-cursor/direction.js

@@ -1,12 +1,12 @@
-export let Direction;
-(function (Direction) {
+export let Direction = /*#__PURE__*/function (Direction) {
   Direction["UP"] = "up";
   Direction["RIGHT"] = "right";
   Direction["DOWN"] = "down";
   Direction["LEFT"] = "left";
   Direction["BACKWARD"] = "backward";
   Direction["FORWARD"] = "forward";
-})(Direction || (Direction = {}));
+  return Direction;
+}({});
 export function isBackward(dir) {
   return [Direction.UP, Direction.LEFT, Direction.BACKWARD].indexOf(dir) !== -1;
 }

package/dist/es2019/plugins/selection/types.js

@@ -2,8 +2,8 @@ import { PluginKey } from 'prosemirror-state';
 import { RelativeSelectionPos } from '@atlaskit/editor-common/selection';
 export const selectionPluginKey = new PluginKey('selection');
 export { RelativeSelectionPos };
-export let SelectionDirection;
-(function (SelectionDirection) {
+export let SelectionDirection = /*#__PURE__*/function (SelectionDirection) {
   SelectionDirection[SelectionDirection["Before"] = -1] = "Before";
   SelectionDirection[SelectionDirection["After"] = 1] = "After";
-})(SelectionDirection || (SelectionDirection = {}));
+  return SelectionDirection;
+}({});

package/dist/es2019/plugins/status/ui/statusPicker.js

@@ -12,12 +12,12 @@ import withOuterListeners from '../../../ui/with-outer-listeners';
 import { DEFAULT_STATUS } from '../actions';
 import { analyticsState, createStatusAnalyticsAndFire } from '../analytics';
 const PopupWithListeners = withOuterListeners(Popup);
-export let InputMethod;
-(function (InputMethod) {
+export let InputMethod = /*#__PURE__*/function (InputMethod) {
   InputMethod["blur"] = "blur";
   InputMethod["escKey"] = "escKey";
   InputMethod["enterKey"] = "enterKey";
-})(InputMethod || (InputMethod = {}));
+  return InputMethod;
+}({});
 const pickerContainer = css`
   background: ${`var(--ds-surface-overlay, ${N0})`};
   padding: ${gridSize()}px 0;
@@ -100,6 +100,9 @@ export class StatusPickerWithoutAnalytcs extends React.Component {
       this.inputMethod = InputMethod.enterKey;
       this.props.onEnter(this.state);
     });
+    // cancel bubbling to fix clickOutside logic:
+    // popup re-renders its content before the click event bubbles up to the document
+    // therefore click target element would be different from the popup content
     _defineProperty(this, "handlePopupClick", event => event.nativeEvent.stopImmediatePropagation());
     this.state = this.extractStateFromProps(props);
     this.createStatusAnalyticsAndFireFunc = createStatusAnalyticsAndFire(props.createAnalyticsEvent);
@@ -214,9 +217,5 @@ export class StatusPickerWithoutAnalytcs extends React.Component {
       onEnter: this.onEnter
     })));
   }
-
-  // cancel bubbling to fix clickOutside logic:
-  // popup re-renders its content before the click event bubbles up to the document
-  // therefore click target element would be different from the popup content
 }
 export default withAnalyticsEvents()(StatusPickerWithoutAnalytcs);

package/dist/es2019/plugins/tasks-and-decisions/nodeviews/taskItem.js

@@ -25,6 +25,15 @@ class Task extends ReactNodeView {
       tr.setMeta('scrollIntoView', false);
       this.view.dispatch(tr);
     });
+    /**
+     * Dynamically generates analytics data relating to the parent list.
+     *
+     * Required to be dynamic, as list (in prosemirror model) may have
+     * changed (e.g. item movements, or additional items in list).
+     * This node view will have not rerendered for those changes, so
+     * cannot render the position and listSize into the
+     * AnalyticsContext at initial render time.
+     */
     _defineProperty(this, "addListAnalyticsData", event => {
       try {
         const resolvedPos = this.view.state.doc.resolve(this.getPos());
@@ -60,17 +69,6 @@ class Task extends ReactNodeView {
   isContentEmpty(node) {
     return node.content.childCount === 0;
   }
-
-  /**
-   * Dynamically generates analytics data relating to the parent list.
-   *
-   * Required to be dynamic, as list (in prosemirror model) may have
-   * changed (e.g. item movements, or additional items in list).
-   * This node view will have not rerendered for those changes, so
-   * cannot render the position and listSize into the
-   * AnalyticsContext at initial render time.
-   */
-
   createDomRef() {
     const domRef = document.createElement('div');
     domRef.style['list-style-type'] = 'none';

package/dist/es2019/plugins/tasks-and-decisions/pm-plugins/commands.js

@@ -1,11 +1,13 @@
 import { findWrapping, ReplaceAroundStep } from 'prosemirror-transform';
 import { findCutBefore } from '../../../utils/commands';
-import { getBlockRange, isActionOrDecisionItem, isActionOrDecisionList, liftBlock } from './helpers';
+import { getBlockRange, isActionOrDecisionItem, isActionOrDecisionList, liftBlock, subtreeHeight } from './helpers';
+import { normalizeTaskItemsSelection } from '../utils';
 export const liftSelection = (state, dispatch) => {
+  const normalizedSelection = normalizeTaskItemsSelection(state.selection);
   const {
     $from,
     $to
-  } = state.selection;
+  } = normalizedSelection;
   const tr = liftBlock(state.tr, $from, $to);
   if (dispatch && tr) {
     dispatch(tr);
@@ -16,7 +18,17 @@ export const wrapSelectionInTaskList = (state, dispatch) => {
   const {
     $from,
     $to
-  } = state.selection;
+  } = normalizeTaskItemsSelection(state.selection);
+
+  // limit ui indentation to 6 levels
+  const {
+    taskList,
+    taskItem
+  } = state.schema.nodes;
+  const maxDepth = subtreeHeight($from, $to, [taskList, taskItem]);
+  if (maxDepth >= 6) {
+    return true;
+  }
   const blockRange = getBlockRange($from, $to);
   if (!blockRange) {
     return true;