@atlaskit/editor-plugin-show-diff 29.0.2 → 30.0.0

package/CHANGELOG.md

@@ -1,5 +1,24 @@
 # @atlaskit/editor-plugin-show-diff
 
+## 30.0.0
+
+### Major Changes
+
+- [`113b5bcc64f0c`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/113b5bcc64f0c) -
+  Remove the platform_editor_diff_reveal_animation feature gate and the unused diff reveal
+  animation. Removes the `RevealMode` and `RevealOptions` types and the `PMDiffParams` `reveal`
+  option.
+
+### Minor Changes
+
+- [`ded1801d8ed0c`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/ded1801d8ed0c) -
+  Remove the platform_editor_diff_hide_pure_deletions feature gate. The clean diff view keeps its
+  existing behaviour.
+
+### Patch Changes
+
+- Updated dependencies
+
 ## 29.0.2
 
 ### Patch Changes

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

@@ -343,26 +343,15 @@ var isDeletedContentPlacedBelow = exports.isDeletedContentPlacedBelow = function
 };
 
 /**
- * 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.
- *
- * Umbrella experiment: within the AI streaming UX M1 population this behaviour is only enabled
- * for the treatment cohort. Non-AI callers (track-changes, etc.) are not enrolled and default
- * to `true`, preserving the pre-umbrella baseline.
+ * Shared by the widget-anchor calculation and the decoration push below so the two cannot drift.
+ * The clean view ("new state") only suppresses a deleted side when the same change also carries
+ * an insertion.
  */
 var shouldHideDeletedSide = function shouldHideDeletedSide(_ref7) {
   var change = _ref7.change,
     hideDeletedDiffs = _ref7.hideDeletedDiffs,
     isInverted = _ref7.isInverted;
-  return !isInverted && !!hideDeletedDiffs && (change.inserted.length > 0 || (0, _fg.fg)('platform_editor_diff_hide_pure_deletions') && (0, _unsafeExpValNoExposure.UNSAFE_expValNoExposure)('platform_editor_ai_streaming_ux_experience_m1', 'isEnabled', false) === true);
+  return !isInverted && !!hideDeletedDiffs && change.inserted.length > 0;
 };
 var calculateDiffDecorationsInner = function calculateDiffDecorationsInner(_ref8) {
   var state = _ref8.state,
@@ -387,7 +376,6 @@ var calculateDiffDecorationsInner = function calculateDiffDecorationsInner(_ref8
     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,
@@ -667,7 +655,6 @@ var calculateDiffDecorationsInner = function calculateDiffDecorationsInner(_ref8
               diffId: showContributorTags ? "blank-line-".concat(pos) : undefined,
               isActive: isActive,
               pos: pos,
-              reveal: reveal,
               side: -1,
               tagMountContext: tagMountContext
             }));
@@ -725,7 +712,6 @@ var calculateDiffDecorationsInner = function calculateDiffDecorationsInner(_ref8
           isDeletedWidgetBelow: isDeletedWidgetBelow,
           isInserted: isInserted,
           leftAnchorId: leftAnchorId,
-          reveal: reveal,
           shouldHideDeleted: shouldHideDeleted,
           showContributorTags: showContributorTags,
           showIndicators: showIndicators,
@@ -763,7 +749,6 @@ var calculateDiffDecorationsInner = function calculateDiffDecorationsInner(_ref8
               hideAddedDiffsUnderline: hideAddedDiffsUnderline,
               hasDeletedWidget: willRenderDeletedWidget,
               isDeletedWidgetBelow: isDeletedWidgetBelow,
-              reveal: reveal,
               tagMountContext: tagMountContext
             })));
           }
@@ -787,7 +772,6 @@ var calculateDiffDecorationsInner = function calculateDiffDecorationsInner(_ref8
           hideAddedDiffsUnderline: hideAddedDiffsUnderline,
           hasDeletedWidget: willRenderDeletedWidget,
           isDeletedWidgetBelow: isDeletedWidgetBelow,
-          reveal: reveal,
           tagMountContext: tagMountContext
         })));
       }
@@ -824,7 +808,6 @@ var calculateDiffDecorationsInner = function calculateDiffDecorationsInner(_ref8
           deletedColumns: deletedColumns,
           // The deleted side has no inline decoration, so the widget carries the actor.
           attributionKey: attributionKey,
-          reveal: reveal,
           change: change,
           doc: originalDoc,
           nodeViewSerializer: nodeViewSerializer,
@@ -904,7 +887,6 @@ var calculateDiffDecorationsInner = function calculateDiffDecorationsInner(_ref8
       doc: tr.doc,
       isActive: isActive,
       isInserted: true,
-      reveal: reveal,
       showContributorTags: showContributorTags,
       tagMountContext: tagMountContext
     })));
