@atlaskit/editor-plugin-annotation 1.23.2 → 1.23.3

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # @atlaskit/editor-plugin-annotation
 
+## 1.23.3
+
+### Patch Changes
+
+- [#158723](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/pull-requests/158723)
+  [`8ff5b01b0d37f`](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/commits/8ff5b01b0d37f) -
+  [ED-24682] Cleanup feature flag for commenting on media in the renderer
+
+  **@atlaskit/editor-plugin-media**: Add optional checks for the api in Comment Badges on media
+
+- Updated dependencies
+
 ## 1.23.2
 
 ### Patch Changes

package/dist/cjs/commands/index.js

@@ -112,10 +112,10 @@ var removeInlineCommentNearSelection = exports.removeInlineCommentNearSelection
     return true;
   };
 };
-var getDraftCommandAction = function getDraftCommandAction(drafting, targetType, targetNodeId, isCommentOnMediaOn, supportedBlockNodes) {
+var getDraftCommandAction = function getDraftCommandAction(drafting, targetType, targetNodeId, supportedBlockNodes) {
   return function (editorState) {
     // validate selection only when entering draft mode
-    if (drafting && (0, _utils.isSelectionValid)(editorState, isCommentOnMediaOn) !== _types2.AnnotationSelectionType.VALID) {
+    if (drafting && (0, _utils.isSelectionValid)(editorState) !== _types2.AnnotationSelectionType.VALID) {
       return false;
     }
     return {
@@ -124,7 +124,6 @@ var getDraftCommandAction = function getDraftCommandAction(drafting, targetType,
         drafting: drafting,
         editorState: editorState,
         targetType: targetType,
-        isCommentOnMediaOn: isCommentOnMediaOn,
         supportedBlockNodes: supportedBlockNodes,
         targetNodeId: targetNodeId
       }
@@ -176,9 +175,8 @@ var setInlineCommentDraftState = exports.setInlineCommentDraftState = function s
   return function (drafting) {
     var inputMethod = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : _analytics.INPUT_METHOD.TOOLBAR;
     var targetType = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : 'inline';
-    var isCommentOnMediaOn = arguments.length > 3 && arguments[3] !== undefined ? arguments[3] : false;
-    var targetNodeId = arguments.length > 4 && arguments[4] !== undefined ? arguments[4] : undefined;
-    var commandAction = getDraftCommandAction(drafting, targetType, targetNodeId, isCommentOnMediaOn, supportedBlockNodes);
+    var targetNodeId = arguments.length > 3 && arguments[3] !== undefined ? arguments[3] : undefined;
+    var commandAction = getDraftCommandAction(drafting, targetType, targetNodeId, supportedBlockNodes);
     return (0, _pluginFactory.createCommand)(commandAction, _transform.default.handleDraftState(editorAnalyticsAPI)(drafting, inputMethod));
   };
 };

package/dist/cjs/plugin.js

@@ -92,7 +92,6 @@ var annotationPlugin = exports.annotationPlugin = function annotationPlugin(_ref
             state: state,
             intl: intl,
             isToolbarAbove: isToolbarAbove,
-            isCommentOnMediaOn: featureFlags === null || featureFlags === void 0 ? void 0 : featureFlags.commentsOnMedia,
             api: api
           });
         }

package/dist/cjs/pm-plugins/plugin-factory.js

@@ -55,7 +55,6 @@ var shouldClearBookMarkCheck = exports.shouldClearBookMarkCheck = function shoul
 };
 var getSelectionChangedHandler = function getSelectionChangedHandler(reopenCommentView) {
   return function (tr, pluginState) {
-    var _pluginState$featureF;
     if (pluginState.skipSelectionHandling) {
       return _objectSpread(_objectSpread({}, pluginState), {}, {
         skipSelectionHandling: false
@@ -63,7 +62,7 @@ var getSelectionChangedHandler = function getSelectionChangedHandler(reopenComme
         isInlineCommentViewClosed: false
       });
     }
-    if ((_pluginState$featureF = pluginState.featureFlagsPluginState) !== null && _pluginState$featureF !== void 0 && _pluginState$featureF.commentsOnMedia &&
+    if (
     // If pluginState.selectedAnnotations is annotations of block node, i.e. when a new comment is created,
     // we keep it as it is so that we can show comment view component with the newly created comment
     (0, _utils2.isBlockNodeAnnotationsSelected)(tr.selection, pluginState.selectedAnnotations)) {
@@ -99,28 +98,25 @@ var _pluginFactory = (0, _utils.pluginFactory)(_utils2.inlineCommentPluginKey, _
     onDocChanged: handleDocChanged,
     mapping: function mapping(tr, pluginState, editorState) {
       var draftDecorationSet = pluginState.draftDecorationSet,
-        bookmark = pluginState.bookmark,
-        featureFlagsPluginState = pluginState.featureFlagsPluginState;
+        bookmark = pluginState.bookmark;
       var mappedDecorationSet = _view.DecorationSet.empty,
         mappedBookmark;
       var hasMappedDecorations = false;
       if (draftDecorationSet) {
         mappedDecorationSet = draftDecorationSet.map(tr.mapping, tr.doc);
       }
-      if (featureFlagsPluginState !== null && featureFlagsPluginState !== void 0 && featureFlagsPluginState.commentsOnMedia) {
-        hasMappedDecorations = mappedDecorationSet.find(undefined, undefined, function (spec) {
-          return Object.values(_utils2.decorationKey).includes(spec.key);
-        }).length > 0;
+      hasMappedDecorations = mappedDecorationSet.find(undefined, undefined, function (spec) {
+        return Object.values(_utils2.decorationKey).includes(spec.key);
+      }).length > 0;
 
-        // When changes to decoration target make decoration invalid (e.g. delete text, add mark to node),
-        // we need to reset bookmark to hide create component and to avoid invalid draft being published
-        // We only perform this change when document selection has changed.
-        if (!hasMappedDecorations && shouldClearBookMarkCheck(tr, editorState, bookmark)) {
-          return _objectSpread(_objectSpread({}, pluginState), {}, {
-            draftDecorationSet: mappedDecorationSet,
-            bookmark: undefined
-          });
-        }
+      // When changes to decoration target make decoration invalid (e.g. delete text, add mark to node),
+      // we need to reset bookmark to hide create component and to avoid invalid draft being published
+      // We only perform this change when document selection has changed.
+      if (!hasMappedDecorations && shouldClearBookMarkCheck(tr, editorState, bookmark)) {
+        return _objectSpread(_objectSpread({}, pluginState), {}, {
+          draftDecorationSet: mappedDecorationSet,
+          bookmark: undefined
+        });
       }
       if (bookmark) {
         mappedBookmark = bookmark.map(tr.mapping);

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

@@ -24,7 +24,7 @@ var _default = exports.default = function _default(pluginState, action) {
         mouseData: mouseData
       });
     case _types.ACTIONS.SET_INLINE_COMMENT_DRAFT_STATE:
-      return getNewDraftState(pluginState, action.data.drafting, action.data.editorState, action.data.targetType, action.data.isCommentOnMediaOn, action.data.supportedBlockNodes, action.data.targetNodeId);
+      return getNewDraftState(pluginState, action.data.drafting, action.data.editorState, action.data.targetType, action.data.supportedBlockNodes, action.data.targetNodeId);
     case _types.ACTIONS.INLINE_COMMENT_CLEAR_DIRTY_MARK:
       return _objectSpread(_objectSpread({}, pluginState), {}, {
         dirtyAnnotations: false,
@@ -62,7 +62,7 @@ var _default = exports.default = function _default(pluginState, action) {
       return pluginState;
   }
 };
-function getNewDraftState(pluginState, drafting, editorState, targetType, isCommentOnMediaOn, supportedBlockNodes, targetNodeId) {
+function getNewDraftState(pluginState, drafting, editorState, targetType, supportedBlockNodes, targetNodeId) {
   var draftDecorationSet = pluginState.draftDecorationSet;
   if (!draftDecorationSet || !drafting) {
     draftDecorationSet = _view.DecorationSet.empty;
@@ -75,9 +75,9 @@ function getNewDraftState(pluginState, drafting, editorState, targetType, isComm
   newState.bookmark = undefined;
   if (drafting && editorState) {
     newState.bookmark = editorState.selection.getBookmark();
-    var _ref = isCommentOnMediaOn ? (0, _utils.resolveDraftBookmark)(editorState, newState.bookmark, supportedBlockNodes) : newState.bookmark.resolve(editorState.doc),
-      from = _ref.from,
-      to = _ref.to;
+    var _resolveDraftBookmark = (0, _utils.resolveDraftBookmark)(editorState, newState.bookmark, supportedBlockNodes),
+      from = _resolveDraftBookmark.from,
+      to = _resolveDraftBookmark.to;
     var draftDecoration = (0, _utils.addDraftDecoration)(from, to, targetType);
     newState.draftDecorationSet = draftDecorationSet.add(editorState.doc, [draftDecoration]);
   }

package/dist/cjs/toolbar.js

@@ -55,13 +55,12 @@ var buildToolbar = exports.buildToolbar = function buildToolbar(editorAnalyticsA
       intl = _ref2.intl,
       _ref2$isToolbarAbove = _ref2.isToolbarAbove,
       isToolbarAbove = _ref2$isToolbarAbove === void 0 ? false : _ref2$isToolbarAbove,
-      isCommentOnMediaOn = _ref2.isCommentOnMediaOn,
       _ref2$_supportedNodes = _ref2._supportedNodes,
       _supportedNodes = _ref2$_supportedNodes === void 0 ? [] : _ref2$_supportedNodes,
       api = _ref2.api;
     var schema = state.schema;
-    var selectionValid = (0, _utils2.isSelectionValid)(state, isCommentOnMediaOn);
-    var isMediaSelected = isCommentOnMediaOn && (0, _mediaSingle.currentMediaNodeWithPos)(state);
+    var selectionValid = (0, _utils2.isSelectionValid)(state);
+    var isMediaSelected = (0, _mediaSingle.currentMediaNodeWithPos)(state);
 
     // comments on media can only be added via media floating toolbar
     if (isMediaSelected || selectionValid === _types.AnnotationSelectionType.INVALID) {

package/dist/cjs/utils.js

@@ -315,15 +315,15 @@ var getDraftCommandAnalyticsPayload = exports.getDraftCommandAnalyticsPayload =
   };
   return payload;
 };
-var isSelectionValid = exports.isSelectionValid = function isSelectionValid(state, isCommentOnMediaOn) {
+var isSelectionValid = exports.isSelectionValid = function isSelectionValid(state) {
   var _currentMediaNodeWith;
-  var _supportedNodes = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : [];
+  var _supportedNodes = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : [];
   var selection = state.selection;
   var _ref4 = getPluginState(state) || {},
     disallowOnWhitespace = _ref4.disallowOnWhitespace;
 
   // Allow media so that it can enter draft mode
-  if (isCommentOnMediaOn && (_currentMediaNodeWith = (0, _mediaSingle.currentMediaNodeWithPos)(state)) !== null && _currentMediaNodeWith !== void 0 && _currentMediaNodeWith.node) {
+  if ((_currentMediaNodeWith = (0, _mediaSingle.currentMediaNodeWithPos)(state)) !== null && _currentMediaNodeWith !== void 0 && _currentMediaNodeWith.node) {
     return _types.AnnotationSelectionType.VALID;
   }
   if (selection.empty || !(selection instanceof _state.TextSelection || selection instanceof _state.AllSelection)) {

package/dist/es2019/commands/index.js

@@ -94,10 +94,10 @@ export const removeInlineCommentNearSelection = (id, supportedNodes = []) => (st
   }
   return true;
 };
-const getDraftCommandAction = (drafting, targetType, targetNodeId, isCommentOnMediaOn, supportedBlockNodes) => {
+const getDraftCommandAction = (drafting, targetType, targetNodeId, supportedBlockNodes) => {
   return editorState => {
     // validate selection only when entering draft mode
-    if (drafting && isSelectionValid(editorState, isCommentOnMediaOn) !== AnnotationSelectionType.VALID) {
+    if (drafting && isSelectionValid(editorState) !== AnnotationSelectionType.VALID) {
       return false;
     }
     return {
@@ -106,7 +106,6 @@ const getDraftCommandAction = (drafting, targetType, targetNodeId, isCommentOnMe
         drafting,
         editorState,
         targetType,
-        isCommentOnMediaOn,
         supportedBlockNodes,
         targetNodeId
       }
@@ -146,8 +145,8 @@ export const showInlineCommentForBlockNode = (supportedBlockNodes = []) => (node
   }
   return false;
 };
-export const setInlineCommentDraftState = (editorAnalyticsAPI, supportedBlockNodes = []) => (drafting, inputMethod = INPUT_METHOD.TOOLBAR, targetType = 'inline', isCommentOnMediaOn = false, targetNodeId = undefined) => {
-  const commandAction = getDraftCommandAction(drafting, targetType, targetNodeId, isCommentOnMediaOn, supportedBlockNodes);
+export const setInlineCommentDraftState = (editorAnalyticsAPI, supportedBlockNodes = []) => (drafting, inputMethod = INPUT_METHOD.TOOLBAR, targetType = 'inline', targetNodeId = undefined) => {
+  const commandAction = getDraftCommandAction(drafting, targetType, targetNodeId, supportedBlockNodes);
   return createCommand(commandAction, transform.handleDraftState(editorAnalyticsAPI)(drafting, inputMethod));
 };
 export const addInlineComment = (editorAnalyticsAPI, editorAPI) => (id, supportedBlockNodes) => {

package/dist/es2019/plugin.js

@@ -81,7 +81,6 @@ export const annotationPlugin = ({
             state,
             intl,
             isToolbarAbove,
-            isCommentOnMediaOn: featureFlags === null || featureFlags === void 0 ? void 0 : featureFlags.commentsOnMedia,
             api
           });
         }

package/dist/es2019/pm-plugins/plugin-factory.js

@@ -45,7 +45,6 @@ export const shouldClearBookMarkCheck = (tr, editorState, bookmark) => {
   return true;
 };
 const getSelectionChangedHandler = reopenCommentView => (tr, pluginState) => {
-  var _pluginState$featureF;
   if (pluginState.skipSelectionHandling) {
     return {
       ...pluginState,
@@ -55,7 +54,7 @@ const getSelectionChangedHandler = reopenCommentView => (tr, pluginState) => {
       })
     };
   }
-  if ((_pluginState$featureF = pluginState.featureFlagsPluginState) !== null && _pluginState$featureF !== void 0 && _pluginState$featureF.commentsOnMedia &&
+  if (
   // If pluginState.selectedAnnotations is annotations of block node, i.e. when a new comment is created,
   // we keep it as it is so that we can show comment view component with the newly created comment
   isBlockNodeAnnotationsSelected(tr.selection, pluginState.selectedAnnotations)) {
@@ -102,8 +101,7 @@ export const {
   mapping: (tr, pluginState, editorState) => {
     let {
       draftDecorationSet,
-      bookmark,
-      featureFlagsPluginState
+      bookmark
     } = pluginState;
     let mappedDecorationSet = DecorationSet.empty,
       mappedBookmark;
@@ -111,19 +109,17 @@ export const {
     if (draftDecorationSet) {
       mappedDecorationSet = draftDecorationSet.map(tr.mapping, tr.doc);
     }
-    if (featureFlagsPluginState !== null && featureFlagsPluginState !== void 0 && featureFlagsPluginState.commentsOnMedia) {
-      hasMappedDecorations = mappedDecorationSet.find(undefined, undefined, spec => Object.values(decorationKey).includes(spec.key)).length > 0;
+    hasMappedDecorations = mappedDecorationSet.find(undefined, undefined, spec => Object.values(decorationKey).includes(spec.key)).length > 0;
 
-      // When changes to decoration target make decoration invalid (e.g. delete text, add mark to node),
-      // we need to reset bookmark to hide create component and to avoid invalid draft being published
-      // We only perform this change when document selection has changed.
-      if (!hasMappedDecorations && shouldClearBookMarkCheck(tr, editorState, bookmark)) {
-        return {
-          ...pluginState,
-          draftDecorationSet: mappedDecorationSet,
-          bookmark: undefined
-        };
-      }
+    // When changes to decoration target make decoration invalid (e.g. delete text, add mark to node),
+    // we need to reset bookmark to hide create component and to avoid invalid draft being published
+    // We only perform this change when document selection has changed.
+    if (!hasMappedDecorations && shouldClearBookMarkCheck(tr, editorState, bookmark)) {
+      return {
+        ...pluginState,
+        draftDecorationSet: mappedDecorationSet,
+        bookmark: undefined
+      };
     }
     if (bookmark) {
       mappedBookmark = bookmark.map(tr.mapping);

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

@@ -18,7 +18,7 @@ export default ((pluginState, action) => {
         mouseData
       };
     case ACTIONS.SET_INLINE_COMMENT_DRAFT_STATE:
-      return getNewDraftState(pluginState, action.data.drafting, action.data.editorState, action.data.targetType, action.data.isCommentOnMediaOn, action.data.supportedBlockNodes, action.data.targetNodeId);
+      return getNewDraftState(pluginState, action.data.drafting, action.data.editorState, action.data.targetType, action.data.supportedBlockNodes, action.data.targetNodeId);
     case ACTIONS.INLINE_COMMENT_CLEAR_DIRTY_MARK:
       return {
         ...pluginState,
@@ -66,7 +66,7 @@ export default ((pluginState, action) => {
       return pluginState;
   }
 });
-function getNewDraftState(pluginState, drafting, editorState, targetType, isCommentOnMediaOn, supportedBlockNodes, targetNodeId) {
+function getNewDraftState(pluginState, drafting, editorState, targetType, supportedBlockNodes, targetNodeId) {
   let {
     draftDecorationSet
   } = pluginState;
@@ -85,7 +85,7 @@ function getNewDraftState(pluginState, drafting, editorState, targetType, isComm
     const {
       from,
       to
-    } = isCommentOnMediaOn ? resolveDraftBookmark(editorState, newState.bookmark, supportedBlockNodes) : newState.bookmark.resolve(editorState.doc);
+    } = resolveDraftBookmark(editorState, newState.bookmark, supportedBlockNodes);
     const draftDecoration = addDraftDecoration(from, to, targetType);
     newState.draftDecorationSet = draftDecorationSet.add(editorState.doc, [draftDecoration]);
   }

package/dist/es2019/toolbar.js

@@ -45,15 +45,14 @@ export const buildToolbar = editorAnalyticsAPI => ({
   state,
   intl,
   isToolbarAbove = false,
-  isCommentOnMediaOn,
   _supportedNodes = [],
   api
 }) => {
   const {
     schema
   } = state;
-  const selectionValid = isSelectionValid(state, isCommentOnMediaOn);
-  const isMediaSelected = isCommentOnMediaOn && currentMediaNodeWithPos(state);
+  const selectionValid = isSelectionValid(state);
+  const isMediaSelected = currentMediaNodeWithPos(state);
 
   // comments on media can only be added via media floating toolbar
   if (isMediaSelected || selectionValid === AnnotationSelectionType.INVALID) {

package/dist/es2019/utils.js

@@ -278,7 +278,7 @@ export const getDraftCommandAnalyticsPayload = (drafting, inputMethod) => {
   };
   return payload;
 };
-export const isSelectionValid = (state, isCommentOnMediaOn, _supportedNodes = []) => {
+export const isSelectionValid = (state, _supportedNodes = []) => {
   var _currentMediaNodeWith;
   const {
     selection
@@ -288,7 +288,7 @@ export const isSelectionValid = (state, isCommentOnMediaOn, _supportedNodes = []
   } = getPluginState(state) || {};
 
   // Allow media so that it can enter draft mode
-  if (isCommentOnMediaOn && (_currentMediaNodeWith = currentMediaNodeWithPos(state)) !== null && _currentMediaNodeWith !== void 0 && _currentMediaNodeWith.node) {
+  if ((_currentMediaNodeWith = currentMediaNodeWithPos(state)) !== null && _currentMediaNodeWith !== void 0 && _currentMediaNodeWith.node) {
     return AnnotationSelectionType.VALID;
   }
   if (selection.empty || !(selection instanceof TextSelection || selection instanceof AllSelection)) {

package/dist/esm/commands/index.js

@@ -105,10 +105,10 @@ export var removeInlineCommentNearSelection = function removeInlineCommentNearSe
     return true;
   };
 };
-var getDraftCommandAction = function getDraftCommandAction(drafting, targetType, targetNodeId, isCommentOnMediaOn, supportedBlockNodes) {
+var getDraftCommandAction = function getDraftCommandAction(drafting, targetType, targetNodeId, supportedBlockNodes) {
   return function (editorState) {
     // validate selection only when entering draft mode
-    if (drafting && isSelectionValid(editorState, isCommentOnMediaOn) !== AnnotationSelectionType.VALID) {
+    if (drafting && isSelectionValid(editorState) !== AnnotationSelectionType.VALID) {
       return false;
     }
     return {
@@ -117,7 +117,6 @@ var getDraftCommandAction = function getDraftCommandAction(drafting, targetType,
         drafting: drafting,
         editorState: editorState,
         targetType: targetType,
-        isCommentOnMediaOn: isCommentOnMediaOn,
         supportedBlockNodes: supportedBlockNodes,
         targetNodeId: targetNodeId
       }
@@ -169,9 +168,8 @@ export var setInlineCommentDraftState = function setInlineCommentDraftState(edit
   return function (drafting) {
     var inputMethod = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : INPUT_METHOD.TOOLBAR;
     var targetType = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : 'inline';
-    var isCommentOnMediaOn = arguments.length > 3 && arguments[3] !== undefined ? arguments[3] : false;
-    var targetNodeId = arguments.length > 4 && arguments[4] !== undefined ? arguments[4] : undefined;
-    var commandAction = getDraftCommandAction(drafting, targetType, targetNodeId, isCommentOnMediaOn, supportedBlockNodes);
+    var targetNodeId = arguments.length > 3 && arguments[3] !== undefined ? arguments[3] : undefined;
+    var commandAction = getDraftCommandAction(drafting, targetType, targetNodeId, supportedBlockNodes);
     return createCommand(commandAction, transform.handleDraftState(editorAnalyticsAPI)(drafting, inputMethod));
   };
 };

package/dist/esm/plugin.js

@@ -79,7 +79,6 @@ export var annotationPlugin = function annotationPlugin(_ref) {
             state: state,
             intl: intl,
             isToolbarAbove: isToolbarAbove,
-            isCommentOnMediaOn: featureFlags === null || featureFlags === void 0 ? void 0 : featureFlags.commentsOnMedia,
             api: api
           });
         }

package/dist/esm/pm-plugins/plugin-factory.js

@@ -48,7 +48,6 @@ export var shouldClearBookMarkCheck = function shouldClearBookMarkCheck(tr, edit
 };
 var getSelectionChangedHandler = function getSelectionChangedHandler(reopenCommentView) {
   return function (tr, pluginState) {
-    var _pluginState$featureF;
     if (pluginState.skipSelectionHandling) {
       return _objectSpread(_objectSpread({}, pluginState), {}, {
         skipSelectionHandling: false
@@ -56,7 +55,7 @@ var getSelectionChangedHandler = function getSelectionChangedHandler(reopenComme
         isInlineCommentViewClosed: false
       });
     }
-    if ((_pluginState$featureF = pluginState.featureFlagsPluginState) !== null && _pluginState$featureF !== void 0 && _pluginState$featureF.commentsOnMedia &&
+    if (
     // If pluginState.selectedAnnotations is annotations of block node, i.e. when a new comment is created,
     // we keep it as it is so that we can show comment view component with the newly created comment
     isBlockNodeAnnotationsSelected(tr.selection, pluginState.selectedAnnotations)) {
@@ -92,28 +91,25 @@ var _pluginFactory = pluginFactory(inlineCommentPluginKey, reducer, {
     onDocChanged: handleDocChanged,
     mapping: function mapping(tr, pluginState, editorState) {
       var draftDecorationSet = pluginState.draftDecorationSet,
-        bookmark = pluginState.bookmark,
-        featureFlagsPluginState = pluginState.featureFlagsPluginState;
+        bookmark = pluginState.bookmark;
       var mappedDecorationSet = DecorationSet.empty,
         mappedBookmark;
       var hasMappedDecorations = false;
       if (draftDecorationSet) {
         mappedDecorationSet = draftDecorationSet.map(tr.mapping, tr.doc);
       }
-      if (featureFlagsPluginState !== null && featureFlagsPluginState !== void 0 && featureFlagsPluginState.commentsOnMedia) {
-        hasMappedDecorations = mappedDecorationSet.find(undefined, undefined, function (spec) {
-          return Object.values(decorationKey).includes(spec.key);
-        }).length > 0;
+      hasMappedDecorations = mappedDecorationSet.find(undefined, undefined, function (spec) {
+        return Object.values(decorationKey).includes(spec.key);
+      }).length > 0;
 
-        // When changes to decoration target make decoration invalid (e.g. delete text, add mark to node),
-        // we need to reset bookmark to hide create component and to avoid invalid draft being published
-        // We only perform this change when document selection has changed.
-        if (!hasMappedDecorations && shouldClearBookMarkCheck(tr, editorState, bookmark)) {
-          return _objectSpread(_objectSpread({}, pluginState), {}, {
-            draftDecorationSet: mappedDecorationSet,
-            bookmark: undefined
-          });
-        }
+      // When changes to decoration target make decoration invalid (e.g. delete text, add mark to node),
+      // we need to reset bookmark to hide create component and to avoid invalid draft being published
+      // We only perform this change when document selection has changed.
+      if (!hasMappedDecorations && shouldClearBookMarkCheck(tr, editorState, bookmark)) {
+        return _objectSpread(_objectSpread({}, pluginState), {}, {
+          draftDecorationSet: mappedDecorationSet,
+          bookmark: undefined
+        });
       }
       if (bookmark) {
         mappedBookmark = bookmark.map(tr.mapping);

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

@@ -17,7 +17,7 @@ export default (function (pluginState, action) {
         mouseData: mouseData
       });
     case ACTIONS.SET_INLINE_COMMENT_DRAFT_STATE:
-      return getNewDraftState(pluginState, action.data.drafting, action.data.editorState, action.data.targetType, action.data.isCommentOnMediaOn, action.data.supportedBlockNodes, action.data.targetNodeId);
+      return getNewDraftState(pluginState, action.data.drafting, action.data.editorState, action.data.targetType, action.data.supportedBlockNodes, action.data.targetNodeId);
     case ACTIONS.INLINE_COMMENT_CLEAR_DIRTY_MARK:
       return _objectSpread(_objectSpread({}, pluginState), {}, {
         dirtyAnnotations: false,
@@ -55,7 +55,7 @@ export default (function (pluginState, action) {
       return pluginState;
   }
 });
-function getNewDraftState(pluginState, drafting, editorState, targetType, isCommentOnMediaOn, supportedBlockNodes, targetNodeId) {
+function getNewDraftState(pluginState, drafting, editorState, targetType, supportedBlockNodes, targetNodeId) {
   var draftDecorationSet = pluginState.draftDecorationSet;
   if (!draftDecorationSet || !drafting) {
     draftDecorationSet = DecorationSet.empty;
@@ -68,9 +68,9 @@ function getNewDraftState(pluginState, drafting, editorState, targetType, isComm
   newState.bookmark = undefined;
   if (drafting && editorState) {
     newState.bookmark = editorState.selection.getBookmark();
-    var _ref = isCommentOnMediaOn ? resolveDraftBookmark(editorState, newState.bookmark, supportedBlockNodes) : newState.bookmark.resolve(editorState.doc),
-      from = _ref.from,
-      to = _ref.to;
+    var _resolveDraftBookmark = resolveDraftBookmark(editorState, newState.bookmark, supportedBlockNodes),
+      from = _resolveDraftBookmark.from,
+      to = _resolveDraftBookmark.to;
     var draftDecoration = addDraftDecoration(from, to, targetType);
     newState.draftDecorationSet = draftDecorationSet.add(editorState.doc, [draftDecoration]);
   }

package/dist/esm/toolbar.js

@@ -48,13 +48,12 @@ export var buildToolbar = function buildToolbar(editorAnalyticsAPI) {
       intl = _ref2.intl,
       _ref2$isToolbarAbove = _ref2.isToolbarAbove,
       isToolbarAbove = _ref2$isToolbarAbove === void 0 ? false : _ref2$isToolbarAbove,
-      isCommentOnMediaOn = _ref2.isCommentOnMediaOn,
       _ref2$_supportedNodes = _ref2._supportedNodes,
       _supportedNodes = _ref2$_supportedNodes === void 0 ? [] : _ref2$_supportedNodes,
       api = _ref2.api;
     var schema = state.schema;
-    var selectionValid = isSelectionValid(state, isCommentOnMediaOn);
-    var isMediaSelected = isCommentOnMediaOn && currentMediaNodeWithPos(state);
+    var selectionValid = isSelectionValid(state);
+    var isMediaSelected = currentMediaNodeWithPos(state);
 
     // comments on media can only be added via media floating toolbar
     if (isMediaSelected || selectionValid === AnnotationSelectionType.INVALID) {

package/dist/esm/utils.js

@@ -288,15 +288,15 @@ export var getDraftCommandAnalyticsPayload = function getDraftCommandAnalyticsPa
   };
   return payload;
 };
-export var isSelectionValid = function isSelectionValid(state, isCommentOnMediaOn) {
+export var isSelectionValid = function isSelectionValid(state) {
   var _currentMediaNodeWith;
-  var _supportedNodes = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : [];
+  var _supportedNodes = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : [];
   var selection = state.selection;
   var _ref4 = getPluginState(state) || {},
     disallowOnWhitespace = _ref4.disallowOnWhitespace;
 
   // Allow media so that it can enter draft mode
-  if (isCommentOnMediaOn && (_currentMediaNodeWith = currentMediaNodeWithPos(state)) !== null && _currentMediaNodeWith !== void 0 && _currentMediaNodeWith.node) {
+  if ((_currentMediaNodeWith = currentMediaNodeWithPos(state)) !== null && _currentMediaNodeWith !== void 0 && _currentMediaNodeWith.node) {
     return AnnotationSelectionType.VALID;
   }
   if (selection.empty || !(selection instanceof TextSelection || selection instanceof AllSelection)) {

package/dist/types/commands/index.d.ts

@@ -14,7 +14,7 @@ export declare const removeInlineCommentNearSelection: (id: string, supportedNod
  * return false if the node has no comments or no unresolved comments.
  */
 export declare const showInlineCommentForBlockNode: (supportedBlockNodes?: string[]) => (node: PMNode | null, viewMethod?: VIEW_METHOD) => Command;
-export declare const setInlineCommentDraftState: (editorAnalyticsAPI: EditorAnalyticsAPI | undefined, supportedBlockNodes?: string[]) => (drafting: boolean, inputMethod?: InlineCommentInputMethod, targetType?: TargetType, isCommentOnMediaOn?: boolean, targetNodeId?: string | undefined) => Command;
+export declare const setInlineCommentDraftState: (editorAnalyticsAPI: EditorAnalyticsAPI | undefined, supportedBlockNodes?: string[]) => (drafting: boolean, inputMethod?: InlineCommentInputMethod, targetType?: TargetType, targetNodeId?: string | undefined) => Command;
 export declare const addInlineComment: (editorAnalyticsAPI: EditorAnalyticsAPI | undefined, editorAPI?: ExtractInjectionAPI<AnnotationPlugin> | undefined) => (id: string, supportedBlockNodes?: string[]) => Command;
 export declare const updateMouseState: (mouseData: InlineCommentMouseData) => Command;
 export declare const setSelectedAnnotation: (id: string) => Command;

package/dist/types/commands/transform.d.ts

@@ -112,7 +112,7 @@ declare const _default: {
         }, import("@atlaskit/editor-plugin-engagement-platform").EngagementPlatformPluginConfig>>];
         actions: {
             stripNonExistingAnnotations: (slice: import("prosemirror-model").Slice, state: EditorState) => boolean | undefined;
-            setInlineCommentDraftState: (drafting: boolean, inputMethod: InlineCommentInputMethod, targetType?: import("../types").TargetType | undefined, isCommentOnMediaOn?: boolean | undefined, targetNodeId?: string | undefined) => import("@atlaskit/editor-common/types").Command;
+            setInlineCommentDraftState: (drafting: boolean, inputMethod: InlineCommentInputMethod, targetType?: import("../types").TargetType | undefined, targetNodeId?: string | undefined) => import("@atlaskit/editor-common/types").Command;
             showCommentForBlockNode: (node: import("prosemirror-model").Node | null, viewMethod?: import("@atlaskit/editor-common/analytics").VIEW_METHOD | undefined) => import("@atlaskit/editor-common/types").Command;
         };
     }> | undefined) => (id: string, supportedBlockNodes?: string[] | undefined) => (transaction: Transaction, state: EditorState) => Transaction;

package/dist/types/pm-plugins/types.d.ts

@@ -38,7 +38,6 @@ export type InlineCommentAction = {
         drafting: boolean;
         editorState: EditorState;
         targetType?: TargetType;
-        isCommentOnMediaOn?: boolean;
         supportedBlockNodes?: string[];
         targetNodeId?: string;
     };

package/dist/types/toolbar.d.ts

@@ -11,5 +11,5 @@ interface BuildToolbarOptions {
     _supportedNodes?: string[];
     api?: ExtractInjectionAPI<AnnotationPlugin>;
 }
-export declare const buildToolbar: (editorAnalyticsAPI: EditorAnalyticsAPI | undefined) => ({ state, intl, isToolbarAbove, isCommentOnMediaOn, _supportedNodes, api, }: BuildToolbarOptions) => FloatingToolbarConfig | undefined;
+export declare const buildToolbar: (editorAnalyticsAPI: EditorAnalyticsAPI | undefined) => ({ state, intl, isToolbarAbove, _supportedNodes, api, }: BuildToolbarOptions) => FloatingToolbarConfig | undefined;
 export {};

package/dist/types/types.d.ts

@@ -23,9 +23,7 @@ export type TargetType = 'block' | 'inline';
 export type InlineCommentInputMethod = INPUT_METHOD.TOOLBAR | INPUT_METHOD.SHORTCUT | INPUT_METHOD.FLOATING_TB;
 type SetInlineCommentDraftState = (drafting: boolean, inputMethod: InlineCommentInputMethod, 
 /** @default 'inline' */
-targetType?: TargetType, 
-/** check for confluence.frontend.fabric.editor.comments-on-media feature flag */
-isCommentOnMediaOn?: boolean, targetNodeId?: string) => Command;
+targetType?: TargetType, targetNodeId?: string) => Command;
 export type AnnotationPlugin = NextEditorPlugin<'annotation', {
     pluginConfiguration: AnnotationProviders | undefined;
     sharedState: InlineCommentPluginState | undefined;

package/dist/types/utils.d.ts

@@ -34,7 +34,7 @@ export declare const getPluginState: (state: EditorState) => InlineCommentPlugin
  * get payload for the open/close analytics event
  */
 export declare const getDraftCommandAnalyticsPayload: (drafting: boolean, inputMethod: InlineCommentInputMethod) => AnalyticsEventPayloadCallback;
-export declare const isSelectionValid: (state: EditorState, isCommentOnMediaOn?: boolean, _supportedNodes?: string[]) => AnnotationSelectionType;
+export declare const isSelectionValid: (state: EditorState, _supportedNodes?: string[]) => AnnotationSelectionType;
 export declare const hasInvalidNodes: (state: EditorState) => boolean;
 export declare const isSupportedBlockNode: (node: Node, supportedBlockNodes?: string[]) => boolean;
 /**

package/dist/types-ts4.5/commands/index.d.ts

@@ -14,7 +14,7 @@ export declare const removeInlineCommentNearSelection: (id: string, supportedNod
  * return false if the node has no comments or no unresolved comments.
  */
 export declare const showInlineCommentForBlockNode: (supportedBlockNodes?: string[]) => (node: PMNode | null, viewMethod?: VIEW_METHOD) => Command;
-export declare const setInlineCommentDraftState: (editorAnalyticsAPI: EditorAnalyticsAPI | undefined, supportedBlockNodes?: string[]) => (drafting: boolean, inputMethod?: InlineCommentInputMethod, targetType?: TargetType, isCommentOnMediaOn?: boolean, targetNodeId?: string | undefined) => Command;
+export declare const setInlineCommentDraftState: (editorAnalyticsAPI: EditorAnalyticsAPI | undefined, supportedBlockNodes?: string[]) => (drafting: boolean, inputMethod?: InlineCommentInputMethod, targetType?: TargetType, targetNodeId?: string | undefined) => Command;
 export declare const addInlineComment: (editorAnalyticsAPI: EditorAnalyticsAPI | undefined, editorAPI?: ExtractInjectionAPI<AnnotationPlugin> | undefined) => (id: string, supportedBlockNodes?: string[]) => Command;
 export declare const updateMouseState: (mouseData: InlineCommentMouseData) => Command;
 export declare const setSelectedAnnotation: (id: string) => Command;

package/dist/types-ts4.5/commands/transform.d.ts

@@ -134,7 +134,7 @@ declare const _default: {
         ];
         actions: {
             stripNonExistingAnnotations: (slice: import("prosemirror-model").Slice, state: EditorState) => boolean | undefined;
-            setInlineCommentDraftState: (drafting: boolean, inputMethod: InlineCommentInputMethod, targetType?: import("../types").TargetType | undefined, isCommentOnMediaOn?: boolean | undefined, targetNodeId?: string | undefined) => import("@atlaskit/editor-common/types").Command;
+            setInlineCommentDraftState: (drafting: boolean, inputMethod: InlineCommentInputMethod, targetType?: import("../types").TargetType | undefined, targetNodeId?: string | undefined) => import("@atlaskit/editor-common/types").Command;
             showCommentForBlockNode: (node: import("prosemirror-model").Node | null, viewMethod?: import("@atlaskit/editor-common/analytics").VIEW_METHOD | undefined) => import("@atlaskit/editor-common/types").Command;
         };
     }> | undefined) => (id: string, supportedBlockNodes?: string[] | undefined) => (transaction: Transaction, state: EditorState) => Transaction;

package/dist/types-ts4.5/pm-plugins/types.d.ts

@@ -38,7 +38,6 @@ export type InlineCommentAction = {
         drafting: boolean;
         editorState: EditorState;
         targetType?: TargetType;
-        isCommentOnMediaOn?: boolean;
         supportedBlockNodes?: string[];
         targetNodeId?: string;
     };

package/dist/types-ts4.5/toolbar.d.ts

@@ -11,5 +11,5 @@ interface BuildToolbarOptions {
     _supportedNodes?: string[];
     api?: ExtractInjectionAPI<AnnotationPlugin>;
 }
-export declare const buildToolbar: (editorAnalyticsAPI: EditorAnalyticsAPI | undefined) => ({ state, intl, isToolbarAbove, isCommentOnMediaOn, _supportedNodes, api, }: BuildToolbarOptions) => FloatingToolbarConfig | undefined;
+export declare const buildToolbar: (editorAnalyticsAPI: EditorAnalyticsAPI | undefined) => ({ state, intl, isToolbarAbove, _supportedNodes, api, }: BuildToolbarOptions) => FloatingToolbarConfig | undefined;
 export {};

package/dist/types-ts4.5/types.d.ts

@@ -23,9 +23,7 @@ export type TargetType = 'block' | 'inline';
 export type InlineCommentInputMethod = INPUT_METHOD.TOOLBAR | INPUT_METHOD.SHORTCUT | INPUT_METHOD.FLOATING_TB;
 type SetInlineCommentDraftState = (drafting: boolean, inputMethod: InlineCommentInputMethod, 
 /** @default 'inline' */
-targetType?: TargetType, 
-/** check for confluence.frontend.fabric.editor.comments-on-media feature flag */
-isCommentOnMediaOn?: boolean, targetNodeId?: string) => Command;
+targetType?: TargetType, targetNodeId?: string) => Command;
 export type AnnotationPlugin = NextEditorPlugin<'annotation', {
     pluginConfiguration: AnnotationProviders | undefined;
     sharedState: InlineCommentPluginState | undefined;

package/dist/types-ts4.5/utils.d.ts

@@ -34,7 +34,7 @@ export declare const getPluginState: (state: EditorState) => InlineCommentPlugin
  * get payload for the open/close analytics event
  */
 export declare const getDraftCommandAnalyticsPayload: (drafting: boolean, inputMethod: InlineCommentInputMethod) => AnalyticsEventPayloadCallback;
-export declare const isSelectionValid: (state: EditorState, isCommentOnMediaOn?: boolean, _supportedNodes?: string[]) => AnnotationSelectionType;
+export declare const isSelectionValid: (state: EditorState, _supportedNodes?: string[]) => AnnotationSelectionType;
 export declare const hasInvalidNodes: (state: EditorState) => boolean;
 export declare const isSupportedBlockNode: (node: Node, supportedBlockNodes?: string[]) => boolean;
 /**

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@atlaskit/editor-plugin-annotation",
-	"version": "1.23.2",
+	"version": "1.23.3",
 	"description": "Annotation plugin for @atlaskit/editor-core",
 	"author": "Atlassian Pty Ltd",
 	"license": "Apache-2.0",
@@ -41,7 +41,7 @@
 		"@atlaskit/icon": "^22.24.0",
 		"@atlaskit/onboarding": "^12.0.0",
 		"@atlaskit/platform-feature-flags": "^0.3.0",
-		"@atlaskit/tmp-editor-statsig": "^2.10.0",
+		"@atlaskit/tmp-editor-statsig": "^2.11.0",
 		"@babel/runtime": "^7.0.0"
 	},
 	"peerDependencies": {