@atlaskit/editor-plugin-table 17.7.0 → 17.7.2

package/CHANGELOG.md

@@ -1,5 +1,26 @@
 # @atlaskit/editor-plugin-table
 
+## 17.7.2
+
+### Patch Changes
+
+- [`1fd2b267eb592`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/1fd2b267eb592) -
+  Cleanup `platform_editor_ai_aifc_patch_ga` flag
+- [`ac4a428022f83`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/ac4a428022f83) -
+  [EDITOR-5480] Removes data-borders-ready fix from table and adds border to prosemirror toDOM
+  instead
+- Updated dependencies
+
+## 17.7.1
+
+### Patch Changes
+
+- [`94dbc76370522`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/94dbc76370522) -
+  [ux] remove unnecessary decoration for last cell element in table
+- [`1d6c1d6ba61dc`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/1d6c1d6ba61dc) -
+  FFCLEANUP-79953 clean up stale experiment platform_editor_editor_width_analytics
+- Updated dependencies
+
 ## 17.7.0
 
 ### Minor Changes

package/dist/cjs/nodeviews/TableContainer.js

@@ -30,9 +30,7 @@ var InnerContainer = /*#__PURE__*/(0, _react.forwardRef)(function (_ref, ref) {
   var className = _ref.className,
     style = _ref.style,
     node = _ref.node,
-    children = _ref.children,
-    tableWrapperHeight = _ref.tableWrapperHeight;
-  var bordersReady = (0, _expValEquals.expValEquals)('platform_editor_vc90_transition_table_border', 'isEnabled', true) ? tableWrapperHeight !== undefined && tableWrapperHeight > 0 : undefined;
+    children = _ref.children;
   return /*#__PURE__*/_react.default.createElement("div", {
     ref: ref
     // eslint-disable-next-line @atlaskit/ui-styling-standard/enforce-style-prop -- Ignored via go/DSP-18766
@@ -43,7 +41,6 @@ var InnerContainer = /*#__PURE__*/(0, _react.forwardRef)(function (_ref, ref) {
     className: className,
     "data-number-column": node.attrs.isNumberColumnEnabled,
     "data-layout": node.attrs.layout,
-    "data-borders-ready": bordersReady,
     "data-testid": "table-container"
   }, children);
 });
@@ -345,8 +342,7 @@ var ResizableTableContainer = exports.ResizableTableContainer = /*#__PURE__*/_re
   // eslint-disable-next-line @atlaskit/ui-styling-standard/no-classname-prop -- Ignored via go/DSP-18766
   , {
     className: className,
-    node: node,
-    tableWrapperHeight: tableWrapperHeight
+    node: node
   }, children))));
 });
 var TableContainer = exports.TableContainer = function TableContainer(_ref5) {
@@ -405,8 +401,7 @@ var TableContainer = exports.TableContainer = function TableContainer(_ref5) {
   var isFullPageAppearance = !isCommentEditor && !isChromelessEditor;
   var resizableTableWidth = isFullPageAppearance ? (0, _misc.getTableResizerContainerForFullPageWidthInCSS)(node, isTableScalingEnabled) : "calc(100cqw - calc(var(--ak-editor--large-gutter-padding) * 2))";
   return /*#__PURE__*/_react.default.createElement(InnerContainer, {
-    node: node,
-    tableWrapperHeight: tableWrapperHeight
+    node: node
     // eslint-disable-next-line @atlaskit/ui-styling-standard/no-classname-prop -- Ignored via go/DSP-18766
     ,
     className: (0, _classnames.default)(className, {

package/dist/cjs/nodeviews/TableRow.js

@@ -17,7 +17,6 @@ var _nesting = require("@atlaskit/editor-common/nesting");
 var _nodeVisibility = require("@atlaskit/editor-common/node-visibility");
 var _ui = require("@atlaskit/editor-common/ui");
 var _utils = require("@atlaskit/editor-prosemirror/utils");
-var _platformFeatureFlags = require("@atlaskit/platform-feature-flags");
 var _pluginFactory = require("../pm-plugins/plugin-factory");
 var _pluginKey = require("../pm-plugins/plugin-key");
 var _commands = require("../pm-plugins/sticky-headers/commands");
@@ -101,20 +100,16 @@ var TableRow = exports.default = /*#__PURE__*/function (_TableNodeView) {
     }
     var pos = _this.getPos();
     _this.isInNestedTable = false;
-    if ((0, _platformFeatureFlags.fg)('platform_editor_ai_aifc_patch_ga')) {
-      try {
-        // We cannot trust that the value from getPos will be defined
-        // https://discuss.prosemirror.net/t/getpos-is-undefined-in-nodeview-constructor/1246/4
-        // There are also scenarios where the value it brings back does not tally with the current doc
-        // E.g. when AI streaming brings in new content, this position brings back incorrect values that cannot be resolved
-        if (pos) {
-          _this.isInNestedTable = (0, _nesting.getParentOfTypeCount)(view.state.schema.nodes.table)(view.state.doc.resolve(pos)) > 1;
-        }
-      } catch (e) {}
-    } else {
+    try {
+      // We cannot trust that the value from getPos will be defined
+      // https://discuss.prosemirror.net/t/getpos-is-undefined-in-nodeview-constructor/1246/4
+      // There are also scenarios where the value it brings back does not tally with the current doc
+      // E.g. when AI streaming brings in new content, this position brings back incorrect values that cannot be resolved
       if (pos) {
         _this.isInNestedTable = (0, _nesting.getParentOfTypeCount)(view.state.schema.nodes.table)(view.state.doc.resolve(pos)) > 1;
       }
+    } catch (_unused) {
+      // Intentionally swallowed — getPos can return stale positions during AI streaming
     }
     if (_this.isHeaderRow) {
       _this.dom.setAttribute('data-vc-nvs', 'true');

package/dist/cjs/nodeviews/toDOM.js

@@ -62,7 +62,16 @@ var tableNodeSpecWithFixedToDOM = exports.tableNodeSpecWithFixedToDOM = function
       }, ['table', attrs, colgroup, ['tbody', 0]]], ['div', {
         class: 'pm-table-sticky-sentinel-bottom',
         'data-testid': 'sticky-sentinel-bottom'
-      }]];
+      }]].concat((0, _toConsumableArray2.default)((0, _expValEquals.expValEquals)('platform_editor_vc90_transition_table_border', 'isEnabled', true) ? [['div', {
+        contenteditable: 'false',
+        class: 'pm-table-left-border',
+        'data-with-numbered-table': node.attrs.isNumberColumnEnabled,
+        'data-testid': 'table-left-border'
+      }], ['div', {
+        contenteditable: 'false',
+        class: 'pm-table-right-border',
+        'data-testid': 'table-right-border'
+      }]] : []));
       if (!config.tableResizingEnabled || config.isNested) {
         return ['div', {
           class: 'tableView-content-wrap',

package/dist/cjs/pm-plugins/utils/decoration.js

@@ -13,6 +13,7 @@ var _utils = require("@atlaskit/editor-common/utils");
 var _view = require("@atlaskit/editor-prosemirror/view");
 var _tableMap = require("@atlaskit/editor-tables/table-map");
 var _utils2 = require("@atlaskit/editor-tables/utils");
+var _expValEquals = require("@atlaskit/tmp-editor-statsig/exp-val-equals");
 var _types = require("../../types");
 var _ColumnResizeWidget = require("../../ui/ColumnResizeWidget");
 // eslint-disable-next-line @atlaskit/platform/prefer-crypto-random-uuid -- Use crypto.randomUUID instead
@@ -287,12 +288,17 @@ var createResizeHandleDecoration = exports.createResizeHandleDecoration = functi
       return element;
     }, {
       key: "".concat(_types.TableDecorations.COLUMN_RESIZING_HANDLE_WIDGET, "_").concat(rowIndex, "_").concat(columnIndex, "_").concat(includeTooltip ? 'with' : 'no', "-tooltip"),
-      destroy: function destroy(node) {
+      destroy: function destroy(_node) {
         nodeViewPortalProviderAPI.remove(decorationRenderKey);
       }
     });
   };
   var createLastCellElementDecoration = function createLastCellElementDecoration(cellColumnPositioning, cellPos, cellNode) {
+    if ((0, _expValEquals.expValEquals)('platform_editor_table_remove_last_cell_decoration', 'isEnabled', true)) {
+      // no longer need to add the last cell decoration to override marginBottom as media wrapper doesn't have margin bottom. This will avoid unnecessary decoration computation/mutation and improve performance
+      // consider clean up ClassName.LAST_ITEM_IN_CELL with platform_editor_table_remove_last_cell_decoration experiment
+      return null;
+    }
     var lastItemPositions;
     cellNode.forEach(function (childNode, offset, index) {
       if (index === cellNode.childCount - 1) {

package/dist/cjs/tablePlugin.js

@@ -433,39 +433,37 @@ var tablePlugin = function tablePlugin(_ref) {
           return new _safePlugin.SafePlugin({
             view: function view(editorView) {
               editorViewRef.current = editorView;
-              var setTimeoutID;
               var rafID;
               var ricID;
-              if ((0, _expValEquals.expValEquals)('platform_editor_editor_width_analytics', 'isEnabled', true)) {
-                // send statistics about the widths of the tables on the page for alerting
-                // only send this event once, after the editorView is first initialised
-                setTimeoutID = setTimeout(function () {
-                  var requestIdleCallbackFn = function requestIdleCallbackFn() {
-                    var _api$width$sharedStat2;
-                    var editorWidth = api === null || api === void 0 || (_api$width$sharedStat2 = api.width.sharedState.currentState()) === null || _api$width$sharedStat2 === void 0 ? void 0 : _api$width$sharedStat2.width;
-                    if (editorViewRef.current) {
-                      if (editorWidth) {
-                        var payload = (0, _analytics2.getWidthInfoPayload)(editorViewRef.current, editorWidth);
-                        if (payload) {
-                          dispatchAnalyticsEvent(payload);
-                        }
+
+              // send statistics about the widths of the tables on the page for alerting
+              // only send this event once, after the editorView is first initialised
+              var setTimeoutID = setTimeout(function () {
+                var requestIdleCallbackFn = function requestIdleCallbackFn() {
+                  var _api$width$sharedStat2;
+                  var editorWidth = api === null || api === void 0 || (_api$width$sharedStat2 = api.width.sharedState.currentState()) === null || _api$width$sharedStat2 === void 0 ? void 0 : _api$width$sharedStat2.width;
+                  if (editorViewRef.current) {
+                    if (editorWidth) {
+                      var payload = (0, _analytics2.getWidthInfoPayload)(editorViewRef.current, editorWidth);
+                      if (payload) {
+                        dispatchAnalyticsEvent(payload);
                       }
-                      if ((0, _platformFeatureFlags.fg)('platform_editor_table_height_analytics_event')) {
-                        var payloadHeight = (0, _analytics2.getHeightInfoPayload)(editorViewRef.current);
-                        if (payloadHeight) {
-                          dispatchAnalyticsEvent(payloadHeight);
-                        }
+                    }
+                    if ((0, _platformFeatureFlags.fg)('platform_editor_table_height_analytics_event')) {
+                      var payloadHeight = (0, _analytics2.getHeightInfoPayload)(editorViewRef.current);
+                      if (payloadHeight) {
+                        dispatchAnalyticsEvent(payloadHeight);
                       }
                     }
-                  };
-                  if (window && typeof window.requestIdleCallback === 'function') {
-                    ricID = window.requestIdleCallback(requestIdleCallbackFn);
-                  } else if (window && typeof window.requestAnimationFrame === 'function') {
-                    // requestIdleCallback is not supported in safari, fallback to requestAnimationFrame
-                    rafID = window.requestAnimationFrame(requestIdleCallbackFn);
                   }
-                }, TABLE_WIDTH_INFO_TIMEOUT);
-              }
+                };
+                if (window && typeof window.requestIdleCallback === 'function') {
+                  ricID = window.requestIdleCallback(requestIdleCallbackFn);
+                } else if (window && typeof window.requestAnimationFrame === 'function') {
+                  // requestIdleCallback is not supported in safari, fallback to requestAnimationFrame
+                  rafID = window.requestAnimationFrame(requestIdleCallbackFn);
+                }
+              }, TABLE_WIDTH_INFO_TIMEOUT);
               return {
                 destroy: function destroy() {
                   editorViewRef.current = null;

package/dist/es2019/nodeviews/TableContainer.js

@@ -20,10 +20,8 @@ const InnerContainer = /*#__PURE__*/forwardRef(({
   className,
   style,
   node,
-  children,
-  tableWrapperHeight
+  children
 }, ref) => {
-  const bordersReady = expValEquals('platform_editor_vc90_transition_table_border', 'isEnabled', true) ? tableWrapperHeight !== undefined && tableWrapperHeight > 0 : undefined;
   return /*#__PURE__*/React.createElement("div", {
     ref: ref
     // eslint-disable-next-line @atlaskit/ui-styling-standard/enforce-style-prop -- Ignored via go/DSP-18766
@@ -34,7 +32,6 @@ const InnerContainer = /*#__PURE__*/forwardRef(({
     className: className,
     "data-number-column": node.attrs.isNumberColumnEnabled,
     "data-layout": node.attrs.layout,
-    "data-borders-ready": bordersReady,
     "data-testid": "table-container"
   }, children);
 });
@@ -339,8 +336,7 @@ export const ResizableTableContainer = /*#__PURE__*/React.memo(({
   // eslint-disable-next-line @atlaskit/ui-styling-standard/no-classname-prop -- Ignored via go/DSP-18766
   , {
     className: className,
-    node: node,
-    tableWrapperHeight: tableWrapperHeight
+    node: node
   }, children))));
 });
 export const TableContainer = ({
@@ -402,8 +398,7 @@ export const TableContainer = ({
   const isFullPageAppearance = !isCommentEditor && !isChromelessEditor;
   const resizableTableWidth = isFullPageAppearance ? getTableResizerContainerForFullPageWidthInCSS(node, isTableScalingEnabled) : `calc(100cqw - calc(var(--ak-editor--large-gutter-padding) * 2))`;
   return /*#__PURE__*/React.createElement(InnerContainer, {
-    node: node,
-    tableWrapperHeight: tableWrapperHeight
+    node: node
     // eslint-disable-next-line @atlaskit/ui-styling-standard/no-classname-prop -- Ignored via go/DSP-18766
     ,
     className: classNames(className, {

package/dist/es2019/nodeviews/TableRow.js

@@ -5,7 +5,6 @@ import { getParentOfTypeCount } from '@atlaskit/editor-common/nesting';
 import { nodeVisibilityManager } from '@atlaskit/editor-common/node-visibility';
 import { findOverflowScrollParent } from '@atlaskit/editor-common/ui';
 import { findParentNodeClosestToPos } from '@atlaskit/editor-prosemirror/utils';
-import { fg } from '@atlaskit/platform-feature-flags';
 import { getPluginState } from '../pm-plugins/plugin-factory';
 import { pluginKey as tablePluginKey } from '../pm-plugins/plugin-key';
 import { updateStickyState } from '../pm-plugins/sticky-headers/commands';
@@ -86,20 +85,16 @@ export default class TableRow extends TableNodeView {
     }
     const pos = this.getPos();
     this.isInNestedTable = false;
-    if (fg('platform_editor_ai_aifc_patch_ga')) {
-      try {
-        // We cannot trust that the value from getPos will be defined
-        // https://discuss.prosemirror.net/t/getpos-is-undefined-in-nodeview-constructor/1246/4
-        // There are also scenarios where the value it brings back does not tally with the current doc
-        // E.g. when AI streaming brings in new content, this position brings back incorrect values that cannot be resolved
-        if (pos) {
-          this.isInNestedTable = getParentOfTypeCount(view.state.schema.nodes.table)(view.state.doc.resolve(pos)) > 1;
-        }
-      } catch (e) {}
-    } else {
+    try {
+      // We cannot trust that the value from getPos will be defined
+      // https://discuss.prosemirror.net/t/getpos-is-undefined-in-nodeview-constructor/1246/4
+      // There are also scenarios where the value it brings back does not tally with the current doc
+      // E.g. when AI streaming brings in new content, this position brings back incorrect values that cannot be resolved
       if (pos) {
         this.isInNestedTable = getParentOfTypeCount(view.state.schema.nodes.table)(view.state.doc.resolve(pos)) > 1;
       }
+    } catch {
+      // Intentionally swallowed — getPos can return stale positions during AI streaming
     }
     if (this.isHeaderRow) {
       this.dom.setAttribute('data-vc-nvs', 'true');

package/dist/es2019/nodeviews/toDOM.js

@@ -46,7 +46,16 @@ export const tableNodeSpecWithFixedToDOM = config => {
       }, ['table', attrs, colgroup, ['tbody', 0]]], ['div', {
         class: 'pm-table-sticky-sentinel-bottom',
         'data-testid': 'sticky-sentinel-bottom'
-      }]];
+      }], ...(expValEquals('platform_editor_vc90_transition_table_border', 'isEnabled', true) ? [['div', {
+        contenteditable: 'false',
+        class: 'pm-table-left-border',
+        'data-with-numbered-table': node.attrs.isNumberColumnEnabled,
+        'data-testid': 'table-left-border'
+      }], ['div', {
+        contenteditable: 'false',
+        class: 'pm-table-right-border',
+        'data-testid': 'table-right-border'
+      }]] : [])];
       if (!config.tableResizingEnabled || config.isNested) {
         return ['div', {
           class: 'tableView-content-wrap',

package/dist/es2019/pm-plugins/utils/decoration.js

@@ -9,6 +9,7 @@ import { nonNullable } from '@atlaskit/editor-common/utils';
 import { Decoration } from '@atlaskit/editor-prosemirror/view';
 import { TableMap } from '@atlaskit/editor-tables/table-map';
 import { findTable, getCellsInRow, getSelectionRect } from '@atlaskit/editor-tables/utils';
+import { expValEquals } from '@atlaskit/tmp-editor-statsig/exp-val-equals';
 import { TableCssClassName as ClassName, TableDecorations } from '../../types';
 import { ColumnResizeWidget } from '../../ui/ColumnResizeWidget';
 const filterDecorationByKey = (key, decorationSet) => decorationSet.find(undefined, undefined, spec => spec.key.indexOf(key) > -1);
@@ -254,12 +255,17 @@ export const createResizeHandleDecoration = (tr, rowIndexTarget, columnEndIndexT
       return element;
     }, {
       key: `${TableDecorations.COLUMN_RESIZING_HANDLE_WIDGET}_${rowIndex}_${columnIndex}_${includeTooltip ? 'with' : 'no'}-tooltip`,
-      destroy: node => {
+      destroy: _node => {
         nodeViewPortalProviderAPI.remove(decorationRenderKey);
       }
     });
   };
   const createLastCellElementDecoration = (cellColumnPositioning, cellPos, cellNode) => {
+    if (expValEquals('platform_editor_table_remove_last_cell_decoration', 'isEnabled', true)) {
+      // no longer need to add the last cell decoration to override marginBottom as media wrapper doesn't have margin bottom. This will avoid unnecessary decoration computation/mutation and improve performance
+      // consider clean up ClassName.LAST_ITEM_IN_CELL with platform_editor_table_remove_last_cell_decoration experiment
+      return null;
+    }
     let lastItemPositions;
     cellNode.forEach((childNode, offset, index) => {
       if (index === cellNode.childCount - 1) {

package/dist/es2019/tablePlugin.js

@@ -423,39 +423,37 @@ const tablePlugin = ({
           return new SafePlugin({
             view: editorView => {
               editorViewRef.current = editorView;
-              let setTimeoutID;
               let rafID;
               let ricID;
-              if (expValEquals('platform_editor_editor_width_analytics', 'isEnabled', true)) {
-                // send statistics about the widths of the tables on the page for alerting
-                // only send this event once, after the editorView is first initialised
-                setTimeoutID = setTimeout(() => {
-                  const requestIdleCallbackFn = () => {
-                    var _api$width$sharedStat2;
-                    const editorWidth = api === null || api === void 0 ? void 0 : (_api$width$sharedStat2 = api.width.sharedState.currentState()) === null || _api$width$sharedStat2 === void 0 ? void 0 : _api$width$sharedStat2.width;
-                    if (editorViewRef.current) {
-                      if (editorWidth) {
-                        const payload = getWidthInfoPayload(editorViewRef.current, editorWidth);
-                        if (payload) {
-                          dispatchAnalyticsEvent(payload);
-                        }
+
+              // send statistics about the widths of the tables on the page for alerting
+              // only send this event once, after the editorView is first initialised
+              const setTimeoutID = setTimeout(() => {
+                const requestIdleCallbackFn = () => {
+                  var _api$width$sharedStat2;
+                  const editorWidth = api === null || api === void 0 ? void 0 : (_api$width$sharedStat2 = api.width.sharedState.currentState()) === null || _api$width$sharedStat2 === void 0 ? void 0 : _api$width$sharedStat2.width;
+                  if (editorViewRef.current) {
+                    if (editorWidth) {
+                      const payload = getWidthInfoPayload(editorViewRef.current, editorWidth);
+                      if (payload) {
+                        dispatchAnalyticsEvent(payload);
                       }
-                      if (fg('platform_editor_table_height_analytics_event')) {
-                        const payloadHeight = getHeightInfoPayload(editorViewRef.current);
-                        if (payloadHeight) {
-                          dispatchAnalyticsEvent(payloadHeight);
-                        }
+                    }
+                    if (fg('platform_editor_table_height_analytics_event')) {
+                      const payloadHeight = getHeightInfoPayload(editorViewRef.current);
+                      if (payloadHeight) {
+                        dispatchAnalyticsEvent(payloadHeight);
                       }
                     }
-                  };
-                  if (window && typeof window.requestIdleCallback === 'function') {
-                    ricID = window.requestIdleCallback(requestIdleCallbackFn);
-                  } else if (window && typeof window.requestAnimationFrame === 'function') {
-                    // requestIdleCallback is not supported in safari, fallback to requestAnimationFrame
-                    rafID = window.requestAnimationFrame(requestIdleCallbackFn);
                   }
-                }, TABLE_WIDTH_INFO_TIMEOUT);
-              }
+                };
+                if (window && typeof window.requestIdleCallback === 'function') {
+                  ricID = window.requestIdleCallback(requestIdleCallbackFn);
+                } else if (window && typeof window.requestAnimationFrame === 'function') {
+                  // requestIdleCallback is not supported in safari, fallback to requestAnimationFrame
+                  rafID = window.requestAnimationFrame(requestIdleCallbackFn);
+                }
+              }, TABLE_WIDTH_INFO_TIMEOUT);
               return {
                 destroy: () => {
                   editorViewRef.current = null;

package/dist/esm/nodeviews/TableContainer.js

@@ -21,9 +21,7 @@ var InnerContainer = /*#__PURE__*/forwardRef(function (_ref, ref) {
   var className = _ref.className,
     style = _ref.style,
     node = _ref.node,
-    children = _ref.children,
-    tableWrapperHeight = _ref.tableWrapperHeight;
-  var bordersReady = expValEquals('platform_editor_vc90_transition_table_border', 'isEnabled', true) ? tableWrapperHeight !== undefined && tableWrapperHeight > 0 : undefined;
+    children = _ref.children;
   return /*#__PURE__*/React.createElement("div", {
     ref: ref
     // eslint-disable-next-line @atlaskit/ui-styling-standard/enforce-style-prop -- Ignored via go/DSP-18766
@@ -34,7 +32,6 @@ var InnerContainer = /*#__PURE__*/forwardRef(function (_ref, ref) {
     className: className,
     "data-number-column": node.attrs.isNumberColumnEnabled,
     "data-layout": node.attrs.layout,
-    "data-borders-ready": bordersReady,
     "data-testid": "table-container"
   }, children);
 });
@@ -336,8 +333,7 @@ export var ResizableTableContainer = /*#__PURE__*/React.memo(function (_ref4) {
   // eslint-disable-next-line @atlaskit/ui-styling-standard/no-classname-prop -- Ignored via go/DSP-18766
   , {
     className: className,
-    node: node,
-    tableWrapperHeight: tableWrapperHeight
+    node: node
   }, children))));
 });
 export var TableContainer = function TableContainer(_ref5) {
@@ -396,8 +392,7 @@ export var TableContainer = function TableContainer(_ref5) {
   var isFullPageAppearance = !isCommentEditor && !isChromelessEditor;
   var resizableTableWidth = isFullPageAppearance ? getTableResizerContainerForFullPageWidthInCSS(node, isTableScalingEnabled) : "calc(100cqw - calc(var(--ak-editor--large-gutter-padding) * 2))";
   return /*#__PURE__*/React.createElement(InnerContainer, {
-    node: node,
-    tableWrapperHeight: tableWrapperHeight
+    node: node
     // eslint-disable-next-line @atlaskit/ui-styling-standard/no-classname-prop -- Ignored via go/DSP-18766
     ,
     className: classNames(className, {

package/dist/esm/nodeviews/TableRow.js

@@ -12,7 +12,6 @@ import { getParentOfTypeCount } from '@atlaskit/editor-common/nesting';
 import { nodeVisibilityManager } from '@atlaskit/editor-common/node-visibility';
 import { findOverflowScrollParent } from '@atlaskit/editor-common/ui';
 import { findParentNodeClosestToPos } from '@atlaskit/editor-prosemirror/utils';
-import { fg } from '@atlaskit/platform-feature-flags';
 import { getPluginState } from '../pm-plugins/plugin-factory';
 import { pluginKey as tablePluginKey } from '../pm-plugins/plugin-key';
 import { updateStickyState } from '../pm-plugins/sticky-headers/commands';
@@ -94,20 +93,16 @@ var TableRow = /*#__PURE__*/function (_TableNodeView) {
     }
     var pos = _this.getPos();
     _this.isInNestedTable = false;
-    if (fg('platform_editor_ai_aifc_patch_ga')) {
-      try {
-        // We cannot trust that the value from getPos will be defined
-        // https://discuss.prosemirror.net/t/getpos-is-undefined-in-nodeview-constructor/1246/4
-        // There are also scenarios where the value it brings back does not tally with the current doc
-        // E.g. when AI streaming brings in new content, this position brings back incorrect values that cannot be resolved
-        if (pos) {
-          _this.isInNestedTable = getParentOfTypeCount(view.state.schema.nodes.table)(view.state.doc.resolve(pos)) > 1;
-        }
-      } catch (e) {}
-    } else {
+    try {
+      // We cannot trust that the value from getPos will be defined
+      // https://discuss.prosemirror.net/t/getpos-is-undefined-in-nodeview-constructor/1246/4
+      // There are also scenarios where the value it brings back does not tally with the current doc
+      // E.g. when AI streaming brings in new content, this position brings back incorrect values that cannot be resolved
       if (pos) {
         _this.isInNestedTable = getParentOfTypeCount(view.state.schema.nodes.table)(view.state.doc.resolve(pos)) > 1;
       }
+    } catch (_unused) {
+      // Intentionally swallowed — getPos can return stale positions during AI streaming
     }
     if (_this.isHeaderRow) {
       _this.dom.setAttribute('data-vc-nvs', 'true');

package/dist/esm/nodeviews/toDOM.js

@@ -55,7 +55,16 @@ export var tableNodeSpecWithFixedToDOM = function tableNodeSpecWithFixedToDOM(co
       }, ['table', attrs, colgroup, ['tbody', 0]]], ['div', {
         class: 'pm-table-sticky-sentinel-bottom',
         'data-testid': 'sticky-sentinel-bottom'
-      }]];
+      }]].concat(_toConsumableArray(expValEquals('platform_editor_vc90_transition_table_border', 'isEnabled', true) ? [['div', {
+        contenteditable: 'false',
+        class: 'pm-table-left-border',
+        'data-with-numbered-table': node.attrs.isNumberColumnEnabled,
+        'data-testid': 'table-left-border'
+      }], ['div', {
+        contenteditable: 'false',
+        class: 'pm-table-right-border',
+        'data-testid': 'table-right-border'
+      }]] : []));
       if (!config.tableResizingEnabled || config.isNested) {
         return ['div', {
           class: 'tableView-content-wrap',

package/dist/esm/pm-plugins/utils/decoration.js

@@ -10,6 +10,7 @@ import { nonNullable } from '@atlaskit/editor-common/utils';
 import { Decoration } from '@atlaskit/editor-prosemirror/view';
 import { TableMap } from '@atlaskit/editor-tables/table-map';
 import { findTable, getCellsInRow, getSelectionRect } from '@atlaskit/editor-tables/utils';
+import { expValEquals } from '@atlaskit/tmp-editor-statsig/exp-val-equals';
 import { TableCssClassName as ClassName, TableDecorations } from '../../types';
 import { ColumnResizeWidget } from '../../ui/ColumnResizeWidget';
 var filterDecorationByKey = function filterDecorationByKey(key, decorationSet) {
@@ -280,12 +281,17 @@ export var createResizeHandleDecoration = function createResizeHandleDecoration(
       return element;
     }, {
       key: "".concat(TableDecorations.COLUMN_RESIZING_HANDLE_WIDGET, "_").concat(rowIndex, "_").concat(columnIndex, "_").concat(includeTooltip ? 'with' : 'no', "-tooltip"),
-      destroy: function destroy(node) {
+      destroy: function destroy(_node) {
         nodeViewPortalProviderAPI.remove(decorationRenderKey);
       }
     });
   };
   var createLastCellElementDecoration = function createLastCellElementDecoration(cellColumnPositioning, cellPos, cellNode) {
+    if (expValEquals('platform_editor_table_remove_last_cell_decoration', 'isEnabled', true)) {
+      // no longer need to add the last cell decoration to override marginBottom as media wrapper doesn't have margin bottom. This will avoid unnecessary decoration computation/mutation and improve performance
+      // consider clean up ClassName.LAST_ITEM_IN_CELL with platform_editor_table_remove_last_cell_decoration experiment
+      return null;
+    }
     var lastItemPositions;
     cellNode.forEach(function (childNode, offset, index) {
       if (index === cellNode.childCount - 1) {

package/dist/esm/tablePlugin.js

@@ -424,39 +424,37 @@ var tablePlugin = function tablePlugin(_ref) {
           return new SafePlugin({
             view: function view(editorView) {
               editorViewRef.current = editorView;
-              var setTimeoutID;
               var rafID;
               var ricID;
-              if (expValEquals('platform_editor_editor_width_analytics', 'isEnabled', true)) {
-                // send statistics about the widths of the tables on the page for alerting
-                // only send this event once, after the editorView is first initialised
-                setTimeoutID = setTimeout(function () {
-                  var requestIdleCallbackFn = function requestIdleCallbackFn() {
-                    var _api$width$sharedStat2;
-                    var editorWidth = api === null || api === void 0 || (_api$width$sharedStat2 = api.width.sharedState.currentState()) === null || _api$width$sharedStat2 === void 0 ? void 0 : _api$width$sharedStat2.width;
-                    if (editorViewRef.current) {
-                      if (editorWidth) {
-                        var payload = getWidthInfoPayload(editorViewRef.current, editorWidth);
-                        if (payload) {
-                          dispatchAnalyticsEvent(payload);
-                        }
+
+              // send statistics about the widths of the tables on the page for alerting
+              // only send this event once, after the editorView is first initialised
+              var setTimeoutID = setTimeout(function () {
+                var requestIdleCallbackFn = function requestIdleCallbackFn() {
+                  var _api$width$sharedStat2;
+                  var editorWidth = api === null || api === void 0 || (_api$width$sharedStat2 = api.width.sharedState.currentState()) === null || _api$width$sharedStat2 === void 0 ? void 0 : _api$width$sharedStat2.width;
+                  if (editorViewRef.current) {
+                    if (editorWidth) {
+                      var payload = getWidthInfoPayload(editorViewRef.current, editorWidth);
+                      if (payload) {
+                        dispatchAnalyticsEvent(payload);
                       }
-                      if (fg('platform_editor_table_height_analytics_event')) {
-                        var payloadHeight = getHeightInfoPayload(editorViewRef.current);
-                        if (payloadHeight) {
-                          dispatchAnalyticsEvent(payloadHeight);
-                        }
+                    }
+                    if (fg('platform_editor_table_height_analytics_event')) {
+                      var payloadHeight = getHeightInfoPayload(editorViewRef.current);
+                      if (payloadHeight) {
+                        dispatchAnalyticsEvent(payloadHeight);
                       }
                     }
-                  };
-                  if (window && typeof window.requestIdleCallback === 'function') {
-                    ricID = window.requestIdleCallback(requestIdleCallbackFn);
-                  } else if (window && typeof window.requestAnimationFrame === 'function') {
-                    // requestIdleCallback is not supported in safari, fallback to requestAnimationFrame
-                    rafID = window.requestAnimationFrame(requestIdleCallbackFn);
                   }
-                }, TABLE_WIDTH_INFO_TIMEOUT);
-              }
+                };
+                if (window && typeof window.requestIdleCallback === 'function') {
+                  ricID = window.requestIdleCallback(requestIdleCallbackFn);
+                } else if (window && typeof window.requestAnimationFrame === 'function') {
+                  // requestIdleCallback is not supported in safari, fallback to requestAnimationFrame
+                  rafID = window.requestAnimationFrame(requestIdleCallbackFn);
+                }
+              }, TABLE_WIDTH_INFO_TIMEOUT);
               return {
                 destroy: function destroy() {
                   editorViewRef.current = null;

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@atlaskit/editor-plugin-table",
-	"version": "17.7.0",
+	"version": "17.7.2",
 	"description": "Table plugin for the @atlaskit/editor",
 	"publishConfig": {
 		"registry": "https://registry.npmjs.org/"
@@ -37,11 +37,11 @@
 		"@atlaskit/editor-plugin-batch-attribute-updates": "^7.0.0",
 		"@atlaskit/editor-plugin-content-insertion": "^7.0.0",
 		"@atlaskit/editor-plugin-editor-viewmode": "^9.0.0",
-		"@atlaskit/editor-plugin-extension": "10.1.17",
+		"@atlaskit/editor-plugin-extension": "10.1.18",
 		"@atlaskit/editor-plugin-guideline": "^7.0.0",
 		"@atlaskit/editor-plugin-interaction": "^14.0.0",
 		"@atlaskit/editor-plugin-limited-mode": "^4.0.0",
-		"@atlaskit/editor-plugin-selection": "^7.0.0",
+		"@atlaskit/editor-plugin-selection": "^7.1.0",
 		"@atlaskit/editor-plugin-toolbar": "^4.1.0",
 		"@atlaskit/editor-plugin-user-intent": "^5.0.0",
 		"@atlaskit/editor-plugin-width": "^8.1.0",
@@ -57,9 +57,9 @@
 		"@atlaskit/pragmatic-drag-and-drop-hitbox": "^1.1.0",
 		"@atlaskit/primitives": "^18.0.0",
 		"@atlaskit/theme": "^22.0.0",
-		"@atlaskit/tmp-editor-statsig": "^34.1.0",
+		"@atlaskit/tmp-editor-statsig": "^35.5.0",
 		"@atlaskit/toggle": "^15.2.0",
-		"@atlaskit/tokens": "^11.0.0",
+		"@atlaskit/tokens": "^11.1.0",
 		"@atlaskit/tooltip": "^20.14.0",
 		"@babel/runtime": "^7.0.0",
 		"@emotion/react": "^11.7.1",
@@ -70,7 +70,7 @@
 		"uuid": "^3.1.0"
 	},
 	"peerDependencies": {
-		"@atlaskit/editor-common": "^111.28.0",
+		"@atlaskit/editor-common": "^111.32.0",
 		"react": "^18.2.0",
 		"react-dom": "^18.2.0",
 		"react-intl-next": "npm:react-intl@^5.18.1"
@@ -136,9 +136,6 @@
 		"platform_editor_table_height_analytics_event": {
 			"type": "boolean"
 		},
-		"platform_editor_ai_aifc_patch_ga": {
-			"type": "boolean"
-		},
 		"platform_editor_change_table_nesting_check": {
 			"type": "boolean"
 		},