@atlaskit/editor-plugin-block-menu 7.0.1 → 7.0.2

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # @atlaskit/editor-plugin-block-menu
 
+## 7.0.2
+
+### Patch Changes
+
+- [`1b1c6f7936019`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/1b1c6f7936019) -
+  Cleanup fg: platform_editor_block_menu_expand_localid_fix
+- Updated dependencies
+
 ## 7.0.1
 
 ### Patch Changes

package/dist/cjs/editor-commands/transform-node-utils/steps/wrapMixedContentStep.js

@@ -8,7 +8,6 @@ exports.wrapMixedContentStep = void 0;
 var _toConsumableArray2 = _interopRequireDefault(require("@babel/runtime/helpers/toConsumableArray"));
 var _utils = require("@atlaskit/editor-common/utils");
 var _model = require("@atlaskit/editor-prosemirror/model");
-var _platformFeatureFlags = require("@atlaskit/platform-feature-flags");
 var _marks = require("../marks");
 var _types = require("../types");
 var _utils2 = require("../utils");
@@ -38,12 +37,8 @@ var createTextContentContainer = function createTextContentContainer(textContent
  * Creates a regular container with node content.
  */
 var createNodeContentContainer = function createNodeContentContainer(nodeContent, targetNodeType) {
-  // [FEATURE FLAG: platform_editor_block_menu_expand_localid_fix]
-  // Pre-assigns a localId so the localId plugin's appendTransaction does not replace the node
-  // object, preserving the expandedState WeakMap entry set in transformNode.ts.
-  // To clean up: remove the if-else, always use the flag-on branch.
   var isExpandType = targetNodeType.name === 'expand' || targetNodeType.name === 'nestedExpand';
-  var nodeAttrs = isExpandType && (0, _platformFeatureFlags.fg)('platform_editor_block_menu_expand_localid_fix') ? {
+  var nodeAttrs = isExpandType ? {
     localId: crypto.randomUUID()
   } : {};
   return targetNodeType.createAndFill(nodeAttrs, nodeContent);
@@ -81,11 +76,8 @@ var handleEmptyContainerEdgeCase = function handleEmptyContainerEdgeCase(result,
     return emptyCodeBlock ? [emptyCodeBlock].concat((0, _toConsumableArray2.default)(result)) : result;
   }
   var emptyParagraph = schema.nodes.paragraph.create();
-  // [FEATURE FLAG: platform_editor_block_menu_expand_localid_fix]
-  // Same localId pre-assignment rationale as createNodeContentContainer above.
-  // To clean up: remove the if-else, always use the flag-on branch.
   var isExpandType = targetNodeTypeName === 'expand' || targetNodeTypeName === 'nestedExpand';
-  var emptyContainerAttrs = isExpandType && (0, _platformFeatureFlags.fg)('platform_editor_block_menu_expand_localid_fix') ? {
+  var emptyContainerAttrs = isExpandType ? {
     localId: crypto.randomUUID()
   } : {};
   var emptyContainer = targetNodeType.create(emptyContainerAttrs, emptyParagraph);

package/dist/cjs/editor-commands/transform-node-utils/utils.js

@@ -1,18 +1,13 @@
 "use strict";
 
-var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault");
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.isTextNode = exports.getTargetNodeTypeNameInContext = exports.getSelectedNode = exports.getBlockNodesInRange = exports.createTextContent = exports.convertTextNodeToParagraph = exports.convertNestedExpandToExpand = exports.convertExpandToNestedExpand = void 0;
-var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/defineProperty"));
 var _state = require("@atlaskit/editor-prosemirror/state");
 var _utils = require("@atlaskit/editor-prosemirror/utils");
 var _editorTables = require("@atlaskit/editor-tables");
-var _platformFeatureFlags = require("@atlaskit/platform-feature-flags");
 var _types = require("./types");
-function ownKeys(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbols) { var o = Object.getOwnPropertySymbols(e); r && (o = o.filter(function (r) { return Object.getOwnPropertyDescriptor(e, r).enumerable; })), t.push.apply(t, o); } return t; }
-function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var t = null != arguments[r] ? arguments[r] : {}; r % 2 ? ownKeys(Object(t), !0).forEach(function (r) { (0, _defineProperty2.default)(e, r, t[r]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(e, Object.getOwnPropertyDescriptors(t)) : ownKeys(Object(t)).forEach(function (r) { Object.defineProperty(e, r, Object.getOwnPropertyDescriptor(t, r)); }); } return e; }
 /**
  * Determines if a node is a text node (heading or paragraph).
  * Text nodes can have their content converted to paragraphs when they can't be wrapped directly.
@@ -84,17 +79,10 @@ var convertNestedExpandToExpand = exports.convertNestedExpandToExpand = function
   if (!expandType) {
     return null;
   }
-
-  // [FEATURE FLAG: platform_editor_block_menu_expand_localid_fix]
-  // Pre-assigns a localId so the localId plugin's appendTransaction does not replace the node
-  // object, preserving the expandedState WeakMap entry set in transformNode.ts.
-  // To clean up: remove the if-else, always include localId in attrs.
-  var localIdAttr = (0, _platformFeatureFlags.fg)('platform_editor_block_menu_expand_localid_fix') ? {
+  return expandType.createAndFill({
+    title: ((_node$attrs = node.attrs) === null || _node$attrs === void 0 ? void 0 : _node$attrs.title) || '',
     localId: crypto.randomUUID()
-  } : {};
-  return expandType.createAndFill(_objectSpread({
-    title: ((_node$attrs = node.attrs) === null || _node$attrs === void 0 ? void 0 : _node$attrs.title) || ''
-  }, localIdAttr), node.content);
+  }, node.content);
 };
 
 /**
@@ -108,16 +96,10 @@ var convertExpandToNestedExpand = exports.convertExpandToNestedExpand = function
   if (!nestedExpandType) {
     return null;
   }
-
-  // [FEATURE FLAG: platform_editor_block_menu_expand_localid_fix]
-  // Same localId pre-assignment rationale as convertNestedExpandToExpand above.
-  // To clean up: remove the if-else, always include localId in attrs.
-  var localIdAttr = (0, _platformFeatureFlags.fg)('platform_editor_block_menu_expand_localid_fix') ? {
+  return nestedExpandType.createAndFill({
+    title: ((_node$attrs2 = node.attrs) === null || _node$attrs2 === void 0 ? void 0 : _node$attrs2.title) || '',
     localId: crypto.randomUUID()
-  } : {};
-  return nestedExpandType.createAndFill(_objectSpread({
-    title: ((_node$attrs2 = node.attrs) === null || _node$attrs2 === void 0 ? void 0 : _node$attrs2.title) || ''
-  }, localIdAttr), node.content);
+  }, node.content);
 };
 
 /**

package/dist/cjs/editor-commands/transform-node-utils/wrapStep.js

@@ -5,7 +5,6 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.wrapStep = void 0;
 var _utils = require("@atlaskit/editor-common/utils");
-var _platformFeatureFlags = require("@atlaskit/platform-feature-flags");
 var _marks = require("./marks");
 var _utils2 = require("./utils");
 /**
@@ -34,14 +33,8 @@ var wrapStep = exports.wrapStep = function wrapStep(nodes, context) {
     });
   }
   var targetNodeType = schema.nodes[targetNodeTypeName];
-
-  // [FEATURE FLAG: platform_editor_block_menu_expand_localid_fix]
-  // Pre-assigns a localId so the localId plugin's appendTransaction does not replace the node
-  // object (via setNodeAttribute), which would invalidate the expandedState WeakMap entry set
-  // in transformNode.ts and cause the expand to render as collapsed.
-  // To clean up: remove the if-else, always use the flag-on branch (isExpandType with uuid).
   var isExpandType = targetNodeTypeName === 'expand' || targetNodeTypeName === 'nestedExpand';
-  var nodeAttrs = isExpandType && (0, _platformFeatureFlags.fg)('platform_editor_block_menu_expand_localid_fix') ? {
+  var nodeAttrs = isExpandType ? {
     localId: crypto.randomUUID()
   } : {};
   var sourceSupportsBreakout = _utils.breakoutResizableNodes.includes(fromNode.type.name);

package/dist/cjs/editor-commands/transformNode.js

@@ -12,7 +12,6 @@ var _selection = require("@atlaskit/editor-common/selection");
 var _state = require("@atlaskit/editor-prosemirror/state");
 var _transform = require("@atlaskit/editor-prosemirror/transform");
 var _editorTables = require("@atlaskit/editor-tables");
-var _platformFeatureFlags = require("@atlaskit/platform-feature-flags");
 var _isNestedNode = require("../ui/utils/isNestedNode");
 var _transform2 = require("./transform-node-utils/transform");
 var _utils = require("./transforms/utils");
@@ -55,23 +54,13 @@ var transformNode = exports.transformNode = function transformNode(api) {
         });
         var content = resultNodes.length > 0 ? resultNodes : sourceNodes;
         var sliceStart = isList ? $from.pos - 1 : $from.pos;
-
-        // [FEATURE FLAG: platform_editor_block_menu_expand_localid_fix]
-        // Pre-populates the expandedState WeakMap so ExpandNodeView initializes newly created
-        // expand/nestedExpand nodes as expanded rather than collapsed. Works in conjunction with
-        // the localId pre-assignment in the transform steps — without a pre-assigned localId the
-        // localId plugin's appendTransaction replaces the node object (via setNodeAttribute),
-        // invalidating this WeakMap entry. The else branch preserves the previous behaviour.
-        // To clean up: remove the if-else block and keep only the flag-on body.
-        if ((0, _platformFeatureFlags.fg)('platform_editor_block_menu_expand_localid_fix')) {
-          var expand = nodes.expand,
-            nestedExpand = nodes.nestedExpand;
-          content.forEach(function (node) {
-            if (node.type === expand || node.type === nestedExpand) {
-              _expand.expandedState.set(node, true);
-            }
-          });
-        }
+        var expand = nodes.expand,
+          nestedExpand = nodes.nestedExpand;
+        content.forEach(function (node) {
+          if (node.type === expand || node.type === nestedExpand) {
+            _expand.expandedState.set(node, true);
+          }
+        });
         if (preservedSelection instanceof _state.NodeSelection && preservedSelection.node.type === nodes.mediaSingle) {
           var _api$blockControls2;
           // when node is media single, use tr.replaceWith freeze editor, if modify position, tr.replaceWith creates duplicats

package/dist/es2019/editor-commands/transform-node-utils/steps/wrapMixedContentStep.js

@@ -1,6 +1,5 @@
 import { breakoutResizableNodes } from '@atlaskit/editor-common/utils';
 import { Fragment } from '@atlaskit/editor-prosemirror/model';
-import { fg } from '@atlaskit/platform-feature-flags';
 import { removeDisallowedMarks } from '../marks';
 import { NODE_CATEGORY_BY_TYPE } from '../types';
 import { convertTextNodeToParagraph, createTextContent, isTextNode } from '../utils';
@@ -31,12 +30,8 @@ const createTextContentContainer = (textContentArray, targetNodeType, schema) =>
  * Creates a regular container with node content.
  */
 const createNodeContentContainer = (nodeContent, targetNodeType) => {
-  // [FEATURE FLAG: platform_editor_block_menu_expand_localid_fix]
-  // Pre-assigns a localId so the localId plugin's appendTransaction does not replace the node
-  // object, preserving the expandedState WeakMap entry set in transformNode.ts.
-  // To clean up: remove the if-else, always use the flag-on branch.
   const isExpandType = targetNodeType.name === 'expand' || targetNodeType.name === 'nestedExpand';
-  const nodeAttrs = isExpandType && fg('platform_editor_block_menu_expand_localid_fix') ? {
+  const nodeAttrs = isExpandType ? {
     localId: crypto.randomUUID()
   } : {};
   return targetNodeType.createAndFill(nodeAttrs, nodeContent);
@@ -74,11 +69,8 @@ const handleEmptyContainerEdgeCase = (result, hasCreatedContainer, fromNode, tar
     return emptyCodeBlock ? [emptyCodeBlock, ...result] : result;
   }
   const emptyParagraph = schema.nodes.paragraph.create();
-  // [FEATURE FLAG: platform_editor_block_menu_expand_localid_fix]
-  // Same localId pre-assignment rationale as createNodeContentContainer above.
-  // To clean up: remove the if-else, always use the flag-on branch.
   const isExpandType = targetNodeTypeName === 'expand' || targetNodeTypeName === 'nestedExpand';
-  const emptyContainerAttrs = isExpandType && fg('platform_editor_block_menu_expand_localid_fix') ? {
+  const emptyContainerAttrs = isExpandType ? {
     localId: crypto.randomUUID()
   } : {};
   const emptyContainer = targetNodeType.create(emptyContainerAttrs, emptyParagraph);

package/dist/es2019/editor-commands/transform-node-utils/utils.js

@@ -1,7 +1,6 @@
 import { NodeSelection, TextSelection } from '@atlaskit/editor-prosemirror/state';
 import { findParentNodeOfType } from '@atlaskit/editor-prosemirror/utils';
 import { CellSelection } from '@atlaskit/editor-tables';
-import { fg } from '@atlaskit/platform-feature-flags';
 import { NODE_CATEGORY_BY_TYPE } from './types';
 
 /**
@@ -76,17 +75,9 @@ export const convertNestedExpandToExpand = (node, schema) => {
   if (!expandType) {
     return null;
   }
-
-  // [FEATURE FLAG: platform_editor_block_menu_expand_localid_fix]
-  // Pre-assigns a localId so the localId plugin's appendTransaction does not replace the node
-  // object, preserving the expandedState WeakMap entry set in transformNode.ts.
-  // To clean up: remove the if-else, always include localId in attrs.
-  const localIdAttr = fg('platform_editor_block_menu_expand_localid_fix') ? {
-    localId: crypto.randomUUID()
-  } : {};
   return expandType.createAndFill({
     title: ((_node$attrs = node.attrs) === null || _node$attrs === void 0 ? void 0 : _node$attrs.title) || '',
-    ...localIdAttr
+    localId: crypto.randomUUID()
   }, node.content);
 };
 
@@ -101,16 +92,9 @@ export const convertExpandToNestedExpand = (node, schema) => {
   if (!nestedExpandType) {
     return null;
   }
-
-  // [FEATURE FLAG: platform_editor_block_menu_expand_localid_fix]
-  // Same localId pre-assignment rationale as convertNestedExpandToExpand above.
-  // To clean up: remove the if-else, always include localId in attrs.
-  const localIdAttr = fg('platform_editor_block_menu_expand_localid_fix') ? {
-    localId: crypto.randomUUID()
-  } : {};
   return nestedExpandType.createAndFill({
     title: ((_node$attrs2 = node.attrs) === null || _node$attrs2 === void 0 ? void 0 : _node$attrs2.title) || '',
-    ...localIdAttr
+    localId: crypto.randomUUID()
   }, node.content);
 };
 

package/dist/es2019/editor-commands/transform-node-utils/wrapStep.js

@@ -1,5 +1,4 @@
 import { breakoutResizableNodes } from '@atlaskit/editor-common/utils';
-import { fg } from '@atlaskit/platform-feature-flags';
 import { removeDisallowedMarks } from './marks';
 import { convertExpandToNestedExpand } from './utils';
 
@@ -31,14 +30,8 @@ export const wrapStep = (nodes, context) => {
     });
   }
   const targetNodeType = schema.nodes[targetNodeTypeName];
-
-  // [FEATURE FLAG: platform_editor_block_menu_expand_localid_fix]
-  // Pre-assigns a localId so the localId plugin's appendTransaction does not replace the node
-  // object (via setNodeAttribute), which would invalidate the expandedState WeakMap entry set
-  // in transformNode.ts and cause the expand to render as collapsed.
-  // To clean up: remove the if-else, always use the flag-on branch (isExpandType with uuid).
   const isExpandType = targetNodeTypeName === 'expand' || targetNodeTypeName === 'nestedExpand';
-  const nodeAttrs = isExpandType && fg('platform_editor_block_menu_expand_localid_fix') ? {
+  const nodeAttrs = isExpandType ? {
     localId: crypto.randomUUID()
   } : {};
   const sourceSupportsBreakout = breakoutResizableNodes.includes(fromNode.type.name);

package/dist/es2019/editor-commands/transformNode.js

@@ -6,7 +6,6 @@ import { expandSelectionToBlockRange, getSourceNodesFromSelectionRange } from '@
 import { NodeSelection } from '@atlaskit/editor-prosemirror/state';
 import { Mapping, StepMap } from '@atlaskit/editor-prosemirror/transform';
 import { CellSelection } from '@atlaskit/editor-tables';
-import { fg } from '@atlaskit/platform-feature-flags';
 import { isNestedNode } from '../ui/utils/isNestedNode';
 import { convertNodesToTargetType } from './transform-node-utils/transform';
 import { isListNode } from './transforms/utils';
@@ -51,25 +50,15 @@ export const transformNode = api => (targetType, metadata) => ({
     });
     const content = resultNodes.length > 0 ? resultNodes : sourceNodes;
     const sliceStart = isList ? $from.pos - 1 : $from.pos;
-
-    // [FEATURE FLAG: platform_editor_block_menu_expand_localid_fix]
-    // Pre-populates the expandedState WeakMap so ExpandNodeView initializes newly created
-    // expand/nestedExpand nodes as expanded rather than collapsed. Works in conjunction with
-    // the localId pre-assignment in the transform steps — without a pre-assigned localId the
-    // localId plugin's appendTransaction replaces the node object (via setNodeAttribute),
-    // invalidating this WeakMap entry. The else branch preserves the previous behaviour.
-    // To clean up: remove the if-else block and keep only the flag-on body.
-    if (fg('platform_editor_block_menu_expand_localid_fix')) {
-      const {
-        expand,
-        nestedExpand
-      } = nodes;
-      content.forEach(node => {
-        if (node.type === expand || node.type === nestedExpand) {
-          expandedState.set(node, true);
-        }
-      });
-    }
+    const {
+      expand,
+      nestedExpand
+    } = nodes;
+    content.forEach(node => {
+      if (node.type === expand || node.type === nestedExpand) {
+        expandedState.set(node, true);
+      }
+    });
     if (preservedSelection instanceof NodeSelection && preservedSelection.node.type === nodes.mediaSingle) {
       var _api$blockControls2;
       // when node is media single, use tr.replaceWith freeze editor, if modify position, tr.replaceWith creates duplicats

package/dist/esm/editor-commands/transform-node-utils/steps/wrapMixedContentStep.js

@@ -1,7 +1,6 @@
 import _toConsumableArray from "@babel/runtime/helpers/toConsumableArray";
 import { breakoutResizableNodes } from '@atlaskit/editor-common/utils';
 import { Fragment } from '@atlaskit/editor-prosemirror/model';
-import { fg } from '@atlaskit/platform-feature-flags';
 import { removeDisallowedMarks } from '../marks';
 import { NODE_CATEGORY_BY_TYPE } from '../types';
 import { convertTextNodeToParagraph, createTextContent, isTextNode } from '../utils';
@@ -32,12 +31,8 @@ var createTextContentContainer = function createTextContentContainer(textContent
  * Creates a regular container with node content.
  */
 var createNodeContentContainer = function createNodeContentContainer(nodeContent, targetNodeType) {
-  // [FEATURE FLAG: platform_editor_block_menu_expand_localid_fix]
-  // Pre-assigns a localId so the localId plugin's appendTransaction does not replace the node
-  // object, preserving the expandedState WeakMap entry set in transformNode.ts.
-  // To clean up: remove the if-else, always use the flag-on branch.
   var isExpandType = targetNodeType.name === 'expand' || targetNodeType.name === 'nestedExpand';
-  var nodeAttrs = isExpandType && fg('platform_editor_block_menu_expand_localid_fix') ? {
+  var nodeAttrs = isExpandType ? {
     localId: crypto.randomUUID()
   } : {};
   return targetNodeType.createAndFill(nodeAttrs, nodeContent);
@@ -75,11 +70,8 @@ var handleEmptyContainerEdgeCase = function handleEmptyContainerEdgeCase(result,
     return emptyCodeBlock ? [emptyCodeBlock].concat(_toConsumableArray(result)) : result;
   }
   var emptyParagraph = schema.nodes.paragraph.create();
-  // [FEATURE FLAG: platform_editor_block_menu_expand_localid_fix]
-  // Same localId pre-assignment rationale as createNodeContentContainer above.
-  // To clean up: remove the if-else, always use the flag-on branch.
   var isExpandType = targetNodeTypeName === 'expand' || targetNodeTypeName === 'nestedExpand';
-  var emptyContainerAttrs = isExpandType && fg('platform_editor_block_menu_expand_localid_fix') ? {
+  var emptyContainerAttrs = isExpandType ? {
     localId: crypto.randomUUID()
   } : {};
   var emptyContainer = targetNodeType.create(emptyContainerAttrs, emptyParagraph);

package/dist/esm/editor-commands/transform-node-utils/utils.js

@@ -1,10 +1,6 @@
-import _defineProperty from "@babel/runtime/helpers/defineProperty";
-function ownKeys(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbols) { var o = Object.getOwnPropertySymbols(e); r && (o = o.filter(function (r) { return Object.getOwnPropertyDescriptor(e, r).enumerable; })), t.push.apply(t, o); } return t; }
-function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var t = null != arguments[r] ? arguments[r] : {}; r % 2 ? ownKeys(Object(t), !0).forEach(function (r) { _defineProperty(e, r, t[r]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(e, Object.getOwnPropertyDescriptors(t)) : ownKeys(Object(t)).forEach(function (r) { Object.defineProperty(e, r, Object.getOwnPropertyDescriptor(t, r)); }); } return e; }
 import { NodeSelection, TextSelection } from '@atlaskit/editor-prosemirror/state';
 import { findParentNodeOfType } from '@atlaskit/editor-prosemirror/utils';
 import { CellSelection } from '@atlaskit/editor-tables';
-import { fg } from '@atlaskit/platform-feature-flags';
 import { NODE_CATEGORY_BY_TYPE } from './types';
 
 /**
@@ -78,17 +74,10 @@ export var convertNestedExpandToExpand = function convertNestedExpandToExpand(no
   if (!expandType) {
     return null;
   }
-
-  // [FEATURE FLAG: platform_editor_block_menu_expand_localid_fix]
-  // Pre-assigns a localId so the localId plugin's appendTransaction does not replace the node
-  // object, preserving the expandedState WeakMap entry set in transformNode.ts.
-  // To clean up: remove the if-else, always include localId in attrs.
-  var localIdAttr = fg('platform_editor_block_menu_expand_localid_fix') ? {
+  return expandType.createAndFill({
+    title: ((_node$attrs = node.attrs) === null || _node$attrs === void 0 ? void 0 : _node$attrs.title) || '',
     localId: crypto.randomUUID()
-  } : {};
-  return expandType.createAndFill(_objectSpread({
-    title: ((_node$attrs = node.attrs) === null || _node$attrs === void 0 ? void 0 : _node$attrs.title) || ''
-  }, localIdAttr), node.content);
+  }, node.content);
 };
 
 /**
@@ -102,16 +91,10 @@ export var convertExpandToNestedExpand = function convertExpandToNestedExpand(no
   if (!nestedExpandType) {
     return null;
   }
-
-  // [FEATURE FLAG: platform_editor_block_menu_expand_localid_fix]
-  // Same localId pre-assignment rationale as convertNestedExpandToExpand above.
-  // To clean up: remove the if-else, always include localId in attrs.
-  var localIdAttr = fg('platform_editor_block_menu_expand_localid_fix') ? {
+  return nestedExpandType.createAndFill({
+    title: ((_node$attrs2 = node.attrs) === null || _node$attrs2 === void 0 ? void 0 : _node$attrs2.title) || '',
     localId: crypto.randomUUID()
-  } : {};
-  return nestedExpandType.createAndFill(_objectSpread({
-    title: ((_node$attrs2 = node.attrs) === null || _node$attrs2 === void 0 ? void 0 : _node$attrs2.title) || ''
-  }, localIdAttr), node.content);
+  }, node.content);
 };
 
 /**

package/dist/esm/editor-commands/transform-node-utils/wrapStep.js

@@ -1,5 +1,4 @@
 import { breakoutResizableNodes } from '@atlaskit/editor-common/utils';
-import { fg } from '@atlaskit/platform-feature-flags';
 import { removeDisallowedMarks } from './marks';
 import { convertExpandToNestedExpand } from './utils';
 
@@ -29,14 +28,8 @@ export var wrapStep = function wrapStep(nodes, context) {
     });
   }
   var targetNodeType = schema.nodes[targetNodeTypeName];
-
-  // [FEATURE FLAG: platform_editor_block_menu_expand_localid_fix]
-  // Pre-assigns a localId so the localId plugin's appendTransaction does not replace the node
-  // object (via setNodeAttribute), which would invalidate the expandedState WeakMap entry set
-  // in transformNode.ts and cause the expand to render as collapsed.
-  // To clean up: remove the if-else, always use the flag-on branch (isExpandType with uuid).
   var isExpandType = targetNodeTypeName === 'expand' || targetNodeTypeName === 'nestedExpand';
-  var nodeAttrs = isExpandType && fg('platform_editor_block_menu_expand_localid_fix') ? {
+  var nodeAttrs = isExpandType ? {
     localId: crypto.randomUUID()
   } : {};
   var sourceSupportsBreakout = breakoutResizableNodes.includes(fromNode.type.name);

package/dist/esm/editor-commands/transformNode.js

@@ -6,7 +6,6 @@ import { expandSelectionToBlockRange, getSourceNodesFromSelectionRange } from '@
 import { NodeSelection } from '@atlaskit/editor-prosemirror/state';
 import { Mapping, StepMap } from '@atlaskit/editor-prosemirror/transform';
 import { CellSelection } from '@atlaskit/editor-tables';
-import { fg } from '@atlaskit/platform-feature-flags';
 import { isNestedNode } from '../ui/utils/isNestedNode';
 import { convertNodesToTargetType } from './transform-node-utils/transform';
 import { isListNode } from './transforms/utils';
@@ -49,23 +48,13 @@ export var transformNode = function transformNode(api) {
         });
         var content = resultNodes.length > 0 ? resultNodes : sourceNodes;
         var sliceStart = isList ? $from.pos - 1 : $from.pos;
-
-        // [FEATURE FLAG: platform_editor_block_menu_expand_localid_fix]
-        // Pre-populates the expandedState WeakMap so ExpandNodeView initializes newly created
-        // expand/nestedExpand nodes as expanded rather than collapsed. Works in conjunction with
-        // the localId pre-assignment in the transform steps — without a pre-assigned localId the
-        // localId plugin's appendTransaction replaces the node object (via setNodeAttribute),
-        // invalidating this WeakMap entry. The else branch preserves the previous behaviour.
-        // To clean up: remove the if-else block and keep only the flag-on body.
-        if (fg('platform_editor_block_menu_expand_localid_fix')) {
-          var expand = nodes.expand,
-            nestedExpand = nodes.nestedExpand;
-          content.forEach(function (node) {
-            if (node.type === expand || node.type === nestedExpand) {
-              expandedState.set(node, true);
-            }
-          });
-        }
+        var expand = nodes.expand,
+          nestedExpand = nodes.nestedExpand;
+        content.forEach(function (node) {
+          if (node.type === expand || node.type === nestedExpand) {
+            expandedState.set(node, true);
+          }
+        });
         if (preservedSelection instanceof NodeSelection && preservedSelection.node.type === nodes.mediaSingle) {
           var _api$blockControls2;
           // when node is media single, use tr.replaceWith freeze editor, if modify position, tr.replaceWith creates duplicats

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@atlaskit/editor-plugin-block-menu",
-	"version": "7.0.1",
+	"version": "7.0.2",
 	"description": "BlockMenu plugin for @atlaskit/editor-core",
 	"author": "Atlassian Pty Ltd",
 	"license": "Apache-2.0",
@@ -50,7 +50,7 @@
 		"bind-event-listener": "^3.0.0"
 	},
 	"peerDependencies": {
-		"@atlaskit/editor-common": "^112.0.0",
+		"@atlaskit/editor-common": "^112.1.0",
 		"react": "^18.2.0",
 		"react-intl-next": "npm:react-intl@^5.18.1"
 	},
@@ -97,9 +97,6 @@
 		"platform_editor_block_menu_copy_section": {
 			"type": "boolean"
 		},
-		"platform_editor_block_menu_expand_localid_fix": {
-			"type": "boolean"
-		},
 		"platform_editor_block_menu_v2_patch_2": {
 			"type": "boolean"
 		},