@atlaskit/editor-plugin-table 7.25.3 → 7.25.5

package/src/toolbar.tsx

@@ -197,6 +197,7 @@ export const getToolbarCellOptionsConfig = (
 	isCellBackgroundDuplicated = false,
 	isTableFixedColumnWidthsOptionEnabled = false,
 	shouldUseIncreasedScalingPercent = false,
+	isCommentEditor = false,
 ): FloatingToolbarDropdown<Command> => {
 	const { top, bottom, right, left } = initialSelectionRect;
 	const numberOfColumns = right - left;
@@ -322,6 +323,7 @@ export const getToolbarCellOptionsConfig = (
 					getEditorContainerWidth,
 					isTableScalingEnabled,
 					isTableFixedColumnWidthsOptionEnabled,
+					isCommentEditor,
 				)
 			: undefined;
 		const wouldChange = newResizeStateWithAnalytics?.changed ?? false;
@@ -549,6 +551,7 @@ export const getToolbarConfig =
 						isCellBackgroundDuplicated,
 						isTableFixedColumnWidthsOptionEnabled,
 						shouldUseIncreasedScalingPercent,
+						options?.isCommentEditor,
 					);
 
 			const columnSettingsItems = pluginState.isDragAndDropEnabled
@@ -560,6 +563,7 @@ export const getToolbarConfig =
 						editorAnalyticsAPI,
 						isTableScalingEnabled,
 						isTableFixedColumnWidthsOptionEnabled,
+						options?.isCommentEditor,
 					)
 				: [];
 			const colorPicker = getColorPicker(state, menu, intl, editorAnalyticsAPI, getEditorView);