@@ -1038,8 +1020,7 @@ function (_ref14, _ref15) {
     showIndicators = _ref16$.showIndicators,
     smartThresholds = _ref16$.smartThresholds,
     deletedDiffPlacement = _ref16$.deletedDiffPlacement,
-    inlineDeletedDiffPlacement = _ref16$.inlineDeletedDiffPlacement,
-    reveal = _ref16$.reveal;
+    inlineDeletedDiffPlacement = _ref16$.inlineDeletedDiffPlacement;
   var _ref17 = (0, _slicedToArray2.default)(_ref15, 1),
     _ref17$ = _ref17[0],
     lastPluginState = _ref17$.pluginState,
@@ -1054,13 +1035,9 @@ function (_ref14, _ref15) {
     lastShowIndicators = _ref17$.showIndicators,
     lastSmartThresholds = _ref17$.smartThresholds,
     lastDeletedDiffPlacement = _ref17$.deletedDiffPlacement,
-    lastInlineDeletedDiffPlacement = _ref17$.inlineDeletedDiffPlacement,
-    lastReveal = _ref17$.reveal;
+    lastInlineDeletedDiffPlacement = _ref17$.inlineDeletedDiffPlacement;
   var originalDocIsSame = lastPluginState.originalDoc && pluginState.originalDoc && pluginState.originalDoc.eq(lastPluginState.originalDoc);
   return (_ref18 = 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 &&
-  // 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 && _ref18 !== void 0 ? _ref18 : 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) !== null && _ref18 !== void 0 ? _ref18 : false;
 });

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

@@ -73,8 +73,6 @@ exports.getAccentBorderColor = getAccentBorderColor;
 exports.getAccentTokens = getAccentTokens;
 exports.getDeletedHighlightHoverBorderColor = getDeletedHighlightHoverBorderColor;
 exports.getDeletedHighlightHoverColor = getDeletedHighlightHoverColor;
-exports.getDeletedInlineRevealColors = getDeletedInlineRevealColors;
-exports.getInsertedInlineRevealColors = getInsertedInlineRevealColors;
 var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/defineProperty"));
 var _agentBrandColorSchemes = require("@atlaskit/agent-color/agent-brand-color-schemes");
 var _cssHelper = require("@atlaskit/editor-common/css-helper");
@@ -931,43 +929,6 @@ 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) {
-  // Nothing to wipe in where the resting state paints no highlight.
-  if (colors.deletedInlineHighlight === 'onEmphasis') {
-    return {
-      background: 'transparent',
-      border: 'transparent'
-    };
-  }
-  return {
-    background: bgSubtlest(colors.deleteColor),
-    border: borderAccent(colors.deleteColor)
-  };
-}
-
 /**
  * Value for `DELETED_HIGHLIGHT_BG_VAR` while a change is hovered — the same tint an 'always' scheme
  * paints at rest, so hovering restores the appearance rather than introducing a third one.

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

@@ -19,7 +19,6 @@ var _createAnchorDecorationWidgets = require("./createAnchorDecorationWidgets");
 var _createContributorTagWidget = require("./createContributorTagWidget");
 var _createInlineChangedDecorationStyles = require("./createInlineChangedDecoration.styles.legacy");
 var _decorationKeys = require("./decorationKeys");
-var _revealStyles = require("./revealStyles");
 function ownKeys(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbols) { var o = Object.getOwnPropertySymbols(e); r && (o = o.filter(function (r) { return Object.getOwnPropertyDescriptor(e, r).enumerable; })), t.push.apply(t, o); } return t; }
 function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var t = null != arguments[r] ? arguments[r] : {}; r % 2 ? ownKeys(Object(t), !0).forEach(function (r) { (0, _defineProperty2.default)(e, r, t[r]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(e, Object.getOwnPropertyDescriptors(t)) : ownKeys(Object(t)).forEach(function (r) { Object.defineProperty(e, r, Object.getOwnPropertyDescriptor(t, r)); }); } return e; }
 var displayNoneStyle = (0, _cssHelper.convertToInlineCss)({
@@ -133,7 +132,6 @@ var getAtomicInlineChangedAttrs = exports.getAtomicInlineChangedAttrs = function
  */
 
 var createInlineChangedDecoration = exports.createInlineChangedDecoration = function createInlineChangedDecoration(_ref2) {
-  var _revealed$style;
   var attributionKey = _ref2.attributionKey,
     change = _ref2.change,
     colorScheme = _ref2.colorScheme,
@@ -158,7 +156,6 @@ var createInlineChangedDecoration = exports.createInlineChangedDecoration = func
     hasDeletedWidget = _ref2$hasDeletedWidge === void 0 ? false : _ref2$hasDeletedWidge,
     _ref2$isDeletedWidget = _ref2.isDeletedWidgetBelow,
     isDeletedWidgetBelow = _ref2$isDeletedWidget === void 0 ? false : _ref2$isDeletedWidget,
-    reveal = _ref2.reveal,
     tagMountContext = _ref2.tagMountContext;
   // Derived from the range so it survives a recalculation: stepping to the next/previous change
   // rebuilds decorations without touching the doc, and a fresh id would change the React key and the
@@ -183,17 +180,7 @@ var createInlineChangedDecoration = exports.createInlineChangedDecoration = func
     }))];
   }
   var colors = getColorScheme(colorScheme);
