@atlaskit/editor-plugin-show-diff 15.0.1 → 15.1.2

package/CHANGELOG.md

@@ -1,5 +1,46 @@
 # @atlaskit/editor-plugin-show-diff
 
+## 15.1.2
+
+### Patch Changes
+
+- [`acf9db40862d2`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/acf9db40862d2) -
+  Contributor tags no longer disappear for every actor when a single one cannot be named.
+  `resolveDiffContributors` now skips only the unresolvable actor — its changes keep their diff
+  colour and go untagged — instead of discarding the whole contributor list, and a user-invoked
+  agent is still credited (without the connection) when its invoking user has no profile. Behind the
+  `confluence_ncs_step_diffing_version_history` feature gate and the
+  `platform_editor_show_diff_color_scheme_refactor` experiment.
+
+## 15.1.1
+
+### Patch Changes
+
+- [`857fbb1c07b89`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/857fbb1c07b89) -
+  [ux] Show a contributor tag at the top left of changed block nodes (panel, quote, expand, media,
+  rule, block card, embed card, code block, extension, bodied extension, multi bodied extension),
+  behind `confluence_ncs_step_diffing_version_history`. One tag per block: a highlight inside a
+  changed block folds into the block's tag, and a change confined to a code block is tagged outside
+  the block so it is not clipped. Tables are excluded.
+- [`236af9e62ab32`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/236af9e62ab32) -
+  [ux] Suppress contributor tags when contributorProfiles is omitted, preserving attribution colours
+  behind confluence_ncs_step_diffing_version_history and
+  platform_editor_show_diff_color_scheme_refactor.
+- Updated dependencies
+
+## 15.1.0
+
+### Minor Changes
+
+- [`d678fef8a856c`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/d678fef8a856c) -
+  Hide pure deletions in the clean diff view behind platform_editor_diff_hide_pure_deletions
+- [`d678fef8a856c`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/d678fef8a856c) -
+  Animate the transition into the diff view behind platform_editor_diff_reveal_animation
+
+### Patch Changes
+
+- Updated dependencies
+
 ## 15.0.1
 
 ### Patch Changes

package/dist/cjs/pm-plugins/calculateDiff/calculateDiffDecorations.js

@@ -177,7 +177,8 @@ var isInsideTable = function isInsideTable(tables, pos) {
   });
 };
 var calculateNodesForBlockDecoration = function calculateNodesForBlockDecoration(_ref5) {
-  var doc = _ref5.doc,
+  var attributionKey = _ref5.attributionKey,
+    doc = _ref5.doc,
     from = _ref5.from,
     to = _ref5.to,
     colorScheme = _ref5.colorScheme,
@@ -186,11 +187,14 @@ var calculateNodesForBlockDecoration = function calculateNodesForBlockDecoration
     activeIndexPos = _ref5.activeIndexPos,
     _ref5$shouldHideDelet = _ref5.shouldHideDeleted,
     shouldHideDeleted = _ref5$shouldHideDelet === void 0 ? false : _ref5$shouldHideDelet,
+    _ref5$showContributor = _ref5.showContributorTags,
+    showContributorTags = _ref5$showContributor === void 0 ? false : _ref5$showContributor,
     _ref5$showIndicators = _ref5.showIndicators,
     showIndicators = _ref5$showIndicators === void 0 ? false : _ref5$showIndicators,
     diffType = _ref5.diffType,
     _ref5$coarseTableCell = _ref5.coarseTableCellsOnly,
-    coarseTableCellsOnly = _ref5$coarseTableCell === void 0 ? false : _ref5$coarseTableCell;
+    coarseTableCellsOnly = _ref5$coarseTableCell === void 0 ? false : _ref5$coarseTableCell,
+    tagMountContext = _ref5.tagMountContext;
   var decorations = [];
   // Coarse path: inside the large table, only cell/header overlays are kept — the
   // table/row/paragraph decorations there are redundant and cause expensive per-cell
@@ -208,6 +212,7 @@ var calculateNodesForBlockDecoration = function calculateNodesForBlockDecoration
       var nodeEnd = pos + node.nodeSize;
       var isActive = isRangeActive(activeIndexPos, pos, nodeEnd);
       decorations.push.apply(decorations, (0, _toConsumableArray2.default)((0, _createBlockChangedDecoration.createBlockChangedDecoration)({
+        attributionKey: attributionKey,
         change: {
           from: pos,
           to: nodeEnd,
@@ -217,9 +222,11 @@ var calculateNodesForBlockDecoration = function calculateNodesForBlockDecoration
         isInserted: isInserted,
         isActive: isActive,
         shouldHideDeleted: shouldHideDeleted,
+        showContributorTags: showContributorTags,
         showIndicators: showIndicators,
         doc: doc,
-        diffType: diffType
+        diffType: diffType,
+        tagMountContext: tagMountContext
       })));
     }
   });
