@atlaskit/renderer 109.19.0 → 109.19.2

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # @atlaskit/renderer
 
+## 109.19.2
+
+### Patch Changes
+
+- [#91468](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/pull-requests/91468) [`4ad8a0bc6051`](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/commits/4ad8a0bc6051) - Fixes preview linking and annotation flow on media with annotation
+
+## 109.19.1
+
+### Patch Changes
+
+- [#91808](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/pull-requests/91808) [`00e5dcfdf240`](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/commits/00e5dcfdf240) - [ux] Does not add comment button on hover on media that is inside comment or inline comment rendereres
+
 ## 109.19.0
 
 ### Minor Changes

package/dist/cjs/react/nodes/media/index.js

@@ -5,7 +5,7 @@ var _typeof = require("@babel/runtime/helpers/typeof");
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
-exports.default = void 0;
+exports.linkStyle = exports.default = exports.borderStyle = void 0;
 var _classCallCheck2 = _interopRequireDefault(require("@babel/runtime/helpers/classCallCheck"));
 var _createClass2 = _interopRequireDefault(require("@babel/runtime/helpers/createClass"));
 var _assertThisInitialized2 = _interopRequireDefault(require("@babel/runtime/helpers/assertThisInitialized"));
@@ -16,6 +16,7 @@ var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/de
 var _extends2 = _interopRequireDefault(require("@babel/runtime/helpers/extends"));
 var _slicedToArray2 = _interopRequireDefault(require("@babel/runtime/helpers/slicedToArray"));
 var _objectWithoutProperties2 = _interopRequireDefault(require("@babel/runtime/helpers/objectWithoutProperties"));
+var _taggedTemplateLiteral2 = _interopRequireDefault(require("@babel/runtime/helpers/taggedTemplateLiteral"));
 var _react = _interopRequireWildcard(require("react"));
 var _react2 = require("@emotion/react");
 var _analyticsNext = require("@atlaskit/analytics-next");
@@ -30,12 +31,12 @@ var _analytics = require("@atlaskit/editor-common/analytics");
 var _events = require("../../../analytics/events");
 var _annotation = _interopRequireDefault(require("../../marks/annotation"));
 var _context = require("../../../ui/annotations/context");
-var _styles = require("./styles");
 var _mediaSingle = require("@atlaskit/editor-common/media-single");
 var _reactIntlNext = require("react-intl-next");
 var _hooks = require("../../../ui/annotations/hooks");
 var _types = require("@atlaskit/editor-common/types");
-var _excluded = ["marks", "mediaElement"];
+var _excluded = ["marks", "mediaElement", "isDrafting"];
+var _templateObject, _templateObject2;
 /** @jsx jsx */
 function _getRequireWildcardCache(e) { if ("function" != typeof WeakMap) return null; var r = new WeakMap(), t = new WeakMap(); return (_getRequireWildcardCache = function _getRequireWildcardCache(e) { return e ? t : r; })(e); }
 function _interopRequireWildcard(e, r) { if (!r && e && e.__esModule) return e; if (null === e || "object" != _typeof(e) && "function" != typeof e) return { default: e }; var t = _getRequireWildcardCache(r); if (t && t.has(e)) return t.get(e); var n = { __proto__: null }, a = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var u in e) if ("default" !== u && Object.prototype.hasOwnProperty.call(e, u)) { var i = a ? Object.getOwnPropertyDescriptor(e, u) : null; i && (i.get || i.set) ? Object.defineProperty(n, u, i) : n[u] = e[u]; } return n.default = e, t && t.set(e, n), n; }
@@ -43,6 +44,10 @@ function ownKeys(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbol
 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; }
 function _createSuper(Derived) { var hasNativeReflectConstruct = _isNativeReflectConstruct(); return function _createSuperInternal() { var Super = (0, _getPrototypeOf2.default)(Derived), result; if (hasNativeReflectConstruct) { var NewTarget = (0, _getPrototypeOf2.default)(this).constructor; result = Reflect.construct(Super, arguments, NewTarget); } else { result = Super.apply(this, arguments); } return (0, _possibleConstructorReturn2.default)(this, result); }; }
 function _isNativeReflectConstruct() { try { var t = !Boolean.prototype.valueOf.call(Reflect.construct(Boolean, [], function () {})); } catch (t) {} return (_isNativeReflectConstruct = function _isNativeReflectConstruct() { return !!t; })(); }
+var linkStyle = exports.linkStyle = (0, _react2.css)(_templateObject || (_templateObject = (0, _taggedTemplateLiteral2.default)(["\n  position: absolute;\n  background: transparent;\n  top: 0;\n  right: 0;\n  bottom: 0;\n  left: 0;\n  cursor: pointer;\n  width: 100% !important;\n  height: 100% !important;\n"])));
+var borderStyle = exports.borderStyle = function borderStyle(color, width) {
+  return (0, _react2.css)(_templateObject2 || (_templateObject2 = (0, _taggedTemplateLiteral2.default)(["\n  position: absolute;\n  width: 100% !important;\n  height: 100% !important;\n  border-radius: ", "px;\n  box-shadow: 0 0 0 ", "px ", ";\n"])), width, width, color);
+};
 var MediaBorder = function MediaBorder(_ref) {
   var _mark$attrs$color, _mark$attrs$size;
   var mark = _ref.mark,
@@ -57,7 +62,7 @@ var MediaBorder = function MediaBorder(_ref) {
     "data-mark-type": "border",
     "data-color": borderColor,
     "data-size": borderWidth,
-    css: (0, _styles.borderStyle)(paletteColorValue, borderWidth)
+    css: borderStyle(paletteColorValue, borderWidth)
   }, (0, _react2.jsx)(_ui.MediaBorderGapFiller, {
     borderColor: borderColor
   }), children);
@@ -75,7 +80,7 @@ var MediaLink = function MediaLink(_ref2) {
     rel: "noreferrer noopener",
     onClick: onClick,
     "data-block-link": linkHref,
-    css: _styles.linkStyle
+    css: linkStyle
   }, children);
 };
 var MediaAnnotation = function MediaAnnotation(_ref3) {
@@ -88,7 +93,8 @@ var MediaAnnotation = function MediaAnnotation(_ref3) {
     id: mark.attrs.id,
     annotationType: mark.attrs.annotationType,
     dataAttributes: {
-      'data-renderer-mark': true
+      'data-renderer-mark': true,
+      'data-block-mark': true
     }
     // This should be fine being empty [] since the serializer serializeFragmentChild getMarkProps call always passes
     ,
@@ -121,6 +127,8 @@ var CommentBadgeWrapper = function CommentBadgeWrapper(_ref5) {
   var _marks$map;
   var marks = _ref5.marks,
     mediaElement = _ref5.mediaElement,
+    _ref5$isDrafting = _ref5.isDrafting,
+    isDrafting = _ref5$isDrafting === void 0 ? false : _ref5$isDrafting,
     rest = (0, _objectWithoutProperties2.default)(_ref5, _excluded);
   var _useState = (0, _react.useState)('default'),
     _useState2 = (0, _slicedToArray2.default)(_useState, 2),
@@ -160,10 +168,11 @@ var CommentBadgeWrapper = function CommentBadgeWrapper(_ref5) {
       observer.disconnect();
     };
   }, [mediaElement, setStatus]);
-  if (!activeParentIds.length) {
+  if (!isDrafting && !activeParentIds.length) {
     return null;
   }
   var onClick = function onClick(e) {
+    e.preventDefault();
     if (updateSubscriber) {
       updateSubscriber.emit(_types.AnnotationUpdateEvent.ON_ANNOTATION_CLICK, {
         annotationIds: activeParentIds,
@@ -201,7 +210,9 @@ var Media = /*#__PURE__*/function (_PureComponent) {
         ssr = _this$props.ssr,
         width = _this$props.width,
         height = _this$props.height,
-        mediaSingleElement = _this$props.mediaSingleElement;
+        mediaSingleElement = _this$props.mediaSingleElement,
+        _this$props$isDraftin = _this$props.isDrafting,
+        isDrafting = _this$props$isDraftin === void 0 ? false : _this$props$isDraftin;
       var annotationMarks = _this.props.isAnnotationMark ? _this.props.marks.filter(_this.props.isAnnotationMark) : undefined;
       var borderMark = _this.props.marks.find(_this.props.isBorderMark);
       var linkMark = _this.props.marks.find(_this.props.isLinkMark);
@@ -223,7 +234,8 @@ var Media = /*#__PURE__*/function (_PureComponent) {
         marks: annotationMarks,
         mediaElement: mediaSingleElement,
         width: width,
-        height: height
+        height: height,
+        isDrafting: isDrafting
       }), (0, _react2.jsx)(_MediaCard.MediaCard, (0, _extends2.default)({
         contextIdentifierProvider: contextIdentifierProvider
       }, _this.props, {
@@ -311,7 +323,8 @@ var MediaWithDraftAnnotation = function MediaWithDraftAnnotation(props) {
     }) : dataAttributes;
   }, [applyDraftAnnotation, dataAttributes]);
   return (0, _react2.jsx)(Media, (0, _extends2.default)({}, props, {
-    dataAttributes: dataAttributesWithDraftAnnotation
+    dataAttributes: dataAttributesWithDraftAnnotation,
+    isDrafting: shouldApplyDraftAnnotation
   }));
 };
 var _default = exports.default = MediaWithDraftAnnotation;

package/dist/cjs/react/nodes/mediaSingle/index.js

@@ -114,8 +114,9 @@ var MediaSingleWithChildren = function MediaSingleWithChildren(props) {
 
   // TODO: put appearance-based padding into theme instead
 
-  var padding = rendererAppearance === 'full-page' ? _style.FullPagePadding * 2 : 0;
+  var isFullPage = rendererAppearance === 'full-page';
   var isFullWidth = rendererAppearance === 'full-width';
+  var padding = isFullPage ? _style.FullPagePadding * 2 : 0;
   var calcDimensions = (0, _react.useCallback)(function (mediaContainerWidth) {
     var containerWidth = getMediaContainerWidth(mediaContainerWidth, layout);
     var maxWidth = containerWidth;
@@ -152,6 +153,7 @@ var MediaSingleWithChildren = function MediaSingleWithChildren(props) {
     cancelTimeout = _useAnnotationHoverDi.cancelTimeout,
     initiateTimeout = _useAnnotationHoverDi.initiateTimeout,
     setIsWithinRange = _useAnnotationHoverDi.setIsWithinRange;
+  var isFullPageRenderer = isFullPage || isFullWidth;
   (0, _react.useEffect)(function () {
     var mediaSingleElement = ref.current;
     var handleMouseEnter = function handleMouseEnter(event) {
@@ -164,17 +166,17 @@ var MediaSingleWithChildren = function MediaSingleWithChildren(props) {
     var handleMouseLeave = function handleMouseLeave() {
       initiateTimeout();
     };
-    if (mediaSingleElement) {
+    if (mediaSingleElement && isFullPageRenderer) {
       mediaSingleElement.addEventListener('mouseenter', handleMouseEnter);
       mediaSingleElement.addEventListener('mouseleave', handleMouseLeave);
     }
     return function () {
-      if (mediaSingleElement) {
+      if (mediaSingleElement && isFullPageRenderer) {
         mediaSingleElement.removeEventListener('mouseenter', handleMouseEnter);
         mediaSingleElement.removeEventListener('mouseleave', handleMouseLeave);
       }
     };
-  }, [setHoverTarget, cancelTimeout, initiateTimeout, setIsWithinRange]);
+  }, [setHoverTarget, isFullPageRenderer, cancelTimeout, initiateTimeout, setIsWithinRange]);
 
   // Note: in SSR mode the `window` object is not defined,
   // therefore width here is 0, see:

package/dist/cjs/ui/Renderer/index.js

@@ -55,7 +55,7 @@ function _isNativeReflectConstruct() { try { var t = !Boolean.prototype.valueOf.
 var NORMAL_SEVERITY_THRESHOLD = exports.NORMAL_SEVERITY_THRESHOLD = 2000;
 var DEGRADED_SEVERITY_THRESHOLD = exports.DEGRADED_SEVERITY_THRESHOLD = 3000;
 var packageName = "@atlaskit/renderer";
-var packageVersion = "109.19.0";
+var packageVersion = "109.19.2";
 var Renderer = exports.Renderer = /*#__PURE__*/function (_PureComponent) {
   (0, _inherits2.default)(Renderer, _PureComponent);
   var _super = _createSuper(Renderer);

package/dist/cjs/ui/annotations/element/mark.js

@@ -34,6 +34,11 @@ var MarkComponent = exports.MarkComponent = function MarkComponent(_ref) {
     return (0, _toConsumableArray2.default)(new Set([].concat((0, _toConsumableArray2.default)(annotationParentIds), [id])));
   }, [id, annotationParentIds]);
   var onMarkClick = (0, _react.useCallback)(function (event) {
+    // prevent inline mark logic for media block marks
+    if (event.currentTarget.getAttribute('data-block-mark')) {
+      return;
+    }
+
     // prevents multiple callback on overlapping annotations
     if (event.defaultPrevented || state !== _adfSchema.AnnotationMarkStates.ACTIVE) {
       return;

package/dist/es2019/react/nodes/media/index.js

@@ -3,7 +3,7 @@ import _extends from "@babel/runtime/helpers/extends";
 /** @jsx jsx */
 
 import React, { PureComponent, Fragment, useEffect, useState, useMemo } from 'react';
-import { jsx } from '@emotion/react';
+import { jsx, css } from '@emotion/react';
 import { AnalyticsContext } from '@atlaskit/analytics-next';
 import { MEDIA_CONTEXT } from '@atlaskit/analytics-namespaced-context';
 import { WithProviders } from '@atlaskit/editor-common/provider-factory';
@@ -16,11 +16,28 @@ import { ACTION, ACTION_SUBJECT, ACTION_SUBJECT_ID, EVENT_TYPE } from '@atlaskit
 import { MODE, PLATFORM } from '../../../analytics/events';
 import AnnotationComponent from '../../marks/annotation';
 import { AnnotationsDraftContext } from '../../../ui/annotations/context';
-import { linkStyle, borderStyle } from './styles';
 import { CommentBadge as CommentBadgeComponent } from '@atlaskit/editor-common/media-single';
 import { injectIntl } from 'react-intl-next';
 import { useInlineCommentSubscriberContext, useInlineCommentsFilter } from '../../../ui/annotations/hooks';
 import { AnnotationUpdateEvent } from '@atlaskit/editor-common/types';
+export const linkStyle = css`
+  position: absolute;
+  background: transparent;
+  top: 0;
+  right: 0;
+  bottom: 0;
+  left: 0;
+  cursor: pointer;
+  width: 100% !important;
+  height: 100% !important;
+`;
+export const borderStyle = (color, width) => css`
+  position: absolute;
+  width: 100% !important;
+  height: 100% !important;
+  border-radius: ${width}px;
+  box-shadow: 0 0 0 ${width}px ${color};
+`;
 const MediaBorder = ({
   mark,
   children
@@ -69,7 +86,8 @@ const MediaAnnotation = ({
     id: mark.attrs.id,
     annotationType: mark.attrs.annotationType,
     dataAttributes: {
-      'data-renderer-mark': true
+      'data-renderer-mark': true,
+      'data-block-mark': true
     }
     // This should be fine being empty [] since the serializer serializeFragmentChild getMarkProps call always passes
     ,
@@ -101,6 +119,7 @@ const CommentBadge = injectIntl(CommentBadgeComponent);
 const CommentBadgeWrapper = ({
   marks,
   mediaElement,
+  isDrafting = false,
   ...rest
 }) => {
   var _marks$map;
@@ -134,10 +153,11 @@ const CommentBadgeWrapper = ({
       observer.disconnect();
     };
   }, [mediaElement, setStatus]);
-  if (!activeParentIds.length) {
+  if (!isDrafting && !activeParentIds.length) {
     return null;
   }
   const onClick = e => {
+    e.preventDefault();
     if (updateSubscriber) {
       updateSubscriber.emit(AnnotationUpdateEvent.ON_ANNOTATION_CLICK, {
         annotationIds: activeParentIds,
@@ -168,7 +188,8 @@ class Media extends PureComponent {
         ssr,
         width,
         height,
-        mediaSingleElement
+        mediaSingleElement,
+        isDrafting = false
       } = this.props;
       const annotationMarks = this.props.isAnnotationMark ? this.props.marks.filter(this.props.isAnnotationMark) : undefined;
       const borderMark = this.props.marks.find(this.props.isBorderMark);
@@ -193,7 +214,8 @@ class Media extends PureComponent {
         marks: annotationMarks,
         mediaElement: mediaSingleElement,
         width: width,
-        height: height
+        height: height,
+        isDrafting: isDrafting
       }), jsx(MediaCard, _extends({
         contextIdentifierProvider: contextIdentifierProvider
       }, this.props, {
@@ -274,7 +296,8 @@ const MediaWithDraftAnnotation = props => {
     'data-renderer-mark': true
   } : dataAttributes, [applyDraftAnnotation, dataAttributes]);
   return jsx(Media, _extends({}, props, {
-    dataAttributes: dataAttributesWithDraftAnnotation
+    dataAttributes: dataAttributesWithDraftAnnotation,
+    isDrafting: shouldApplyDraftAnnotation
   }));
 };
 export default MediaWithDraftAnnotation;

package/dist/es2019/react/nodes/mediaSingle/index.js

@@ -101,8 +101,9 @@ const MediaSingleWithChildren = props => {
 
   // TODO: put appearance-based padding into theme instead
 
-  const padding = rendererAppearance === 'full-page' ? FullPagePadding * 2 : 0;
+  const isFullPage = rendererAppearance === 'full-page';
   const isFullWidth = rendererAppearance === 'full-width';
+  const padding = isFullPage ? FullPagePadding * 2 : 0;
   const calcDimensions = useCallback(mediaContainerWidth => {
     const containerWidth = getMediaContainerWidth(mediaContainerWidth, layout);
     const maxWidth = containerWidth;
@@ -139,6 +140,7 @@ const MediaSingleWithChildren = props => {
     initiateTimeout,
     setIsWithinRange
   } = useAnnotationHoverDispatch();
+  const isFullPageRenderer = isFullPage || isFullWidth;
   useEffect(() => {
     const mediaSingleElement = ref.current;
     const handleMouseEnter = event => {
@@ -151,17 +153,17 @@ const MediaSingleWithChildren = props => {
     const handleMouseLeave = () => {
       initiateTimeout();
     };
-    if (mediaSingleElement) {
+    if (mediaSingleElement && isFullPageRenderer) {
       mediaSingleElement.addEventListener('mouseenter', handleMouseEnter);
       mediaSingleElement.addEventListener('mouseleave', handleMouseLeave);
     }
     return () => {
-      if (mediaSingleElement) {
+      if (mediaSingleElement && isFullPageRenderer) {
         mediaSingleElement.removeEventListener('mouseenter', handleMouseEnter);
         mediaSingleElement.removeEventListener('mouseleave', handleMouseLeave);
       }
     };
-  }, [setHoverTarget, cancelTimeout, initiateTimeout, setIsWithinRange]);
+  }, [setHoverTarget, isFullPageRenderer, cancelTimeout, initiateTimeout, setIsWithinRange]);
 
   // Note: in SSR mode the `window` object is not defined,
   // therefore width here is 0, see:

package/dist/es2019/ui/Renderer/index.js

@@ -36,7 +36,7 @@ import { EditorMediaClientProvider } from '../../react/utils/EditorMediaClientPr
 export const NORMAL_SEVERITY_THRESHOLD = 2000;
 export const DEGRADED_SEVERITY_THRESHOLD = 3000;
 const packageName = "@atlaskit/renderer";
-const packageVersion = "109.19.0";
+const packageVersion = "109.19.2";
 export class Renderer extends PureComponent {
   constructor(props) {
     super(props);

package/dist/es2019/ui/annotations/element/mark.js

@@ -33,6 +33,11 @@ export const MarkComponent = ({
 }) => {
   const annotationIds = useMemo(() => [...new Set([...annotationParentIds, id])], [id, annotationParentIds]);
   const onMarkClick = useCallback(event => {
+    // prevent inline mark logic for media block marks
+    if (event.currentTarget.getAttribute('data-block-mark')) {
+      return;
+    }
+
     // prevents multiple callback on overlapping annotations
     if (event.defaultPrevented || state !== AnnotationMarkStates.ACTIVE) {
       return;

package/dist/esm/react/nodes/media/index.js

@@ -8,7 +8,9 @@ import _defineProperty from "@babel/runtime/helpers/defineProperty";
 import _extends from "@babel/runtime/helpers/extends";
 import _slicedToArray from "@babel/runtime/helpers/slicedToArray";
 import _objectWithoutProperties from "@babel/runtime/helpers/objectWithoutProperties";
-var _excluded = ["marks", "mediaElement"];
+import _taggedTemplateLiteral from "@babel/runtime/helpers/taggedTemplateLiteral";
+var _excluded = ["marks", "mediaElement", "isDrafting"];
+var _templateObject, _templateObject2;
 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; }
 function _createSuper(Derived) { var hasNativeReflectConstruct = _isNativeReflectConstruct(); return function _createSuperInternal() { var Super = _getPrototypeOf(Derived), result; if (hasNativeReflectConstruct) { var NewTarget = _getPrototypeOf(this).constructor; result = Reflect.construct(Super, arguments, NewTarget); } else { result = Super.apply(this, arguments); } return _possibleConstructorReturn(this, result); }; }
@@ -16,7 +18,7 @@ function _isNativeReflectConstruct() { try { var t = !Boolean.prototype.valueOf.
 /** @jsx jsx */
 
 import React, { PureComponent, Fragment, useEffect, useState, useMemo } from 'react';
-import { jsx } from '@emotion/react';
+import { jsx, css } from '@emotion/react';
 import { AnalyticsContext } from '@atlaskit/analytics-next';
 import { MEDIA_CONTEXT } from '@atlaskit/analytics-namespaced-context';
 import { WithProviders } from '@atlaskit/editor-common/provider-factory';
@@ -29,11 +31,14 @@ import { ACTION, ACTION_SUBJECT, ACTION_SUBJECT_ID, EVENT_TYPE } from '@atlaskit
 import { MODE, PLATFORM } from '../../../analytics/events';
 import AnnotationComponent from '../../marks/annotation';
 import { AnnotationsDraftContext } from '../../../ui/annotations/context';
-import { linkStyle, borderStyle } from './styles';
 import { CommentBadge as CommentBadgeComponent } from '@atlaskit/editor-common/media-single';
 import { injectIntl } from 'react-intl-next';
 import { useInlineCommentSubscriberContext, useInlineCommentsFilter } from '../../../ui/annotations/hooks';
 import { AnnotationUpdateEvent } from '@atlaskit/editor-common/types';
+export var linkStyle = css(_templateObject || (_templateObject = _taggedTemplateLiteral(["\n  position: absolute;\n  background: transparent;\n  top: 0;\n  right: 0;\n  bottom: 0;\n  left: 0;\n  cursor: pointer;\n  width: 100% !important;\n  height: 100% !important;\n"])));
+export var borderStyle = function borderStyle(color, width) {
+  return css(_templateObject2 || (_templateObject2 = _taggedTemplateLiteral(["\n  position: absolute;\n  width: 100% !important;\n  height: 100% !important;\n  border-radius: ", "px;\n  box-shadow: 0 0 0 ", "px ", ";\n"])), width, width, color);
+};
 var MediaBorder = function MediaBorder(_ref) {
   var _mark$attrs$color, _mark$attrs$size;
   var mark = _ref.mark,
@@ -79,7 +84,8 @@ var MediaAnnotation = function MediaAnnotation(_ref3) {
     id: mark.attrs.id,
     annotationType: mark.attrs.annotationType,
     dataAttributes: {
-      'data-renderer-mark': true
+      'data-renderer-mark': true,
+      'data-block-mark': true
     }
     // This should be fine being empty [] since the serializer serializeFragmentChild getMarkProps call always passes
     ,
@@ -112,6 +118,8 @@ var CommentBadgeWrapper = function CommentBadgeWrapper(_ref5) {
   var _marks$map;
   var marks = _ref5.marks,
     mediaElement = _ref5.mediaElement,
+    _ref5$isDrafting = _ref5.isDrafting,
+    isDrafting = _ref5$isDrafting === void 0 ? false : _ref5$isDrafting,
     rest = _objectWithoutProperties(_ref5, _excluded);
   var _useState = useState('default'),
     _useState2 = _slicedToArray(_useState, 2),
@@ -151,10 +159,11 @@ var CommentBadgeWrapper = function CommentBadgeWrapper(_ref5) {
       observer.disconnect();
     };
   }, [mediaElement, setStatus]);
-  if (!activeParentIds.length) {
+  if (!isDrafting && !activeParentIds.length) {
     return null;
   }
   var onClick = function onClick(e) {
+    e.preventDefault();
     if (updateSubscriber) {
       updateSubscriber.emit(AnnotationUpdateEvent.ON_ANNOTATION_CLICK, {
         annotationIds: activeParentIds,
@@ -192,7 +201,9 @@ var Media = /*#__PURE__*/function (_PureComponent) {
         ssr = _this$props.ssr,
         width = _this$props.width,
         height = _this$props.height,
-        mediaSingleElement = _this$props.mediaSingleElement;
+        mediaSingleElement = _this$props.mediaSingleElement,
+        _this$props$isDraftin = _this$props.isDrafting,
+        isDrafting = _this$props$isDraftin === void 0 ? false : _this$props$isDraftin;
       var annotationMarks = _this.props.isAnnotationMark ? _this.props.marks.filter(_this.props.isAnnotationMark) : undefined;
       var borderMark = _this.props.marks.find(_this.props.isBorderMark);
       var linkMark = _this.props.marks.find(_this.props.isLinkMark);
@@ -214,7 +225,8 @@ var Media = /*#__PURE__*/function (_PureComponent) {
         marks: annotationMarks,
         mediaElement: mediaSingleElement,
         width: width,
-        height: height
+        height: height,
+        isDrafting: isDrafting
       }), jsx(MediaCard, _extends({
         contextIdentifierProvider: contextIdentifierProvider
       }, _this.props, {
@@ -302,7 +314,8 @@ var MediaWithDraftAnnotation = function MediaWithDraftAnnotation(props) {
     }) : dataAttributes;
   }, [applyDraftAnnotation, dataAttributes]);
   return jsx(Media, _extends({}, props, {
-    dataAttributes: dataAttributesWithDraftAnnotation
+    dataAttributes: dataAttributesWithDraftAnnotation,
+    isDrafting: shouldApplyDraftAnnotation
   }));
 };
 export default MediaWithDraftAnnotation;

package/dist/esm/react/nodes/mediaSingle/index.js

@@ -104,8 +104,9 @@ var MediaSingleWithChildren = function MediaSingleWithChildren(props) {
 
   // TODO: put appearance-based padding into theme instead
 
-  var padding = rendererAppearance === 'full-page' ? FullPagePadding * 2 : 0;
+  var isFullPage = rendererAppearance === 'full-page';
   var isFullWidth = rendererAppearance === 'full-width';
+  var padding = isFullPage ? FullPagePadding * 2 : 0;
   var calcDimensions = useCallback(function (mediaContainerWidth) {
     var containerWidth = getMediaContainerWidth(mediaContainerWidth, layout);
     var maxWidth = containerWidth;
@@ -142,6 +143,7 @@ var MediaSingleWithChildren = function MediaSingleWithChildren(props) {
     cancelTimeout = _useAnnotationHoverDi.cancelTimeout,
     initiateTimeout = _useAnnotationHoverDi.initiateTimeout,
     setIsWithinRange = _useAnnotationHoverDi.setIsWithinRange;
+  var isFullPageRenderer = isFullPage || isFullWidth;
   useEffect(function () {
     var mediaSingleElement = ref.current;
     var handleMouseEnter = function handleMouseEnter(event) {
@@ -154,17 +156,17 @@ var MediaSingleWithChildren = function MediaSingleWithChildren(props) {
     var handleMouseLeave = function handleMouseLeave() {
       initiateTimeout();
     };
-    if (mediaSingleElement) {
+    if (mediaSingleElement && isFullPageRenderer) {
       mediaSingleElement.addEventListener('mouseenter', handleMouseEnter);
       mediaSingleElement.addEventListener('mouseleave', handleMouseLeave);
     }
     return function () {
-      if (mediaSingleElement) {
+      if (mediaSingleElement && isFullPageRenderer) {
         mediaSingleElement.removeEventListener('mouseenter', handleMouseEnter);
         mediaSingleElement.removeEventListener('mouseleave', handleMouseLeave);
       }
     };
-  }, [setHoverTarget, cancelTimeout, initiateTimeout, setIsWithinRange]);
+  }, [setHoverTarget, isFullPageRenderer, cancelTimeout, initiateTimeout, setIsWithinRange]);
 
   // Note: in SSR mode the `window` object is not defined,
   // therefore width here is 0, see:

package/dist/esm/ui/Renderer/index.js

@@ -46,7 +46,7 @@ import { EditorMediaClientProvider } from '../../react/utils/EditorMediaClientPr
 export var NORMAL_SEVERITY_THRESHOLD = 2000;
 export var DEGRADED_SEVERITY_THRESHOLD = 3000;
 var packageName = "@atlaskit/renderer";
-var packageVersion = "109.19.0";
+var packageVersion = "109.19.2";
 export var Renderer = /*#__PURE__*/function (_PureComponent) {
   _inherits(Renderer, _PureComponent);
   var _super = _createSuper(Renderer);

package/dist/esm/ui/annotations/element/mark.js

@@ -29,6 +29,11 @@ export var MarkComponent = function MarkComponent(_ref) {
     return _toConsumableArray(new Set([].concat(_toConsumableArray(annotationParentIds), [id])));
   }, [id, annotationParentIds]);
   var onMarkClick = useCallback(function (event) {
+    // prevent inline mark logic for media block marks
+    if (event.currentTarget.getAttribute('data-block-mark')) {
+      return;
+    }
+
     // prevents multiple callback on overlapping annotations
     if (event.defaultPrevented || state !== AnnotationMarkStates.ACTIVE) {
       return;

package/dist/types/react/nodes/media/index.d.ts

@@ -25,6 +25,9 @@ export type MediaProps = MediaCardProps & {
     mediaSingleElement?: HTMLElement | null;
     width?: number;
     height?: number;
+    isDrafting: boolean;
 };
+export declare const linkStyle: import("@emotion/react").SerializedStyles;
+export declare const borderStyle: (color: string, width: number) => import("@emotion/react").SerializedStyles;
 declare const MediaWithDraftAnnotation: (props: PropsWithChildren<MediaProps>) => jsx.JSX.Element;
 export default MediaWithDraftAnnotation;

package/dist/types/react/types.d.ts

@@ -41,6 +41,7 @@ export interface NodeMeta {
 export interface MarkMeta {
     dataAttributes: {
         'data-renderer-mark': true;
+        'data-block-mark'?: true;
     };
     eventHandlers?: EventHandlers;
     fireAnalyticsEvent?: (event: AnalyticsEventPayload) => void;

package/dist/types-ts4.5/react/nodes/media/index.d.ts

@@ -25,6 +25,9 @@ export type MediaProps = MediaCardProps & {
     mediaSingleElement?: HTMLElement | null;
     width?: number;
     height?: number;
+    isDrafting: boolean;
 };
+export declare const linkStyle: import("@emotion/react").SerializedStyles;
+export declare const borderStyle: (color: string, width: number) => import("@emotion/react").SerializedStyles;
 declare const MediaWithDraftAnnotation: (props: PropsWithChildren<MediaProps>) => jsx.JSX.Element;
 export default MediaWithDraftAnnotation;

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

@@ -41,6 +41,7 @@ export interface NodeMeta {
 export interface MarkMeta {
     dataAttributes: {
         'data-renderer-mark': true;
+        'data-block-mark'?: true;
     };
     eventHandlers?: EventHandlers;
     fireAnalyticsEvent?: (event: AnalyticsEventPayload) => void;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@atlaskit/renderer",
-  "version": "109.19.0",
+  "version": "109.19.2",
   "description": "Renderer component",
   "publishConfig": {
     "registry": "https://registry.npmjs.org/"

package/dist/cjs/react/nodes/media/styles.js

@@ -1,14 +0,0 @@
-"use strict";
-
-var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault");
-Object.defineProperty(exports, "__esModule", {
-  value: true
-});
-exports.linkStyle = exports.borderStyle = void 0;
-var _taggedTemplateLiteral2 = _interopRequireDefault(require("@babel/runtime/helpers/taggedTemplateLiteral"));
-var _react = require("@emotion/react");
-var _templateObject, _templateObject2;
-var linkStyle = exports.linkStyle = (0, _react.css)(_templateObject || (_templateObject = (0, _taggedTemplateLiteral2.default)(["\n  position: absolute;\n  background: transparent;\n  top: 0;\n  right: 0;\n  bottom: 0;\n  left: 0;\n  cursor: pointer;\n  width: 100% !important;\n  height: 100% !important;\n"])));
-var borderStyle = exports.borderStyle = function borderStyle(color, width) {
-  return (0, _react.css)(_templateObject2 || (_templateObject2 = (0, _taggedTemplateLiteral2.default)(["\n  position: absolute;\n  width: 100% !important;\n  height: 100% !important;\n  border-radius: ", "px;\n  box-shadow: 0 0 0 ", "px ", ";\n"])), width, width, color);
-};

package/dist/es2019/react/nodes/media/styles.js

@@ -1,19 +0,0 @@
-import { css } from '@emotion/react';
-export const linkStyle = css`
-  position: absolute;
-  background: transparent;
-  top: 0;
-  right: 0;
-  bottom: 0;
-  left: 0;
-  cursor: pointer;
-  width: 100% !important;
-  height: 100% !important;
-`;
-export const borderStyle = (color, width) => css`
-  position: absolute;
-  width: 100% !important;
-  height: 100% !important;
-  border-radius: ${width}px;
-  box-shadow: 0 0 0 ${width}px ${color};
-`;

package/dist/esm/react/nodes/media/styles.js

@@ -1,7 +0,0 @@
-import _taggedTemplateLiteral from "@babel/runtime/helpers/taggedTemplateLiteral";
-var _templateObject, _templateObject2;
-import { css } from '@emotion/react';
-export var linkStyle = css(_templateObject || (_templateObject = _taggedTemplateLiteral(["\n  position: absolute;\n  background: transparent;\n  top: 0;\n  right: 0;\n  bottom: 0;\n  left: 0;\n  cursor: pointer;\n  width: 100% !important;\n  height: 100% !important;\n"])));
-export var borderStyle = function borderStyle(color, width) {
-  return css(_templateObject2 || (_templateObject2 = _taggedTemplateLiteral(["\n  position: absolute;\n  width: 100% !important;\n  height: 100% !important;\n  border-radius: ", "px;\n  box-shadow: 0 0 0 ", "px ", ";\n"])), width, width, color);
-};

package/dist/types/react/nodes/media/styles.d.ts

@@ -1,2 +0,0 @@
-export declare const linkStyle: import("@emotion/react").SerializedStyles;
-export declare const borderStyle: (color: string, width: number) => import("@emotion/react").SerializedStyles;

package/dist/types-ts4.5/react/nodes/media/styles.d.ts

@@ -1,2 +0,0 @@
-export declare const linkStyle: import("@emotion/react").SerializedStyles;
-export declare const borderStyle: (color: string, width: number) => import("@emotion/react").SerializedStyles;