-
-  // The reveal withholds the static highlight so the wipe has something to reveal, so it supplies
-  // the whole style rather than adding to the usual one.
-  var revealed = (0, _revealStyles.resolveRevealStyle)({
-    colorScheme: colorScheme,
-    hideAddedDiffsUnderline: hideAddedDiffsUnderline,
-    isActive: isActive,
-    isInserted: isInserted,
-    reveal: reveal
-  });
-  var style = (_revealed$style = revealed === null || revealed === void 0 ? void 0 : revealed.style) !== null && _revealed$style !== void 0 ? _revealed$style : resolveInlineChangedStyle({
+  var style = resolveInlineChangedStyle({
     colors: colors,
     colorScheme: colorScheme,
     hideAddedDiffsUnderline: hideAddedDiffsUnderline,
@@ -224,13 +211,13 @@ var createInlineChangedDecoration = exports.createInlineChangedDecoration = func
     mountContext: tagMountContext
   }) : undefined;
   var inlineStyle = [style, atomicInlineAttrs === null || atomicInlineAttrs === void 0 ? void 0 : atomicInlineAttrs.styleSuffix, deletedInlineNodeAttrs === null || deletedInlineNodeAttrs === void 0 ? void 0 : deletedInlineNodeAttrs.styleSuffix, tagWidget ? stackBelowContributorTagStyle : undefined, (0, _fg.fg)('platform_editor_ai_show_diff_patch_1') ? _decorationKeys.scrollMarginTopStyle : undefined].filter(Boolean).join('');
-  var decorations = [_view.Decoration.inline(change.fromB, change.toB, _objectSpread(_objectSpread(_objectSpread(_objectSpread({
+  var decorations = [_view.Decoration.inline(change.fromB, change.toB, _objectSpread(_objectSpread(_objectSpread({
     style: inlineStyle
   }, atomicInlineAttrs && {
     class: atomicInlineAttrs.className
   }), deletedInlineNodeAttrs && {
     class: deletedInlineNodeAttrs.className
-  }), revealed && (0, _defineProperty2.default)({}, _revealStyles.REVEAL_ATTR, revealed.role)), {}, {
+  }), {}, {
     'data-testid': 'show-diff-changed-decoration'
   }, (showContributorTags || (0, _fg.fg)('platform_editor_ai_show_diff_patch_2')) && {
     'data-diff-id': diffId

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

@@ -177,7 +177,6 @@ var createNodeChangedDecorationWidget = exports.createNodeChangedDecorationWidge
     placeBelow = _ref2$placeBelow === void 0 ? false : _ref2$placeBelow,
     _ref2$hideAddedDiffsU = _ref2.hideAddedDiffsUnderline,
     hideAddedDiffsUnderline = _ref2$hideAddedDiffsU === void 0 ? false : _ref2$hideAddedDiffsU,
-    reveal = _ref2.reveal,
     tagMountContext = _ref2.tagMountContext,
     deletedColumns = _ref2.deletedColumns;
   var slice = doc.slice(change.fromA, change.toA);
@@ -208,7 +207,6 @@ var createNodeChangedDecorationWidget = exports.createNodeChangedDecorationWidge
       diffId: showContributorTags ? "widget-".concat(change.fromA, "-").concat(change.toA) : undefined,
       isActive: isActive,
       pos: safeInsertPos,
-      reveal: reveal,
       side: placeBelow ? 1 : -1,
       tagMountContext: tagMountContext
     })];
@@ -445,7 +443,7 @@ var createNodeChangedDecorationWidget = exports.createNodeChangedDecorationWidge
           // view's styles.
           dom.append(createAtomicInlineOutline(attrOnlyNodeName, nodeView, colorScheme));
         } else {
-          var wrapper = (0, _wrapBlockNodeView.createContentWrapper)(colorScheme, isActive, isInserted, reveal);
+          var wrapper = (0, _wrapBlockNodeView.createContentWrapper)(colorScheme, isActive, isInserted);
           wrapper.append(nodeView);
           dom.append(wrapper);
         }
@@ -474,12 +472,11 @@ var createNodeChangedDecorationWidget = exports.createNodeChangedDecorationWidge
             node: fallbackNode,
             colorScheme: colorScheme,
             isActive: isActive,
-            isInserted: isInserted,
-            reveal: reveal
+            isInserted: isInserted
           });
           dom.append(injectedNode);
         } else {
-          var _wrapper2 = (0, _wrapBlockNodeView.createContentWrapper)(colorScheme, isActive, isInserted, reveal);
+          var _wrapper2 = (0, _wrapBlockNodeView.createContentWrapper)(colorScheme, isActive, isInserted);
           _wrapper2.append(fallbackNode);
           dom.append(_wrapper2);
         }

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

@@ -330,7 +330,6 @@ var createReplacedCellDecorations = exports.createReplacedCellDecorations = func
     isDeletedWidgetBelow = _ref5.isDeletedWidgetBelow,
     isInserted = _ref5.isInserted,
     leftAnchorId = _ref5.leftAnchorId,
-    reveal = _ref5.reveal,
     shouldHideDeleted = _ref5.shouldHideDeleted,
     showContributorTags = _ref5.showContributorTags,
     showIndicators = _ref5.showIndicators,
@@ -378,7 +377,6 @@ var createReplacedCellDecorations = exports.createReplacedCellDecorations = func
       isDeletedWidgetBelow: isDeletedWidgetBelow,
       isInserted: isInserted,
       leftAnchorId: leftAnchorId,
-      reveal: reveal,
       shouldHideDeleted: hidesLiveCell,
       showContributorTags: showContributorTags,
       showIndicators: showIndicators,

package/dist/cjs/pm-plugins/decorations/utils/createDeletedLineBreakWidget.js

@@ -28,7 +28,7 @@ var RETURN_GLYPH = exports.RETURN_GLYPH = '⤶';
  * in for the absent content keeps the deletion visible without inventing content that was not there.
  *
  * `createContentWrapper` supplies the highlight and the strikethrough line, so the glyph is styled
- * exactly like any other run of deleted inline content, including the active and reveal states.
+ * exactly like any other run of deleted inline content, including the active state.
  *
  * Hidden from assistive technology: the glyph is a picture of a key, and read out it is announced as
  * its Unicode name rather than as a removed line.
@@ -38,8 +38,7 @@ var createDeletedLineBreakWidget = exports.createDeletedLineBreakWidget = functi
     _ref$count = _ref.count,
     count = _ref$count === void 0 ? 1 : _ref$count,
     _ref$isActive = _ref.isActive,
-    isActive = _ref$isActive === void 0 ? false : _ref$isActive,
-    reveal = _ref.reveal;
+    isActive = _ref$isActive === void 0 ? false : _ref$isActive;
   var dom = document.createElement('span');
   // The testid deleted content is matched by, so the glyph is found wherever deleted content is.
   dom.setAttribute('data-testid', 'show-diff-deleted-decoration');
@@ -54,7 +53,7 @@ var createDeletedLineBreakWidget = exports.createDeletedLineBreakWidget = functi
       // deleted-content wrapper is styled as an inline run.
       dom.append(dom.ownerDocument.createElement('br'));
     }
-    var wrapper = (0, _wrapBlockNodeView.createContentWrapper)(colorScheme, isActive, false, reveal);
+    var wrapper = (0, _wrapBlockNodeView.createContentWrapper)(colorScheme, isActive, false);
     wrapper.append(dom.ownerDocument.createTextNode(RETURN_GLYPH));
     dom.append(wrapper);
   }
@@ -79,14 +78,12 @@ var createDeletedLineBreakDecoration = exports.createDeletedLineBreakDecoration
     _ref2$isActive = _ref2.isActive,
     isActive = _ref2$isActive === void 0 ? false : _ref2$isActive,
     pos = _ref2.pos,
-    reveal = _ref2.reveal,
     side = _ref2.side,
     tagMountContext = _ref2.tagMountContext;
   var dom = createDeletedLineBreakWidget({
     colorScheme: colorScheme,
     count: count,
-    isActive: isActive,
-    reveal: reveal
+    isActive: isActive
   });
   var tagHost = attributionKey ? (0, _createContributorTagWidget.createContributorTagHost)({
     diffId: diffId,

package/dist/cjs/pm-plugins/decorations/utils/wrapBlockNodeView.js

@@ -14,7 +14,6 @@ var _fg = require("@atlaskit/platform-feature-flags/fg");
 var _expValEquals = require("@atlaskit/tmp-editor-statsig/exp-val-equals");
 var _baseNodeName = require("../../utils/baseNodeName");
 var _createInlineChangedDecoration = require("../createInlineChangedDecoration");
-var _revealStyles = require("../revealStyles");
 var _getAttrChangeRanges = require("./getAttrChangeRanges");
 var _tableCellEdgeAttrs = require("./tableCellEdgeAttrs");
 var _wrapBlockNodeViewStyles = require("./wrapBlockNodeViewStyles");
@@ -642,17 +641,9 @@ var injectInnerWrapper = exports.injectInnerWrapper = function injectInnerWrappe
   var node = _ref12.node,
     colorScheme = _ref12.colorScheme,
     isActive = _ref12.isActive,
-    isInserted = _ref12.isInserted,
-    reveal = _ref12.reveal;
+    isInserted = _ref12.isInserted;
   var wrapper = document.createElement('span');
-  var revealStyle = (0, _revealStyles.applyRevealToElement)({
-    colorScheme: colorScheme,
-    element: wrapper,
-    isActive: isActive !== null && isActive !== void 0 ? isActive : false,
-    isInserted: isInserted !== null && isInserted !== void 0 ? isInserted : false,
-    reveal: reveal
-  });
-  wrapper.setAttribute('style', (isInserted ? (0, _wrapBlockNodeViewStyles.getInsertedContentStyle)(colorScheme, isActive) : (0, _wrapBlockNodeViewStyles.getDeletedContentStyle)(colorScheme, isActive, Boolean(revealStyle))) + revealStyle);
+  wrapper.setAttribute('style', isInserted ? (0, _wrapBlockNodeViewStyles.getInsertedContentStyle)(colorScheme, isActive) : (0, _wrapBlockNodeViewStyles.getDeletedContentStyle)(colorScheme, isActive));
   (0, _toConsumableArray2.default)(node.childNodes).forEach(function (child) {
     var removedChild = node.removeChild(child);
     wrapper.append(removedChild);
@@ -667,26 +658,16 @@ var injectInnerWrapper = exports.injectInnerWrapper = function injectInnerWrappe
 var createContentWrapper = exports.createContentWrapper = function createContentWrapper(colorScheme) {
   var isActive = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : false;
   var isInserted = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : false;
-  var reveal = arguments.length > 3 ? arguments[3] : undefined;
   var wrapper = document.createElement('span');
   var baseStyle = (0, _cssHelper.convertToInlineCss)({
     position: 'relative',
     width: 'fit-content'
   });
-  // Empty unless the reveal is running, in which case the static highlight below is withheld so
-  // the animation can wipe it in instead.
-  var revealStyle = (0, _revealStyles.applyRevealToElement)({
-    colorScheme: colorScheme,
-    element: wrapper,
-    isActive: isActive,
-    isInserted: isInserted,
-    reveal: reveal
-  });
-  var deletedStyle = (0, _wrapBlockNodeViewStyles.getDeletedContentStyle)(colorScheme, isActive, Boolean(revealStyle));
+  var deletedStyle = (0, _wrapBlockNodeViewStyles.getDeletedContentStyle)(colorScheme, isActive);
   if (isInserted) {
-    wrapper.setAttribute('style', "".concat(baseStyle).concat((0, _wrapBlockNodeViewStyles.getInsertedContentStyle)(colorScheme, isActive)).concat(revealStyle));
+    wrapper.setAttribute('style', "".concat(baseStyle).concat((0, _wrapBlockNodeViewStyles.getInsertedContentStyle)(colorScheme, isActive)));
   } else {
-    wrapper.setAttribute('style', "".concat(baseStyle).concat(deletedStyle).concat(revealStyle));
+    wrapper.setAttribute('style', "".concat(baseStyle).concat(deletedStyle));
     var strikethrough = document.createElement('span');
     strikethrough.setAttribute('style', (0, _wrapBlockNodeViewStyles.getDeletedContentStyleUnbounded)(colorScheme, isActive));
     wrapper.append(strikethrough);

package/dist/cjs/pm-plugins/decorations/utils/wrapBlockNodeViewStyles.js

@@ -109,12 +109,11 @@ var getInsertedContentStyleNext = function getInsertedContentStyleNext(colorSche
 };
 var getDeletedContentStyleNext = function getDeletedContentStyleNext(colorScheme) {
   var isActive = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : false;
-  var omitHighlight = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : false;
   var colors = getColorScheme(colorScheme);
   var base = (0, _factory.buildDeletedInlineContentStyle)(colors, isActive ? 'active' : 'new');
 
   // glyphTint adds a background highlight and border-bottom over the tint; strikethrough does not.
-  var needsExtendedHighlight = colors.deletedInlineTreatment === 'glyphTint' && !omitHighlight;
+  var needsExtendedHighlight = colors.deletedInlineTreatment === 'glyphTint';
   return needsExtendedHighlight ? base + (0, _factory.buildDeletedInlineContentStyleExtended)(colors, isActive) : base;
 };
 var resolveCellOverlayStyleNext = function resolveCellOverlayStyleNext(_ref) {
@@ -173,8 +172,7 @@ var getInsertedContentStyle = exports.getInsertedContentStyle = function getInse
 /** Inline style for deleted content. */
 var getDeletedContentStyle = exports.getDeletedContentStyle = function getDeletedContentStyle(colorScheme) {
   var isActive = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : false;
-  var omitHighlight = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : false;
-  return (0, _isExperimentEnabled.isExperimentEnabled)('platform_editor_show_diff_color_scheme_refactor') ? getDeletedContentStyleNext(colorScheme, isActive, omitHighlight) : (0, _wrapBlockNodeViewStyles.getDeletedContentStyleLegacy)((0, _schemes.getLegacyColorScheme)(colorScheme), isActive, omitHighlight);
+  return (0, _isExperimentEnabled.isExperimentEnabled)('platform_editor_show_diff_color_scheme_refactor') ? getDeletedContentStyleNext(colorScheme, isActive) : (0, _wrapBlockNodeViewStyles.getDeletedContentStyleLegacy)((0, _schemes.getLegacyColorScheme)(colorScheme), isActive);
 };
 
 /**

package/dist/cjs/pm-plugins/decorations/utils/wrapBlockNodeViewStyles.legacy.js

@@ -173,14 +173,10 @@ var getInsertedContentStyleLegacy = exports.getInsertedContentStyleLegacy = func
 /** Pre-refactor inline style for deleted content. */
 var getDeletedContentStyleLegacy = exports.getDeletedContentStyleLegacy = function getDeletedContentStyleLegacy(colorScheme) {
   var isActive = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : false;
-  var omitHighlight = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : false;
   if (colorScheme === 'traditional') {
     return (0, _traditional.getDeletedTraditionalInlineStyle)(isActive);
   }
-  if (!omitHighlight) {
-    return (isActive ? (0, _standard.getStandardDeletedContentStyleActive)() : (0, _standard.getStandardDeletedContentStyleNew)()) + _standard.deletedInlineContentStyleExtended;
-  }
-  return isActive ? (0, _standard.getStandardDeletedContentStyleActive)() : (0, _standard.getStandardDeletedContentStyleNew)();
+  return (isActive ? (0, _standard.getStandardDeletedContentStyleActive)() : (0, _standard.getStandardDeletedContentStyleNew)()) + _standard.deletedInlineContentStyleExtended;
 };
 
 /**

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

@@ -21,7 +21,6 @@ var _enforceCustomStepRegisters = require("./enforceCustomStepRegisters");
 var _getDefaultDiffType = require("./getDefaultDiffType");
 var _getScrollableDecorations = require("./getScrollableDecorations");
 var _NodeViewSerializer = require("./NodeViewSerializer");
-var _revealAnimation = require("./revealAnimation");
 var _scrollToDiff = require("./scrollToDiff");
 function ownKeys(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbols) { var o = Object.getOwnPropertySymbols(e); r && (o = o.filter(function (r) { return Object.getOwnPropertyDescriptor(e, r).enumerable; })), t.push.apply(t, o); } return t; }
 function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var t = null != arguments[r] ? arguments[r] : {}; r % 2 ? ownKeys(Object(t), !0).forEach(function (r) { (0, _defineProperty2.default)(e, r, t[r]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(e, Object.getOwnPropertyDescriptors(t)) : ownKeys(Object(t)).forEach(function (r) { Object.defineProperty(e, r, Object.getOwnPropertyDescriptor(t, r)); }); } return e; }
@@ -93,21 +92,11 @@ var createPlugin = exports.createPlugin = function createPlugin(config, getIntl,
         var meta = tr.getMeta(showDiffPluginKey);
         var newPluginState = currentPluginState;
         if (meta) {
-          if ((meta === null || meta === void 0 ? void 0 : meta.action) === 'SHOW_DIFF' || (meta === null || meta === void 0 ? void 0 : meta.action) === 'REVEAL_COMPLETE') {
-            var _newPluginState$color, _newPluginState, _newPluginState2, _newPluginState3, _newPluginState4, _newPluginState5, _newPluginState6, _newPluginState7, _newPluginState8, _newPluginState9, _newPluginState0;
-            // REVEAL_COMPLETE repaints with the reveal dropped, so the decorations render their
-            // ordinary resting style. Without it the reveal stays in state indefinitely and the
-            // next unrelated repaint re-emits the hidden, zero-width highlight with no animation
-            // left to bring it in — the highlight simply vanishes.
-            newPluginState = meta.action === 'REVEAL_COMPLETE' ? _objectSpread(_objectSpread({}, currentPluginState), {}, {
-              reveal: undefined
-            }) : _objectSpread(_objectSpread(_objectSpread({}, currentPluginState), meta), {}, {
+          if ((meta === null || meta === void 0 ? void 0 : meta.action) === 'SHOW_DIFF') {
+            var _newPluginState$color, _newPluginState, _newPluginState2, _newPluginState3, _newPluginState4, _newPluginState5, _newPluginState6, _newPluginState7, _newPluginState8, _newPluginState9;
+            newPluginState = _objectSpread(_objectSpread(_objectSpread({}, currentPluginState), meta), {}, {
               isDisplayingChanges: true,
-              activeIndex: undefined,
-              // Explicit rather than left to the spread: an absent key in `meta` would let
-              // the previous paint's choreography leak into this one, so closing the diff
-              // would animate too.
-              reveal: meta.reveal
+              activeIndex: undefined
             });
             // Calculate and store decorations in state
             var _calculateDiffDecorat = (0, _calculateDiffDecorations.calculateDiffDecorations)({
@@ -126,10 +115,7 @@ var createPlugin = exports.createPlugin = function createPlugin(config, getIntl,
                 showIndicators: (_newPluginState6 = newPluginState) === null || _newPluginState6 === void 0 ? void 0 : _newPluginState6.showIndicators,
                 smartThresholds: (_newPluginState7 = newPluginState) === null || _newPluginState7 === void 0 ? void 0 : _newPluginState7.smartThresholds,
                 deletedDiffPlacement: (_newPluginState8 = newPluginState) === null || _newPluginState8 === void 0 ? void 0 : _newPluginState8.deletedDiffPlacement,
-                inlineDeletedDiffPlacement: (_newPluginState9 = newPluginState) === null || _newPluginState9 === void 0 ? void 0 : _newPluginState9.inlineDeletedDiffPlacement,
-                // SHOW_DIFF only. The scroll-to-next recalculation further down deliberately
-                // omits this so stepping through changes cannot replay the choreography.
-                reveal: (_newPluginState0 = newPluginState) === null || _newPluginState0 === void 0 ? void 0 : _newPluginState0.reveal
+                inlineDeletedDiffPlacement: (_newPluginState9 = newPluginState) === null || _newPluginState9 === void 0 ? void 0 : _newPluginState9.inlineDeletedDiffPlacement
               }),
               contributorTags = _calculateDiffDecorat.contributorTags,
               decorations = _calculateDiffDecorat.decorations,
@@ -144,7 +130,6 @@ var createPlugin = exports.createPlugin = function createPlugin(config, getIntl,
               isDisplayingChanges: false,
               activeIndex: undefined,
               contributorTags: [],
-              reveal: undefined,
               // Per-call override — do not let it leak into the next, unrelated `showDiff` call.
               colorScheme: undefined,
               /**
@@ -161,7 +146,7 @@ var createPlugin = exports.createPlugin = function createPlugin(config, getIntl,
             // Update the active index in plugin state and recalculate decorations
             var _decorations = (0, _getScrollableDecorations.getScrollableDecorations)(currentPluginState.decorations, newState.doc);
             if (_decorations.length > 0) {
-              var _currentPluginState$a, _newPluginState$color2, _newPluginState1;
+              var _currentPluginState$a, _newPluginState$color2, _newPluginState0;
               // Initialize to -1 if undefined so that the first "next" scroll takes us to index 0 (first change).
               // This allows the UI to start with no selection and only highlight on first user interaction.
               var nextIndex = (_currentPluginState$a = currentPluginState.activeIndex) !== null && _currentPluginState$a !== void 0 ? _currentPluginState$a : -1;
@@ -191,7 +176,7 @@ var createPlugin = exports.createPlugin = function createPlugin(config, getIntl,
                   state: newState,
                   pluginState: newPluginState,
                   nodeViewSerializer: nodeViewSerializer,
-                  colorScheme: (_newPluginState$color2 = (_newPluginState1 = newPluginState) === null || _newPluginState1 === void 0 ? void 0 : _newPluginState1.colorScheme) !== null && _newPluginState$color2 !== void 0 ? _newPluginState$color2 : config === null || config === void 0 ? void 0 : config.colorScheme,
+                  colorScheme: (_newPluginState$color2 = (_newPluginState0 = newPluginState) === null || _newPluginState0 === void 0 ? void 0 : _newPluginState0.colorScheme) !== null && _newPluginState$color2 !== void 0 ? _newPluginState$color2 : config === null || config === void 0 ? void 0 : config.colorScheme,
                   intl: getIntl(),
                   activeIndexPos: newPluginState.activeIndexPos,
                   api: api,
@@ -248,10 +233,6 @@ var createPlugin = exports.createPlugin = function createPlugin(config, getIntl,
           }
           var pluginState = showDiffPluginKey.getState(view.state);
 
-          // A repaint rebuilds inline decorations, destroying any animation bound to them, so
-          // an in-flight reveal has to be re-attached to the new nodes. No-op when idle.
-          (0, _revealAnimation.rebindReveal)(view, view.dom);
-
           // Scroll to the first decoration when scrollIntoView was requested.
           // Use the same filtered/position-sorted list as the active-index path
           // so "first" reliably means the topmost scrollable diff in the document.

package/dist/cjs/showDiffPlugin.js

@@ -8,14 +8,11 @@ exports.showDiffPlugin = void 0;
 var _objectWithoutProperties2 = _interopRequireDefault(require("@babel/runtime/helpers/objectWithoutProperties"));
 var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/defineProperty"));
 var _react = _interopRequireDefault(require("react"));
-var _unsafeExpValNoExposure = require("@atlaskit/platform-feature-experiments/unsafe-exp-val-no-exposure");
-var _fg = require("@atlaskit/platform-feature-flags/fg");
 var _attributions = require("./pm-plugins/decorations/colorSchemes/attributions");
 var _getDeletedWidgets2 = require("./pm-plugins/getDeletedWidgets");
 var _getScrollableDecorations = require("./pm-plugins/getScrollableDecorations");
 var _main = require("./pm-plugins/main");
 var _resolveDiffContributors = require("./pm-plugins/resolveDiffContributors");
-var _revealAnimation = require("./pm-plugins/revealAnimation");
 var _IndicatorBarContentComponent = require("./ui/IndicatorBar/IndicatorBarContentComponent");
 var _excluded = ["stepsWithAttribution", "contributorProfiles", "showContributorTags"];
 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; }
@@ -47,25 +44,6 @@ var normalizeShowDiffParams = function normalizeShowDiffParams(params) {
     })
   });
 };
-var prefersReducedMotion = function prefersReducedMotion() {
-  return typeof window !== 'undefined' && typeof window.matchMedia === 'function' && window.matchMedia('(prefers-reduced-motion: reduce)').matches;
-};
-
-/**
- * Resolve reveal preconditions once here so decorations and animation share one answer. If
- * decorations think a reveal is happening but animation doesn't, highlight paints hidden forever.
- * All preconditions resolved here (not just gate) ensures "unable to animate" means "do not reveal".
- *
- * Umbrella experiment: within the AI streaming UX M1 population this behaviour is only enabled
- * for the treatment cohort. Non-AI callers (track-changes, etc.) are not enrolled and default
- * to `true`, preserving the pre-umbrella baseline.
- */
-var resolveReveal = function resolveReveal(reveal, editorView) {
-  return (
-    // Gate first, so it is the primary switch and is always observable once a reveal is requested.
-    reveal && (0, _fg.fg)('platform_editor_diff_reveal_animation') && (0, _unsafeExpValNoExposure.UNSAFE_expValNoExposure)('platform_editor_ai_streaming_ux_experience_m1', 'isEnabled', false) === true && editorView && typeof window !== 'undefined' && !prefersReducedMotion() ? reveal : undefined
-  );
-};
 var showDiffPlugin = exports.showDiffPlugin = function showDiffPlugin(_ref3) {
   var api = _ref3.api,
     config = _ref3.config;
@@ -89,25 +67,7 @@ var showDiffPlugin = exports.showDiffPlugin = function showDiffPlugin(_ref3) {
           api === null || api === void 0 || (_api$userIntent = api.userIntent) === null || _api$userIntent === void 0 || _api$userIntent.commands.setCurrentUserIntent('viewingDiff')({
             tr: tr
           });
-          var reveal = resolveReveal(params === null || params === void 0 ? void 0 : params.reveal, editorView);
-
-          // Capture snapshot BEFORE transaction dispatch (last moment outgoing DOM exists).
-          if (reveal && editorView) {
-            var view = editorView;
-            (0, _revealAnimation.beginReveal)({
-              editorView: view,
-              reveal: reveal,
-              onComplete: function onComplete() {
-                // Repaint without the reveal so the decorations settle into their resting
-                // style. Kept out of history: this is presentation, not an edit.
-                view.dispatch(view.state.tr.setMeta(_main.showDiffPluginKey, {
-                  action: 'REVEAL_COMPLETE'
-                }).setMeta('addToHistory', false));
-              }
-            });
-          }
           return tr.setMeta(_main.showDiffPluginKey, _objectSpread(_objectSpread({}, normalizeShowDiffParams(params)), {}, {
-            reveal: reveal,
             action: 'SHOW_DIFF'
           }));
         };
@@ -115,9 +75,6 @@ var showDiffPlugin = exports.showDiffPlugin = function showDiffPlugin(_ref3) {
       hideDiff: function hideDiff(_ref5) {
         var _api$userIntent2;
         var tr = _ref5.tr;
-        if (editorView) {
-          (0, _revealAnimation.cancelReveal)(editorView);
-        }
         api === null || api === void 0 || (_api$userIntent2 = api.userIntent) === null || _api$userIntent2 === void 0 || _api$userIntent2.commands.setCurrentUserIntent('default')({
           tr: tr
         });

package/dist/cjs/showDiffPluginType.js

@@ -83,18 +83,6 @@ var ASSERT_DIFF_AGENT_BRANDS_ARE_REGISTERED = exports.ASSERT_DIFF_AGENT_BRANDS_A
 // Re-export the canonical `SmartDiffThresholds` declaration (single source of truth) so the
 // public plugin types stay in sync with the smart-diff implementation.
 
-/**
- * How the diff is revealed when it is painted.
- *
- * - `phased` — the two-phase choreography used when opening the diff from a clean "new state":
- *   the outgoing state cross-fades to the incoming one while the agent highlight wipes out to the
- *   right, then every highlight wipes back in from the left.
- *
- * Deliberately separate from {@link DiffType}: that describes how changes are computed and grouped,
- * this describes how the result is presented over time. Folding one into the other would make every
- * `DiffType` consumer — version history, track-changes, publish diff — care about presentation.
- */
-
 /**
  * Attributed alternative to `PMDiffParams`. When agent colouring is enabled, changes are grouped
  * by `userId`, with `agentId` used as a tie-breaker.