@@ -248,30 +255,51 @@ var isDeletedContentPlacedBelow = exports.isDeletedContentPlacedBelow = function
   // Inline-level (undefined) and sentence-level changes.
   return inlineDeletedDiffPlacement === 'after';
 };
-var calculateDiffDecorationsInner = function calculateDiffDecorationsInner(_ref7) {
-  var state = _ref7.state,
-    pluginState = _ref7.pluginState,
-    nodeViewSerializer = _ref7.nodeViewSerializer,
-    colorScheme = _ref7.colorScheme,
-    intl = _ref7.intl,
-    activeIndexPos = _ref7.activeIndexPos,
-    api = _ref7.api,
-    _ref7$isInverted = _ref7.isInverted,
-    isInverted = _ref7$isInverted === void 0 ? false : _ref7$isInverted,
-    _ref7$diffType = _ref7.diffType,
-    diffType = _ref7$diffType === void 0 ? (0, _isExtendedEnabled.getDefaultDiffType)() : _ref7$diffType,
-    _ref7$hideDeletedDiff = _ref7.hideDeletedDiffs,
-    hideDeletedDiffs = _ref7$hideDeletedDiff === void 0 ? false : _ref7$hideDeletedDiff,
-    _ref7$hideAddedDiffsU = _ref7.hideAddedDiffsUnderline,
-    hideAddedDiffsUnderlineParam = _ref7$hideAddedDiffsU === void 0 ? false : _ref7$hideAddedDiffsU,
-    _ref7$showIndicators = _ref7.showIndicators,
-    showIndicators = _ref7$showIndicators === void 0 ? false : _ref7$showIndicators,
-    smartThresholds = _ref7.smartThresholds,
-    _ref7$deletedDiffPlac = _ref7.deletedDiffPlacement,
-    deletedDiffPlacement = _ref7$deletedDiffPlac === void 0 ? 'top' : _ref7$deletedDiffPlac,
-    _ref7$inlineDeletedDi = _ref7.inlineDeletedDiffPlacement,
-    inlineDeletedDiffPlacement = _ref7$inlineDeletedDi === void 0 ? 'before' : _ref7$inlineDeletedDi,
-    tagMountContext = _ref7.tagMountContext;
+
+/**
+ * Single place `platform_editor_diff_hide_pure_deletions` is read, shared by the widget-anchor
+ * calculation and the decoration push below so the two cannot drift.
+ *
+ * The clean view ("new state") historically only suppressed a deleted side when the same change
+ * also carried an insertion. A PURE deletion — text removed with nothing put in its place — fell
+ * through and leaked struck-through text into a view that is supposed to show none. `smart` never
+ * showed this because it pairs a rewrite into a single change that always carries an insertion;
+ * `inline` emits delete-only changes, so it did.
+ *
+ * With the gate off this returns exactly what both callers computed before it existed.
+ */
+var shouldHideDeletedSide = function shouldHideDeletedSide(_ref7) {
+  var change = _ref7.change,
+    diffType = _ref7.diffType,
+    hideDeletedDiffs = _ref7.hideDeletedDiffs,
+    isInverted = _ref7.isInverted;
+  return (0, _isExtendedEnabled.isExtendedEnabled)(diffType) && !isInverted && !!hideDeletedDiffs && (change.inserted.length > 0 || (0, _fg.fg)('platform_editor_diff_hide_pure_deletions'));
+};
+var calculateDiffDecorationsInner = function calculateDiffDecorationsInner(_ref8) {
+  var state = _ref8.state,
+    pluginState = _ref8.pluginState,
+    nodeViewSerializer = _ref8.nodeViewSerializer,
+    colorScheme = _ref8.colorScheme,
+    intl = _ref8.intl,
+    activeIndexPos = _ref8.activeIndexPos,
+    api = _ref8.api,
+    _ref8$isInverted = _ref8.isInverted,
+    isInverted = _ref8$isInverted === void 0 ? false : _ref8$isInverted,
+    _ref8$diffType = _ref8.diffType,
+    diffType = _ref8$diffType === void 0 ? (0, _isExtendedEnabled.getDefaultDiffType)() : _ref8$diffType,
+    _ref8$hideDeletedDiff = _ref8.hideDeletedDiffs,
+    hideDeletedDiffs = _ref8$hideDeletedDiff === void 0 ? false : _ref8$hideDeletedDiff,
+    _ref8$hideAddedDiffsU = _ref8.hideAddedDiffsUnderline,
+    hideAddedDiffsUnderlineParam = _ref8$hideAddedDiffsU === void 0 ? false : _ref8$hideAddedDiffsU,
+    _ref8$showIndicators = _ref8.showIndicators,
+    showIndicators = _ref8$showIndicators === void 0 ? false : _ref8$showIndicators,
+    smartThresholds = _ref8.smartThresholds,
+    _ref8$deletedDiffPlac = _ref8.deletedDiffPlacement,
+    deletedDiffPlacement = _ref8$deletedDiffPlac === void 0 ? 'top' : _ref8$deletedDiffPlac,
+    _ref8$inlineDeletedDi = _ref8.inlineDeletedDiffPlacement,
+    inlineDeletedDiffPlacement = _ref8$inlineDeletedDi === void 0 ? 'before' : _ref8$inlineDeletedDi,
+    reveal = _ref8.reveal,
+    tagMountContext = _ref8.tagMountContext;
   var originalDoc = pluginState.originalDoc,
     steps = pluginState.steps,
     stepAttributions = pluginState.stepAttributions,
@@ -298,12 +326,12 @@ var calculateDiffDecorationsInner = function calculateDiffDecorationsInner(_ref7
   var simplifiedStepAttributions = [];
   if (attributedChangesetEnabled) {
     var simplifiedStepsWithAttribution = (0, _simplifySteps.simplifyStepsWithAttribution)(steps, stepAttributions !== null && stepAttributions !== void 0 ? stepAttributions : [], originalDoc);
-    simplifiedSteps = simplifiedStepsWithAttribution.map(function (_ref8) {
-      var step = _ref8.step;
+    simplifiedSteps = simplifiedStepsWithAttribution.map(function (_ref9) {
+      var step = _ref9.step;
       return step;
     });
-    simplifiedStepAttributions = simplifiedStepsWithAttribution.map(function (_ref9) {
-      var stepAttribution = _ref9.stepAttribution;
+    simplifiedStepAttributions = simplifiedStepsWithAttribution.map(function (_ref0) {
+      var stepAttribution = _ref0.stepAttribution;
       return stepAttribution;
     });
   } else {
@@ -337,11 +365,11 @@ var calculateDiffDecorationsInner = function calculateDiffDecorationsInner(_ref7
   } finally {
     _iterator.f();
   }
-  var attrStepContexts = (0, _fg.fg)('platform_editor_reduce_diff_attr_sensitivity') ? pendingAttrSteps.flatMap(function (_ref0) {
-    var afterNode = _ref0.afterNode,
-      beforeNode = _ref0.beforeNode,
-      mapIndex = _ref0.mapIndex,
-      step = _ref0.step;
+  var attrStepContexts = (0, _fg.fg)('platform_editor_reduce_diff_attr_sensitivity') ? pendingAttrSteps.flatMap(function (_ref1) {
+    var afterNode = _ref1.afterNode,
+      beforeNode = _ref1.beforeNode,
+      mapIndex = _ref1.mapIndex,
+      step = _ref1.step;
     var finalPosition = new _transform.Mapping(stepMaps.slice(mapIndex + 1)).mapResult(step.pos);
     if (finalPosition.deleted) {
       return [];
@@ -356,8 +384,8 @@ var calculateDiffDecorationsInner = function calculateDiffDecorationsInner(_ref7
     }), {}, {
       step: step
     })];
-  }) : pendingAttrSteps.map(function (_ref1) {
-    var step = _ref1.step;
+  }) : pendingAttrSteps.map(function (_ref10) {
+    var step = _ref10.step;
     return {
       afterNode: tr.doc.nodeAt(step.pos),
       beforeNode: originalDoc.nodeAt(step.pos),
@@ -494,7 +522,12 @@ var calculateDiffDecorationsInner = function calculateDiffDecorationsInner(_ref7
       // change. Used to decide if indicator anchor positions should be adjusted
       // inward — when the widget is present the anchor must stay at the block
       // boundary to keep the indicator bar continuous with the deleted content.
-      var willRenderDeletedWidget = change.deleted.length > 0 && !isMarkOnly && !((0, _isExtendedEnabled.isExtendedEnabled)(diffType) && !isInverted && hideDeletedDiffs && change.inserted.length > 0);
+      var willRenderDeletedWidget = change.deleted.length > 0 && !isMarkOnly && !shouldHideDeletedSide({
+        change: change,
+        diffType: diffType,
+        hideDeletedDiffs: hideDeletedDiffs,
+        isInverted: isInverted
+      });
       if (isSmartNodeLevel) {
         // For a large inserted table, skip the O(cells) inline decorations inside the
         // table (the cell overlays added below provide the highlight without the
@@ -528,6 +561,7 @@ var calculateDiffDecorationsInner = function calculateDiffDecorationsInner(_ref7
               hideAddedDiffsUnderline: hideAddedDiffsUnderline,
               hasDeletedWidget: willRenderDeletedWidget,
               isDeletedWidgetBelow: isDeletedWidgetBelow,
+              reveal: reveal,
               tagMountContext: tagMountContext
             }))));
           }
@@ -552,10 +586,12 @@ var calculateDiffDecorationsInner = function calculateDiffDecorationsInner(_ref7
           hideAddedDiffsUnderline: hideAddedDiffsUnderline,
           hasDeletedWidget: willRenderDeletedWidget,
           isDeletedWidgetBelow: isDeletedWidgetBelow,
+          reveal: reveal,
           tagMountContext: tagMountContext
         }))));
       }
       decorations.push.apply(decorations, (0, _toConsumableArray2.default)(calculateNodesForBlockDecoration(_objectSpread(_objectSpread({
+        attributionKey: attributionKey,
         doc: tr.doc,
         from: change.fromB,
         to: change.toB,
@@ -563,7 +599,9 @@ var calculateDiffDecorationsInner = function calculateDiffDecorationsInner(_ref7
       }, (0, _isExtendedEnabled.isExtendedEnabled)(diffType) && {
         isInserted: isInserted,
         shouldHideDeleted: shouldHideDeleted,
-        showIndicators: showIndicators
+        showContributorTags: showContributorTags,
+        showIndicators: showIndicators,
+        tagMountContext: tagMountContext
       }), {}, {
         activeIndexPos: activeIndexPos,
         intl: intl,
@@ -572,11 +610,19 @@ var calculateDiffDecorationsInner = function calculateDiffDecorationsInner(_ref7
       }))));
     }
     if (change.deleted.length > 0 && !isMarkOnly) {
-      var _shouldHideDeleted = (0, _isExtendedEnabled.isExtendedEnabled)(diffType) ? !isInverted && hideDeletedDiffs && change.inserted.length > 0 : false;
+      var _shouldHideDeleted = shouldHideDeletedSide({
+        change: change,
+        diffType: diffType,
+        hideDeletedDiffs: hideDeletedDiffs,
+        isInverted: isInverted
+      });
       if (!_shouldHideDeleted) {
         decorations.push.apply(decorations, (0, _toConsumableArray2.default)((0, _createNodeChangedDecorationWidget.createNodeChangedDecorationWidget)(_objectSpread(_objectSpread({
           // The deleted side has no inline decoration, so the widget carries the actor.
           attributionKey: attributionKey,
+          // Extended pipeline only, matching the inline path: the non-extended styles have no
+          // background for the wipe to act on.
+          reveal: (0, _isExtendedEnabled.isExtendedEnabled)(diffType) ? reveal : undefined,
           change: change,
           doc: originalDoc,
           nodeViewSerializer: nodeViewSerializer,
@@ -604,9 +650,9 @@ var calculateDiffDecorationsInner = function calculateDiffDecorationsInner(_ref7
   // smart path. Only add the legacy step-range decoration when no equivalent changeset range was
   // produced; otherwise the same content receives two inline decorations (and two navigation
   // changes), usually one attribution colour and one default purple decoration.
-  (0, _getMarkChangeRanges.getMarkChangeRanges)(steps).filter(function (_ref10) {
-    var fromB = _ref10.fromB,
-      toB = _ref10.toB;
+  (0, _getMarkChangeRanges.getMarkChangeRanges)(steps).filter(function (_ref11) {
+    var fromB = _ref11.fromB,
+      toB = _ref11.toB;
     return !(0, _fg.fg)('confluence_ncs_step_diffing_version_history') || !changes.some(function (change) {
       return change.fromB === fromB && change.toB === toB;
     });
@@ -617,7 +663,8 @@ var calculateDiffDecorationsInner = function calculateDiffDecorationsInner(_ref7
       colorScheme: colorScheme,
       diffType: diffType,
       isActive: isActive,
-      isInserted: true
+      isInserted: true,
+      reveal: reveal
     })));
   });
   (0, _getAttrChangeRanges.getAttrChangeRanges)(tr.doc, attrStepContexts, originalDoc).forEach(function (change) {
@@ -707,44 +754,49 @@ var calculateDiffDecorationsInner = function calculateDiffDecorationsInner(_ref7
 var calculateDiffDecorations = exports.calculateDiffDecorations = (0, _memoizeOne.default)(calculateDiffDecorationsInner,
 // Cache results unless relevant inputs change. `tagMountContext` is deliberately absent: it is
 // built once per plugin instance, so it can never be the reason a diff needs recalculating.
-function (_ref11, _ref12) {
-  var _ref16;
-  var _ref13 = (0, _slicedToArray2.default)(_ref11, 1),
-    _ref13$ = _ref13[0],
-    pluginState = _ref13$.pluginState,
-    state = _ref13$.state,
-    colorScheme = _ref13$.colorScheme,
-    intl = _ref13$.intl,
-    activeIndexPos = _ref13$.activeIndexPos,
-    isInverted = _ref13$.isInverted,
-    diffType = _ref13$.diffType,
-    hideDeletedDiffs = _ref13$.hideDeletedDiffs,
-    hideAddedDiffsUnderline = _ref13$.hideAddedDiffsUnderline,
-    showIndicators = _ref13$.showIndicators,
-    smartThresholds = _ref13$.smartThresholds,
-    deletedDiffPlacement = _ref13$.deletedDiffPlacement,
-    inlineDeletedDiffPlacement = _ref13$.inlineDeletedDiffPlacement;
+function (_ref12, _ref13) {
+  var _ref17;
   var _ref14 = (0, _slicedToArray2.default)(_ref12, 1),
     _ref14$ = _ref14[0],
-    lastPluginState = _ref14$.pluginState,
-    lastState = _ref14$.state,
-    lastColorScheme = _ref14$.colorScheme,
-    lastIntl = _ref14$.intl,
-    lastActiveIndexPos = _ref14$.activeIndexPos,
-    lastIsInverted = _ref14$.isInverted,
-    lastDiffType = _ref14$.diffType,
-    lastHideDeletedDiffs = _ref14$.hideDeletedDiffs,
-    lastHideAddedDiffsUnderline = _ref14$.hideAddedDiffsUnderline,
-    lastShowIndicators = _ref14$.showIndicators,
-    lastSmartThresholds = _ref14$.smartThresholds,
-    lastDeletedDiffPlacement = _ref14$.deletedDiffPlacement,
-    lastInlineDeletedDiffPlacement = _ref14$.inlineDeletedDiffPlacement;
+    pluginState = _ref14$.pluginState,
+    state = _ref14$.state,
+    colorScheme = _ref14$.colorScheme,
+    intl = _ref14$.intl,
+    activeIndexPos = _ref14$.activeIndexPos,
+    isInverted = _ref14$.isInverted,
+    diffType = _ref14$.diffType,
+    hideDeletedDiffs = _ref14$.hideDeletedDiffs,
+    hideAddedDiffsUnderline = _ref14$.hideAddedDiffsUnderline,
+    showIndicators = _ref14$.showIndicators,
+    smartThresholds = _ref14$.smartThresholds,
+    deletedDiffPlacement = _ref14$.deletedDiffPlacement,
+    inlineDeletedDiffPlacement = _ref14$.inlineDeletedDiffPlacement,
+    reveal = _ref14$.reveal;
+  var _ref15 = (0, _slicedToArray2.default)(_ref13, 1),
+    _ref15$ = _ref15[0],
+    lastPluginState = _ref15$.pluginState,
+    lastState = _ref15$.state,
+    lastColorScheme = _ref15$.colorScheme,
+    lastIntl = _ref15$.intl,
+    lastActiveIndexPos = _ref15$.activeIndexPos,
+    lastIsInverted = _ref15$.isInverted,
+    lastDiffType = _ref15$.diffType,
+    lastHideDeletedDiffs = _ref15$.hideDeletedDiffs,
+    lastHideAddedDiffsUnderline = _ref15$.hideAddedDiffsUnderline,
+    lastShowIndicators = _ref15$.showIndicators,
+    lastSmartThresholds = _ref15$.smartThresholds,
+    lastDeletedDiffPlacement = _ref15$.deletedDiffPlacement,
+    lastInlineDeletedDiffPlacement = _ref15$.inlineDeletedDiffPlacement,
+    lastReveal = _ref15$.reveal;
   var originalDocIsSame = lastPluginState.originalDoc && pluginState.originalDoc && pluginState.originalDoc.eq(lastPluginState.originalDoc);
   if ((0, _isExtendedEnabled.isExtendedEnabled)(diffType)) {
-    var _ref15;
-    return (_ref15 = colorScheme === lastColorScheme && intl.locale === lastIntl.locale && isInverted === lastIsInverted && diffType === lastDiffType && (0, _isEqual.default)(activeIndexPos, lastActiveIndexPos) && originalDocIsSame && (0, _isEqual.default)(pluginState.steps, lastPluginState.steps) && (0, _isEqual.default)(pluginState.stepAttributions, lastPluginState.stepAttributions) &&
+    var _ref16;
+    return (_ref16 = colorScheme === lastColorScheme && intl.locale === lastIntl.locale && isInverted === lastIsInverted && diffType === lastDiffType && (0, _isEqual.default)(activeIndexPos, lastActiveIndexPos) && originalDocIsSame && (0, _isEqual.default)(pluginState.steps, lastPluginState.steps) && (0, _isEqual.default)(pluginState.stepAttributions, lastPluginState.stepAttributions) &&
     // Contributors can arrive after the first calculation; without this, tags never appear.
-    (0, _isEqual.default)(pluginState.contributors, lastPluginState.contributors) && state.doc.eq(lastState.doc) && hideDeletedDiffs === lastHideDeletedDiffs && hideAddedDiffsUnderline === lastHideAddedDiffsUnderline && showIndicators === lastShowIndicators && (0, _isEqual.default)(smartThresholds, lastSmartThresholds) && deletedDiffPlacement === lastDeletedDiffPlacement && inlineDeletedDiffPlacement === lastInlineDeletedDiffPlacement) !== null && _ref15 !== void 0 ? _ref15 : false;
+    (0, _isEqual.default)(pluginState.contributors, lastPluginState.contributors) && state.doc.eq(lastState.doc) && hideDeletedDiffs === lastHideDeletedDiffs && hideAddedDiffsUnderline === lastHideAddedDiffsUnderline && showIndicators === lastShowIndicators && (0, _isEqual.default)(smartThresholds, lastSmartThresholds) && deletedDiffPlacement === lastDeletedDiffPlacement && inlineDeletedDiffPlacement === lastInlineDeletedDiffPlacement &&
+    // Turning the reveal on or off changes the emitted styles, so a cached result from the
+    // other setting would leave the diff painted in the wrong resting state.
+    (0, _isEqual.default)(reveal, lastReveal)) !== null && _ref16 !== void 0 ? _ref16 : false;
   }
-  return (_ref16 = originalDocIsSame && (0, _isEqual.default)(pluginState.steps, lastPluginState.steps) && (0, _isEqual.default)(pluginState.stepAttributions, lastPluginState.stepAttributions) && (0, _isEqual.default)(pluginState.contributors, lastPluginState.contributors) && state.doc.eq(lastState.doc) && colorScheme === lastColorScheme && intl.locale === lastIntl.locale && (0, _isEqual.default)(activeIndexPos, lastActiveIndexPos) && hideDeletedDiffs === lastHideDeletedDiffs) !== null && _ref16 !== void 0 ? _ref16 : false;
+  return (_ref17 = originalDocIsSame && (0, _isEqual.default)(pluginState.steps, lastPluginState.steps) && (0, _isEqual.default)(pluginState.stepAttributions, lastPluginState.stepAttributions) && (0, _isEqual.default)(pluginState.contributors, lastPluginState.contributors) && state.doc.eq(lastState.doc) && colorScheme === lastColorScheme && intl.locale === lastIntl.locale && (0, _isEqual.default)(activeIndexPos, lastActiveIndexPos) && hideDeletedDiffs === lastHideDeletedDiffs) !== null && _ref17 !== void 0 ? _ref17 : false;
 });

package/dist/cjs/pm-plugins/decorations/colorSchemes/factory.js

@@ -66,6 +66,8 @@ exports.buildInsertStyleRuleNodeNew = buildInsertStyleRuleNodeNew;
 exports.buildInsertedBlockNodeStyle = buildInsertedBlockNodeStyle;
 exports.buildInsertedInlineStyle = buildInsertedInlineStyle;
 exports.getAccentTokens = getAccentTokens;
+exports.getDeletedInlineRevealColors = getDeletedInlineRevealColors;
+exports.getInsertedInlineRevealColors = getInsertedInlineRevealColors;
 var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/defineProperty"));
 var _lazyNodeView = require("@atlaskit/editor-common/lazy-node-view");
 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; }
@@ -750,6 +752,36 @@ function buildInsertedInlineStyle(colors, isActive, hideAddedDiffsUnderline) {
   return hideAddedDiffsUnderline ? buildInsertStyleExtendedNoUnderline(colors) : buildInsertStyleExtended(colors);
 }
 
+/**
+ * The colours `buildInsertedInlineStyle()` above would paint, for the reveal to animate towards.
+ *
+ * MUST mirror that function: the reveal replaces the flat highlight with a sized gradient, so a
+ * divergence here means the animation settles on a different colour than a static render produces.
+ * `revealStyles.ts` in the test package asserts the two agree across the isActive /
+ * hideAddedDiffsUnderline matrix.
+ *
+ * A transparent border is how "no underline" is expressed: the box still reserves the 2px so the
+ * text does not shift when the underline arrives.
+ */
+function getInsertedInlineRevealColors(colors, isActive, hideAddedDiffsUnderline) {
+  return {
+    background: isActive ? bgSubtlerPressed(colors.insertActiveColor) : bgSubtlest(colors.insertColor),
+    border: hideAddedDiffsUnderline ? 'transparent' : borderAccent(colors.insertColor)
+  };
+}
+
+/**
+ * The colours `buildDeletedInlineContentStyleExtended()` paints, for the reveal to animate towards.
+ * Deleted content's highlight does not vary with the active state — only its text does, which the
+ * reveal keeps as-is.
+ */
+function getDeletedInlineRevealColors(colors) {
+  return {
+    background: bgSubtlest(colors.deleteColor),
+    border: borderAccent(colors.deleteColor)
+  };
+}
+
 /** The three visual states deleted inline content can be in. */
 
 /**

package/dist/cjs/pm-plugins/decorations/createBlockChangedDecoration.js

@@ -8,12 +8,14 @@ exports.createBlockChangedDecoration = void 0;
 var _toConsumableArray2 = _interopRequireDefault(require("@babel/runtime/helpers/toConsumableArray"));
 var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/defineProperty"));
 var _lazyNodeView = require("@atlaskit/editor-common/lazy-node-view");
+var _utils = require("@atlaskit/editor-prosemirror/utils");
 var _view = require("@atlaskit/editor-prosemirror/view");
 var _isExperimentEnabled = require("@atlaskit/platform-feature-experiments/is-experiment-enabled");
 var _isExtendedEnabled = require("../isExtendedEnabled");
 var _factory = require("./colorSchemes/factory");
 var _schemes = require("./colorSchemes/schemes");
 var _createAnchorDecorationWidgets = require("./createAnchorDecorationWidgets");
+var _createContributorTagWidget = require("./createContributorTagWidget");
 var _decorationKeys = require("./decorationKeys");
 var _createBlockChangedDecorationStyles = require("./createBlockChangedDecoration.styles.legacy");
 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; }
@@ -31,6 +33,29 @@ var UNSTYLED_NODES = ['mediaSingle', 'mediaGroup', 'table',
 'heading', 'hardBreak', 'decisionList', 'taskList', 'bulletList', 'orderedList', 'layoutSection'];
 var CELL_NODES = ['tableCell', 'tableHeader'];
 
+/**
+ * Block nodes that host a contributor tag of their own — those whose whole box reads as one change.
+ * Lists and tables are excluded: they decorate per item and per cell, with no one-tag-per-block
+ * design yet. `blockCard` covers the datasource variant, which is the same node type.
+ */
+var TAGGABLE_BLOCK_NODES = new Set(['blockCard', 'blockquote', 'bodiedExtension', 'codeBlock', 'embedCard', 'expand', 'extension', 'media', 'multiBodiedExtension', 'panel', 'rule']);
+
+/** Panels, rules and media can also sit in a table cell, which is out of scope for block tags. */
+var isInsideTable = function isInsideTable(doc, pos) {
+  return (0, _utils.findParentNodeClosestToPos)(doc.resolve(pos), function (node) {
+    return node.type.name === 'table';
+  }) !== undefined;
+};
+
+/** Whether this block can host a contributor tag of its own. */
+var canTagBlock = function canTagBlock(_ref) {
+  var diffType = _ref.diffType,
+    doc = _ref.doc,
+    from = _ref.from,
+    name = _ref.name;
+  return doc !== undefined && (0, _isExtendedEnabled.isExtendedEnabled)(diffType) && TAGGABLE_BLOCK_NODES.has(name) && !isInsideTable(doc, from);
+};
+
 /** Positioning context for the cell overlay widget decorations. */
 var cellPositionStyle = (0, _lazyNodeView.convertToInlineCss)({
   position: 'relative'
@@ -77,14 +102,14 @@ var getNodeClass = function getNodeClass(name) {
       return undefined;
   }
 };
-var getBlockNodeStyleNext = function getBlockNodeStyleNext(_ref) {
-  var nodeName = _ref.nodeName,
-    colorScheme = _ref.colorScheme,
-    _ref$isInserted = _ref.isInserted,
-    isInserted = _ref$isInserted === void 0 ? true : _ref$isInserted,
-    _ref$isActive = _ref.isActive,
-    isActive = _ref$isActive === void 0 ? false : _ref$isActive,
-    diffType = _ref.diffType;
+var getBlockNodeStyleNext = function getBlockNodeStyleNext(_ref2) {
+  var nodeName = _ref2.nodeName,
+    colorScheme = _ref2.colorScheme,
+    _ref2$isInserted = _ref2.isInserted,
+    isInserted = _ref2$isInserted === void 0 ? true : _ref2$isInserted,
+    _ref2$isActive = _ref2.isActive,
+    isActive = _ref2$isActive === void 0 ? false : _ref2$isActive,
+    diffType = _ref2.diffType;
   if (UNSTYLED_NODES.includes(nodeName)) {
     return undefined;
   }
@@ -146,21 +171,33 @@ var isCellEmpty = function isCellEmpty(cellNode) {
  * @param isActive Whether this node is part of the currently active/focused change
  * @returns Prosemirror node decoration or undefined
  */
-var createBlockChangedDecoration = exports.createBlockChangedDecoration = function createBlockChangedDecoration(_ref2) {
-  var change = _ref2.change,
-    colorScheme = _ref2.colorScheme,
-    _ref2$isInserted = _ref2.isInserted,
-    isInserted = _ref2$isInserted === void 0 ? true : _ref2$isInserted,
-    _ref2$isActive = _ref2.isActive,
-    isActive = _ref2$isActive === void 0 ? false : _ref2$isActive,
-    _ref2$shouldHideDelet = _ref2.shouldHideDeleted,
-    shouldHideDeleted = _ref2$shouldHideDelet === void 0 ? false : _ref2$shouldHideDelet,
-    _ref2$showIndicators = _ref2.showIndicators,
-    showIndicators = _ref2$showIndicators === void 0 ? false : _ref2$showIndicators,
-    doc = _ref2.doc,
-    diffType = _ref2.diffType;
+var createBlockChangedDecoration = exports.createBlockChangedDecoration = function createBlockChangedDecoration(_ref3) {
+  var attributionKey = _ref3.attributionKey,
+    change = _ref3.change,
+    colorScheme = _ref3.colorScheme,
+    _ref3$isInserted = _ref3.isInserted,
+    isInserted = _ref3$isInserted === void 0 ? true : _ref3$isInserted,
+    _ref3$isActive = _ref3.isActive,
+    isActive = _ref3$isActive === void 0 ? false : _ref3$isActive,
+    _ref3$shouldHideDelet = _ref3.shouldHideDeleted,
+    shouldHideDeleted = _ref3$shouldHideDelet === void 0 ? false : _ref3$shouldHideDelet,
+    _ref3$showContributor = _ref3.showContributorTags,
+    showContributorTags = _ref3$showContributor === void 0 ? false : _ref3$showContributor,
+    _ref3$showIndicators = _ref3.showIndicators,
+    showIndicators = _ref3$showIndicators === void 0 ? false : _ref3$showIndicators,
+    doc = _ref3.doc,
+    diffType = _ref3.diffType,
+    tagMountContext = _ref3.tagMountContext;
   var decorations = [];
-  var diffId = crypto.randomUUID();
+  // Derived from the node range so it survives a recalculation, as in
+  // `createInlineChangedDecoration`. Changes are disjoint, so no two of them decorate one node.
+  var diffId = showContributorTags ? "block-".concat(change.from, "-").concat(change.to) : crypto.randomUUID();
+  var shouldTagBlock = showContributorTags && canTagBlock({
+    diffType: diffType,
+    doc: doc,
+    from: change.from,
+    name: change.name
+  });
   if (shouldHideDeleted) {
     return [_view.Decoration.node(change.from, change.to, {
       style: displayNoneStyle
@@ -206,19 +243,27 @@ var createBlockChangedDecoration = exports.createBlockChangedDecoration = functi
   });
   var className = getNodeClass(change.name);
   if (style || className) {
-    decorations.push(_view.Decoration.node(change.from, change.to, {
+    decorations.push(_view.Decoration.node(change.from, change.to, _objectSpread({
       style: style,
       'data-testid': 'show-diff-changed-decoration-node',
       class: className
-    }, (0, _decorationKeys.buildDiffDecorationSpec)({
+    }, shouldTagBlock && {
+      'data-diff-id': diffId
+    }), (0, _decorationKeys.buildDiffDecorationSpec)({
+      attributionKey: shouldTagBlock ? attributionKey : undefined,
+      colorScheme: colorScheme,
       decorationType: 'block',
       diffId: diffId,
       isActive: isActive,
+      isInserted: isInserted,
       nodeName: change.name,
       diffType: diffType
     })));
   }
-  if (decorations.length > 0 && showIndicators && doc && (0, _isExtendedEnabled.isExtendedEnabled)(diffType)) {
+  if (decorations.length === 0) {
+    return decorations;
+  }
+  if (showIndicators && doc && (0, _isExtendedEnabled.isExtendedEnabled)(diffType)) {
     decorations.push.apply(decorations, (0, _toConsumableArray2.default)((0, _createAnchorDecorationWidgets.createBlockIndicatorAnchorWidgets)({
       doc: doc,
       from: change.from,
@@ -226,5 +271,22 @@ var createBlockChangedDecoration = exports.createBlockChangedDecoration = functi
       diffId: diffId
     })));
   }
+
+  // `change.from` is just before the node, so the host sits outside it and the tag renders on the
+  // block's own top-left corner. Resolving inline content instead would put the host inside a
+  // container block's content DOM (panel, expand, blockquote).
+  if (shouldTagBlock && doc) {
+    var tagWidget = (0, _createContributorTagWidget.createContributorTagWidget)({
+      anchorAtRangeStart: true,
+      doc: doc,
+      from: change.from,
+      to: change.to,
+      diffId: diffId,
+      mountContext: tagMountContext
+    });
+    if (tagWidget) {
+      decorations.push(tagWidget);
+    }
+  }
   return decorations;
 };