@@ -661,6 +665,7 @@ const getCellItems = (
 	isCellBackgroundDuplicated = false,
 	isTableFixedColumnWidthsOptionEnabled = false,
 	shouldUseIncreasedScalingPercent = false,
+	isCommentEditor = false,
 ): Array<FloatingToolbarItem<Command>> => {
 	const initialSelectionRect = getClosestSelectionRect(state);
 	if (initialSelectionRect) {
@@ -675,6 +680,7 @@ const getCellItems = (
 			isCellBackgroundDuplicated,
 			isTableFixedColumnWidthsOptionEnabled,
 			shouldUseIncreasedScalingPercent,
+			isCommentEditor,
 		);
 		return [cellOptions, separator(cellOptions.hidden!)];
 	}
@@ -687,6 +693,7 @@ export const getDistributeConfig =
 		editorAnalyticsAPI: EditorAnalyticsAPI | undefined | null,
 		isTableScalingEnabled = false,
 		isTableFixedColumnWidthsOptionEnabled = false,
+		isCommentEditor = false,
 	): Command =>
 	(state, dispatch, editorView) => {
 		const selectionOrTableRect = getClosestSelectionOrTableRect(state);
@@ -701,6 +708,7 @@ export const getDistributeConfig =
 			getEditorContainerWidth,
 			isTableScalingEnabled,
 			isTableFixedColumnWidthsOptionEnabled,
+			isCommentEditor,
 		);
 
 		if (newResizeStateWithAnalytics) {
@@ -723,13 +731,13 @@ const getColumnSettingItems = (
 	editorAnalyticsAPI: EditorAnalyticsAPI | undefined | null,
 	isTableScalingEnabled = false,
 	isTableFixedColumnWidthsOptionEnabled = false,
+	isCommentEditor = false,
 ): Array<FloatingToolbarItem<Command>> => {
 	const pluginState = getPluginState(editorState);
 	const selectionOrTableRect = getClosestSelectionOrTableRect(editorState);
 	if (!selectionOrTableRect || !editorView) {
 		return [];
 	}
-
 	const newResizeStateWithAnalytics = getNewResizeStateFromSelectedColumns(
 		selectionOrTableRect,
 		editorState,
@@ -755,6 +763,7 @@ const getColumnSettingItems = (
 					editorAnalyticsAPI,
 					isTableScalingEnabled,
 					isTableFixedColumnWidthsOptionEnabled,
+					isCommentEditor,
 				)(state, dispatch, view),
 			disabled: !wouldChange,
 		});

package/src/ui/DragHandle/index.tsx

@@ -72,7 +72,7 @@ const DragHandleComponent = ({
 		state: { selection },
 	} = editorView;
 	const { hoveredColumns, hoveredRows } = getPluginState(state);
-	const { isDragMenuOpen } = getDnDPluginState(state);
+	const { isDragMenuOpen = false } = getDnDPluginState(state);
 
 	const isRow = direction === 'row';
 	const isColumn = direction === 'column';

package/src/ui/FloatingContextualMenu/ContextualMenu.tsx

@@ -101,6 +101,7 @@ export interface Props {
 	getEditorContainerWidth: GetEditorContainerWidth;
 	getEditorFeatureFlags?: GetEditorFeatureFlags;
 	isCellMenuOpenByKeyboard?: boolean;
+	isCommentEditor?: boolean;
 }
 
 export interface State {
@@ -651,6 +652,7 @@ export class ContextualMenu extends Component<Props & WrappedComponentProps, Sta
 			getEditorContainerWidth,
 			getEditorFeatureFlags,
 			isCellMenuOpenByKeyboard,
+			isCommentEditor,
 		} = this.props;
 		// TargetCellPosition could be outdated: https://product-fabric.atlassian.net/browse/ED-8129
 		const { state, dispatch } = editorView;
@@ -679,8 +681,10 @@ export class ContextualMenu extends Component<Props & WrappedComponentProps, Sta
 
 		const shouldUseIncreasedScalingPercent =
 			isTableScalingEnabled &&
-			tableWithFixedColumnWidthsOption &&
-			fg('platform.editor.table.use-increased-scaling-percent');
+			((tableWithFixedColumnWidthsOption &&
+				fg('platform.editor.table.use-increased-scaling-percent')) ||
+				// When in comment editor, we need the scaling percent to be 40% while tableWithFixedColumnWidthsOption is not visible
+				isCommentEditor);
 
 		switch (item.value.name) {
 			case 'sort_column_desc':

package/src/ui/FloatingContextualMenu/index.tsx

@@ -44,6 +44,7 @@ export interface Props {
 	pluginConfig?: PluginConfig;
 	editorAnalyticsAPI?: EditorAnalyticsAPI;
 	isCellMenuOpenByKeyboard?: boolean;
+	isCommentEditor?: boolean;
 }
 
 const FloatingContextualMenu = ({
@@ -57,6 +58,7 @@ const FloatingContextualMenu = ({
 	getEditorContainerWidth,
 	getEditorFeatureFlags,
 	isCellMenuOpenByKeyboard,
+	isCommentEditor,
 }: Props) => {
 	// TargetCellPosition could be outdated: https://product-fabric.atlassian.net/browse/ED-8129
 	const { targetCellPosition, isDragAndDropEnabled } = getPluginState(editorView.state);
@@ -114,6 +116,7 @@ const FloatingContextualMenu = ({
 					getEditorContainerWidth={getEditorContainerWidth}
 					getEditorFeatureFlags={getEditorFeatureFlags}
 					isCellMenuOpenByKeyboard={isCellMenuOpenByKeyboard}
+					isCommentEditor={isCommentEditor}
 				/>
 			</div>
 		</Popup>

package/dist/cjs/nodeviews/ignore-mutation-delegate.js

@@ -1,23 +0,0 @@
-"use strict";
-
-Object.defineProperty(exports, "__esModule", {
-  value: true
-});
-exports.default = void 0;
-var _default = exports.default = function _default(mutation) {
-  var type = mutation.type,
-    _mutation$target = mutation.target,
-    nodeName = _mutation$target.nodeName,
-    firstChild = _mutation$target.firstChild;
-  if (type === 'selection' && (nodeName === null || nodeName === void 0 ? void 0 : nodeName.toUpperCase()) === 'DIV' && (firstChild === null || firstChild === void 0 ? void 0 : firstChild.nodeName.toUpperCase()) === 'TABLE') {
-    return false;
-  }
-
-  // ED-16668
-  // Do not remove this fixes an issue with windows firefox that relates to
-  // the addition of the shadow sentinels
-  if (type === 'selection' && (nodeName === null || nodeName === void 0 ? void 0 : nodeName.toUpperCase()) === 'TABLE' && ((firstChild === null || firstChild === void 0 ? void 0 : firstChild.nodeName.toUpperCase()) === 'COLGROUP' || (firstChild === null || firstChild === void 0 ? void 0 : firstChild.nodeName.toUpperCase()) === 'SPAN')) {
-    return false;
-  }
-  return true;
-};

package/dist/es2019/nodeviews/ignore-mutation-delegate.js

@@ -1,20 +0,0 @@
-export default (mutation => {
-  const {
-    type,
-    target: {
-      nodeName,
-      firstChild
-    }
-  } = mutation;
-  if (type === 'selection' && (nodeName === null || nodeName === void 0 ? void 0 : nodeName.toUpperCase()) === 'DIV' && (firstChild === null || firstChild === void 0 ? void 0 : firstChild.nodeName.toUpperCase()) === 'TABLE') {
-    return false;
-  }
-
-  // ED-16668
-  // Do not remove this fixes an issue with windows firefox that relates to
-  // the addition of the shadow sentinels
-  if (type === 'selection' && (nodeName === null || nodeName === void 0 ? void 0 : nodeName.toUpperCase()) === 'TABLE' && ((firstChild === null || firstChild === void 0 ? void 0 : firstChild.nodeName.toUpperCase()) === 'COLGROUP' || (firstChild === null || firstChild === void 0 ? void 0 : firstChild.nodeName.toUpperCase()) === 'SPAN')) {
-    return false;
-  }
-  return true;
-});

package/dist/esm/nodeviews/ignore-mutation-delegate.js

@@ -1,17 +0,0 @@
-export default (function (mutation) {
-  var type = mutation.type,
-    _mutation$target = mutation.target,
-    nodeName = _mutation$target.nodeName,
-    firstChild = _mutation$target.firstChild;
-  if (type === 'selection' && (nodeName === null || nodeName === void 0 ? void 0 : nodeName.toUpperCase()) === 'DIV' && (firstChild === null || firstChild === void 0 ? void 0 : firstChild.nodeName.toUpperCase()) === 'TABLE') {
-    return false;
-  }
-
-  // ED-16668
-  // Do not remove this fixes an issue with windows firefox that relates to
-  // the addition of the shadow sentinels
-  if (type === 'selection' && (nodeName === null || nodeName === void 0 ? void 0 : nodeName.toUpperCase()) === 'TABLE' && ((firstChild === null || firstChild === void 0 ? void 0 : firstChild.nodeName.toUpperCase()) === 'COLGROUP' || (firstChild === null || firstChild === void 0 ? void 0 : firstChild.nodeName.toUpperCase()) === 'SPAN')) {
-    return false;
-  }
-  return true;
-});

package/dist/types/nodeviews/ignore-mutation-delegate.d.ts

@@ -1,5 +0,0 @@
-declare const _default: (mutation: MutationRecord | {
-    type: 'selection';
-    target: Element;
-}) => boolean;
-export default _default;

package/dist/types-ts4.5/nodeviews/ignore-mutation-delegate.d.ts

@@ -1,5 +0,0 @@
-declare const _default: (mutation: MutationRecord | {
-    type: 'selection';
-    target: Element;
-}) => boolean;
-export default _default;

package/src/nodeviews/ignore-mutation-delegate.ts

@@ -1,28 +0,0 @@
-export default (mutation: MutationRecord | { type: 'selection'; target: Element }) => {
-	const {
-		type,
-		target: { nodeName, firstChild },
-	} = mutation;
-
-	if (
-		type === 'selection' &&
-		nodeName?.toUpperCase() === 'DIV' &&
-		firstChild?.nodeName.toUpperCase() === 'TABLE'
-	) {
-		return false;
-	}
-
-	// ED-16668
-	// Do not remove this fixes an issue with windows firefox that relates to
-	// the addition of the shadow sentinels
-	if (
-		type === 'selection' &&
-		nodeName?.toUpperCase() === 'TABLE' &&
-		(firstChild?.nodeName.toUpperCase() === 'COLGROUP' ||
-			firstChild?.nodeName.toUpperCase() === 'SPAN')
-	) {
-		return false;
-	}
-
-	return true;
-};