@atlaskit/editor-plugin-table 7.16.11 → 7.16.13

package/src/ui/ColumnResizeWidget/index.tsx

@@ -2,61 +2,54 @@
 import { jsx } from '@emotion/react';
 import { useIntl } from 'react-intl-next';
 
-import {
-  startColumnResizing,
-  ToolTipContent,
-} from '@atlaskit/editor-common/keymaps';
+import { startColumnResizing, ToolTipContent } from '@atlaskit/editor-common/keymaps';
 import { tableMessages as messages } from '@atlaskit/editor-common/messages';
 import Tooltip from '@atlaskit/tooltip';
 
 import { TableCssClassName } from '../../types';
 
 type Props = {
-  startIndex: number;
-  endIndex: number;
-  includeTooltip?: boolean;
+	startIndex: number;
+	endIndex: number;
+	includeTooltip?: boolean;
 };
 
-export const ColumnResizeWidget = ({
-  startIndex,
-  endIndex,
-  includeTooltip,
-}: Props) => {
-  const { formatMessage } = useIntl();
+export const ColumnResizeWidget = ({ startIndex, endIndex, includeTooltip }: Props) => {
+	const { formatMessage } = useIntl();
 
-  if (!includeTooltip) {
-    return (
-      <div
-// eslint-disable-next-line @atlaskit/ui-styling-standard/no-classname-prop -- Ignored via go/DSP-18766
-        className={TableCssClassName.RESIZE_HANDLE_DECORATION}
-        data-start-index={startIndex}
-        data-end-index={endIndex}
-      />
-    );
-  }
+	if (!includeTooltip) {
+		return (
+			<div
+				// eslint-disable-next-line @atlaskit/ui-styling-standard/no-classname-prop -- Ignored via go/DSP-18766
+				className={TableCssClassName.RESIZE_HANDLE_DECORATION}
+				data-start-index={startIndex}
+				data-end-index={endIndex}
+			/>
+		);
+	}
 
-  return (
-    <Tooltip
-      content={
-        <ToolTipContent
-          description={formatMessage(messages.adjustColumns)}
-          keymap={startColumnResizing}
-        />
-      }
-      hideTooltipOnClick
-      hideTooltipOnMouseDown
-      position="mouse"
-      mousePosition="auto-start"
-    >
-      {(tooltipProps) => (
-        <div
-// eslint-disable-next-line @atlaskit/ui-styling-standard/no-classname-prop -- Ignored via go/DSP-18766
-          className={TableCssClassName.RESIZE_HANDLE_DECORATION}
-          data-start-index={startIndex}
-          data-end-index={endIndex}
-          {...tooltipProps}
-        />
-      )}
-    </Tooltip>
-  );
+	return (
+		<Tooltip
+			content={
+				<ToolTipContent
+					description={formatMessage(messages.adjustColumns)}
+					keymap={startColumnResizing}
+				/>
+			}
+			hideTooltipOnClick
+			hideTooltipOnMouseDown
+			position="mouse"
+			mousePosition="auto-start"
+		>
+			{(tooltipProps) => (
+				<div
+					// eslint-disable-next-line @atlaskit/ui-styling-standard/no-classname-prop -- Ignored via go/DSP-18766
+					className={TableCssClassName.RESIZE_HANDLE_DECORATION}
+					data-start-index={startIndex}
+					data-end-index={endIndex}
+					{...tooltipProps}
+				/>
+			)}
+		</Tooltip>
+	);
 };

package/src/ui/DragHandle/HandleIconComponent.tsx

@@ -1,23 +1,19 @@
 import React from 'react';
 
-import {
-  DragHandleDisabledIcon,
-  DragHandleIcon,
-  MinimisedHandleIcon,
-} from '../icons';
+import { DragHandleDisabledIcon, DragHandleIcon, MinimisedHandleIcon } from '../icons';
 
 type HandleIconProps = {
-  forceDefaultHandle: boolean;
-  isHandleHovered: boolean;
-  hasMergedCells: boolean;
+	forceDefaultHandle: boolean;
+	isHandleHovered: boolean;
+	hasMergedCells: boolean;
 };
 
 export const HandleIconComponent = (props: HandleIconProps) => {
-  const { forceDefaultHandle, isHandleHovered, hasMergedCells } = props;
+	const { forceDefaultHandle, isHandleHovered, hasMergedCells } = props;
 
-  if (isHandleHovered || forceDefaultHandle) {
-    return hasMergedCells ? <DragHandleDisabledIcon /> : <DragHandleIcon />;
-  }
+	if (isHandleHovered || forceDefaultHandle) {
+		return hasMergedCells ? <DragHandleDisabledIcon /> : <DragHandleIcon />;
+	}
 
-  return <MinimisedHandleIcon />;
+	return <MinimisedHandleIcon />;
 };

package/src/ui/DragHandle/index.tsx

@@ -26,281 +26,252 @@ import { DragPreview } from '../DragPreview';
 
 import { HandleIconComponent } from './HandleIconComponent';
 
-export type DragHandleAppearance =
-  | 'default'
-  | 'selected'
-  | 'disabled'
-  | 'danger'
-  | 'placeholder';
+export type DragHandleAppearance = 'default' | 'selected' | 'disabled' | 'danger' | 'placeholder';
 
 type DragHandleProps = {
-  tableLocalId: string;
-  indexes: number[];
-  forceDefaultHandle?: boolean;
-  previewWidth?: number;
-  previewHeight?: number;
-  hoveredCell?: CellHoverMeta;
-  direction?: TableDirection;
-  appearance?: DragHandleAppearance;
-  onClick?: MouseEventHandler;
-  onMouseOver?: MouseEventHandler;
-  onMouseOut?: MouseEventHandler;
-  toggleDragMenu?: (
-    trigger: TriggerType,
-    event?: React.MouseEvent<HTMLButtonElement, MouseEvent>,
-  ) => void;
-  editorView: EditorView;
-  isDragMenuTarget: boolean; // this is identify which current handle component is
+	tableLocalId: string;
+	indexes: number[];
+	forceDefaultHandle?: boolean;
+	previewWidth?: number;
+	previewHeight?: number;
+	hoveredCell?: CellHoverMeta;
+	direction?: TableDirection;
+	appearance?: DragHandleAppearance;
+	onClick?: MouseEventHandler;
+	onMouseOver?: MouseEventHandler;
+	onMouseOut?: MouseEventHandler;
+	toggleDragMenu?: (
+		trigger: TriggerType,
+		event?: React.MouseEvent<HTMLButtonElement, MouseEvent>,
+	) => void;
+	editorView: EditorView;
+	isDragMenuTarget: boolean; // this is identify which current handle component is
 };
 
 const DragHandleComponent = ({
-  isDragMenuTarget,
-  tableLocalId,
-  direction = 'row',
-  appearance = 'default',
-  indexes,
-  forceDefaultHandle = false,
-  previewWidth,
-  previewHeight,
-  onMouseOver,
-  onMouseOut,
-  toggleDragMenu,
-  hoveredCell,
-  onClick,
-  editorView,
-  intl: { formatMessage },
+	isDragMenuTarget,
+	tableLocalId,
+	direction = 'row',
+	appearance = 'default',
+	indexes,
+	forceDefaultHandle = false,
+	previewWidth,
+	previewHeight,
+	onMouseOver,
+	onMouseOut,
+	toggleDragMenu,
+	hoveredCell,
+	onClick,
+	editorView,
+	intl: { formatMessage },
 }: DragHandleProps & WrappedComponentProps) => {
-  const dragHandleDivRef = useRef<HTMLButtonElement>(null);
-  const [previewContainer, setPreviewContainer] = useState<HTMLElement | null>(
-    null,
-  );
-  const {
-    state,
-    state: { selection },
-  } = editorView;
-  const { hoveredColumns, hoveredRows } = getPluginState(state);
-  const { isDragMenuOpen } = getDnDPluginState(state);
+	const dragHandleDivRef = useRef<HTMLButtonElement>(null);
+	const [previewContainer, setPreviewContainer] = useState<HTMLElement | null>(null);
+	const {
+		state,
+		state: { selection },
+	} = editorView;
+	const { hoveredColumns, hoveredRows } = getPluginState(state);
+	const { isDragMenuOpen } = getDnDPluginState(state);
 
-  const isRow = direction === 'row';
-  const isColumn = direction === 'column';
+	const isRow = direction === 'row';
+	const isColumn = direction === 'column';
 
-  const isRowHandleHovered = isRow && hoveredRows.length > 0;
-  const isColumnHandleHovered = isColumn && hoveredColumns.length > 0;
+	const isRowHandleHovered = isRow && hoveredRows.length > 0;
+	const isColumnHandleHovered = isColumn && hoveredColumns.length > 0;
 
-  const hasMergedCells = useMemo(() => {
-    const table = findTable(selection);
-    if (!table) {
-      return false;
-    }
+	const hasMergedCells = useMemo(() => {
+		const table = findTable(selection);
+		if (!table) {
+			return false;
+		}
 
-    const map = TableMap.get(table?.node);
+		const map = TableMap.get(table?.node);
 
-    if (!map.hasMergedCells() || indexes.length < 1) {
-      return false;
-    }
+		if (!map.hasMergedCells() || indexes.length < 1) {
+			return false;
+		}
 
-    const { mapByColumn, mapByRow } = map;
+		const { mapByColumn, mapByRow } = map;
 
-    // this handle when hover to first column or row which has merged cells.
-    if (
-      hoveredCell &&
-      hoveredCell.rowIndex !== undefined &&
-      hoveredCell.colIndex !== undefined &&
-      selection instanceof TextSelection
-    ) {
-      const { rowIndex, colIndex } = hoveredCell;
+		// this handle when hover to first column or row which has merged cells.
+		if (
+			hoveredCell &&
+			hoveredCell.rowIndex !== undefined &&
+			hoveredCell.colIndex !== undefined &&
+			selection instanceof TextSelection
+		) {
+			const { rowIndex, colIndex } = hoveredCell;
 
-      const mergedPositionInRow = findDuplicatePosition(mapByRow[rowIndex]);
-      const mergedPositionInCol = findDuplicatePosition(mapByColumn[colIndex]);
+			const mergedPositionInRow = findDuplicatePosition(mapByRow[rowIndex]);
+			const mergedPositionInCol = findDuplicatePosition(mapByColumn[colIndex]);
 
-      const hasMergedCellsInFirstRowOrColumn =
-        direction === 'column'
-          ? mergedPositionInRow.includes(mapByRow[0][colIndex])
-          : mergedPositionInCol.includes(mapByColumn[0][rowIndex]);
+			const hasMergedCellsInFirstRowOrColumn =
+				direction === 'column'
+					? mergedPositionInRow.includes(mapByRow[0][colIndex])
+					: mergedPositionInCol.includes(mapByColumn[0][rowIndex]);
 
-      const isHoveredOnFirstRowOrColumn =
-        direction === 'column'
-          ? hoveredCell.rowIndex === 0 && hasMergedCellsInFirstRowOrColumn
-          : hoveredCell.colIndex === 0 && hasMergedCellsInFirstRowOrColumn;
+			const isHoveredOnFirstRowOrColumn =
+				direction === 'column'
+					? hoveredCell.rowIndex === 0 && hasMergedCellsInFirstRowOrColumn
+					: hoveredCell.colIndex === 0 && hasMergedCellsInFirstRowOrColumn;
 
-      if (isHoveredOnFirstRowOrColumn) {
-        const mergedSizes =
-          direction === 'column'
-            ? mapByRow[0].filter((el: number) => el === mapByRow[0][colIndex])
-                .length
-            : mapByColumn[0].filter(
-                (el: number) => el === mapByColumn[0][rowIndex],
-              ).length;
+			if (isHoveredOnFirstRowOrColumn) {
+				const mergedSizes =
+					direction === 'column'
+						? mapByRow[0].filter((el: number) => el === mapByRow[0][colIndex]).length
+						: mapByColumn[0].filter((el: number) => el === mapByColumn[0][rowIndex]).length;
 
-        const mergedSelection = hasMergedCellsInSelection(
-          direction === 'column'
-            ? [colIndex, colIndex + mergedSizes - 1]
-            : [rowIndex, rowIndex + mergedSizes - 1],
-          direction,
-        )(selection);
+				const mergedSelection = hasMergedCellsInSelection(
+					direction === 'column'
+						? [colIndex, colIndex + mergedSizes - 1]
+						: [rowIndex, rowIndex + mergedSizes - 1],
+					direction,
+				)(selection);
 
-        return mergedSelection;
-      }
-    }
+				return mergedSelection;
+			}
+		}
 
-    return hasMergedCellsInSelection(indexes, direction)(selection);
-  }, [indexes, selection, direction, hoveredCell]);
+		return hasMergedCellsInSelection(indexes, direction)(selection);
+	}, [indexes, selection, direction, hoveredCell]);
 
-  const handleIconProps = {
-    forceDefaultHandle,
-    isHandleHovered: isColumnHandleHovered || isRowHandleHovered,
-    hasMergedCells,
-  };
+	const handleIconProps = {
+		forceDefaultHandle,
+		isHandleHovered: isColumnHandleHovered || isRowHandleHovered,
+		hasMergedCells,
+	};
 
-  useEffect(() => {
-    const dragHandleDivRefCurrent = dragHandleDivRef.current;
+	useEffect(() => {
+		const dragHandleDivRefCurrent = dragHandleDivRef.current;
 
-    if (dragHandleDivRefCurrent) {
-      return draggable({
-        element: dragHandleDivRefCurrent,
-        canDrag: () => {
-          return !hasMergedCells;
-        },
-        getInitialData() {
-          return {
-            localId: tableLocalId,
-            type: `table-${direction}`,
-            indexes,
-          };
-        },
-        onGenerateDragPreview: ({ nativeSetDragImage }) => {
-          setCustomNativeDragPreview({
-            getOffset: ({ container }) => {
-              const rect = container.getBoundingClientRect();
-              if (browser.safari) {
-                // See: https://product-fabric.atlassian.net/browse/ED-21442
-                // We need to ensure that the preview is not overlaying screen content when the snapshot is taken, otherwise
-                // safari will composite the screen text elements into the bitmap snapshot. The container is a wrapper which is already
-                // positioned fixed at top/left 0.
-                // IMPORTANT: we must not exceed more then the width of the container off-screen otherwise not preview will
-                // be generated.
-                container.style.left = `-${rect.width - 0.0001}px`;
-              }
-              if (isRow) {
-                return { x: 12, y: rect.height / 2 };
-              } else {
-                return { x: rect.width / 2 + 4, y: 12 };
-              }
-            },
-            render: function render({ container }) {
-              setPreviewContainer(container);
-              return () => setPreviewContainer(null);
-            },
-            nativeSetDragImage,
-          });
-        },
-      });
-    }
-  }, [
-    tableLocalId,
-    direction,
-    indexes,
-    isRow,
-    editorView.state.selection,
-    hasMergedCells,
-  ]);
+		if (dragHandleDivRefCurrent) {
+			return draggable({
+				element: dragHandleDivRefCurrent,
+				canDrag: () => {
+					return !hasMergedCells;
+				},
+				getInitialData() {
+					return {
+						localId: tableLocalId,
+						type: `table-${direction}`,
+						indexes,
+					};
+				},
+				onGenerateDragPreview: ({ nativeSetDragImage }) => {
+					setCustomNativeDragPreview({
+						getOffset: ({ container }) => {
+							const rect = container.getBoundingClientRect();
+							if (browser.safari) {
+								// See: https://product-fabric.atlassian.net/browse/ED-21442
+								// We need to ensure that the preview is not overlaying screen content when the snapshot is taken, otherwise
+								// safari will composite the screen text elements into the bitmap snapshot. The container is a wrapper which is already
+								// positioned fixed at top/left 0.
+								// IMPORTANT: we must not exceed more then the width of the container off-screen otherwise not preview will
+								// be generated.
+								container.style.left = `-${rect.width - 0.0001}px`;
+							}
+							if (isRow) {
+								return { x: 12, y: rect.height / 2 };
+							} else {
+								return { x: rect.width / 2 + 4, y: 12 };
+							}
+						},
+						render: function render({ container }) {
+							setPreviewContainer(container);
+							return () => setPreviewContainer(null);
+						},
+						nativeSetDragImage,
+					});
+				},
+			});
+		}
+	}, [tableLocalId, direction, indexes, isRow, editorView.state.selection, hasMergedCells]);
 
-  const showDragMenuAnchorId = isRow
-    ? 'drag-handle-button-row'
-    : 'drag-handle-button-column';
+	const showDragMenuAnchorId = isRow ? 'drag-handle-button-row' : 'drag-handle-button-column';
 
-  return (
-    <>
-      <button
-        type="button"
-// eslint-disable-next-line @atlaskit/ui-styling-standard/no-classname-prop -- Ignored via go/DSP-18766
-        className={ClassName.DRAG_HANDLE_BUTTON_CLICKABLE_ZONE}
-        data-testid="table-drag-handle-clickable-zone-button"
-        style={{
-          height: isRow
-            ? `calc(100% - ${dragTableInsertColumnButtonSize}px)`
-            : `${token('space.200', '16px')}`, // 16px here because it's the size of drag handle button's large side
-          width: isRow
-            ? `${token('space.200', '16px')}` // 16px here because it's the size of drag handle button's large side
-            : `calc(100% - ${dragTableInsertColumnButtonSize}px)`,
-          left: isRow ? `${token('space.050', '4px')}` : undefined,
-          bottom: isColumn ? `${token('space.0', '0px')}` : undefined,
-          alignSelf: isColumn ? 'none' : 'center',
-          zIndex: isColumn ? '-1' : 'auto',
-        }}
-        onMouseUp={(e) => {
-          // should toggle menu if current drag menu open.
-          // return focus to editor so copying table selections whilst still works, i cannot call e.preventDefault in a mousemove event as this stops dragstart events from firing
-          // -> this is bad for a11y but is the current standard new copy/paste keyboard shortcuts should be introduced instead
-          editorView.focus();
-          if (isDragMenuOpen) {
-            toggleDragMenu && toggleDragMenu('mouse', e);
-          }
-        }}
-        onClick={onClick}
-      />
-      <button
-        type="button"
-        id={isDragMenuTarget ? showDragMenuAnchorId : undefined}
-// eslint-disable-next-line @atlaskit/ui-styling-standard/no-classname-prop -- Ignored via go/DSP-18766
-        className={classnames(
-          ClassName.DRAG_HANDLE_BUTTON_CONTAINER,
-          appearance,
-          {
-            [ClassName.DRAG_HANDLE_DISABLED]: hasMergedCells,
-          },
-        )}
-        ref={dragHandleDivRef}
-        style={{
-          transform: isColumn ? 'none' : 'rotate(90deg)',
-          alignSelf: isColumn ? 'none' : 'center',
-        }}
-        data-testid="table-drag-handle-button"
-        aria-label={formatMessage(
-          isRow ? messages.rowDragHandle : messages.columnDragHandle,
-        )}
-        aria-expanded={isDragMenuOpen && isDragMenuTarget ? 'true' : 'false'}
-        aria-haspopup="menu"
-        onMouseOver={onMouseOver}
-        onMouseOut={onMouseOut}
-        onMouseUp={(e) => {
-          // return focus to editor so copying table selections whilst still works, i cannot call e.preventDefault in a mousemove event as this stops dragstart events from firing
-          // -> this is bad for a11y but is the current standard new copy/paste keyboard shortcuts should be introduced instead
-          editorView.focus();
-          toggleDragMenu && toggleDragMenu('mouse', e);
-        }}
-        onClick={onClick}
-        onKeyDown={(e) => {
-          if (e.key === 'Enter' || e.key === ' ') {
-            toggleDragMenu && toggleDragMenu('keyboard');
-          }
-        }}
-      >
-        {appearance !== 'placeholder' ? (
-          //  cannot block pointer events in Firefox as it breaks Dragging functionality
-          browser.gecko ? (
-            <HandleIconComponent {...handleIconProps} />
-          ) : (
-// eslint-disable-next-line @atlaskit/ui-styling-standard/enforce-style-prop -- Ignored via go/DSP-18766
-            <span style={{ pointerEvents: 'none' }}>
-              <HandleIconComponent {...handleIconProps} />
-            </span>
-          )
-        ) : null}
-      </button>
-      {previewContainer &&
-        previewWidth !== undefined &&
-        previewHeight !== undefined &&
-        ReactDOM.createPortal(
-          <DragPreview
-            direction={direction}
-            width={previewWidth}
-            height={previewHeight}
-          />,
-          previewContainer,
-        )}
-    </>
-  );
+	return (
+		<>
+			<button
+				type="button"
+				// eslint-disable-next-line @atlaskit/ui-styling-standard/no-classname-prop -- Ignored via go/DSP-18766
+				className={ClassName.DRAG_HANDLE_BUTTON_CLICKABLE_ZONE}
+				data-testid="table-drag-handle-clickable-zone-button"
+				style={{
+					height: isRow
+						? `calc(100% - ${dragTableInsertColumnButtonSize}px)`
+						: `${token('space.200', '16px')}`, // 16px here because it's the size of drag handle button's large side
+					width: isRow
+						? `${token('space.200', '16px')}` // 16px here because it's the size of drag handle button's large side
+						: `calc(100% - ${dragTableInsertColumnButtonSize}px)`,
+					left: isRow ? `${token('space.050', '4px')}` : undefined,
+					bottom: isColumn ? `${token('space.0', '0px')}` : undefined,
+					alignSelf: isColumn ? 'none' : 'center',
+					zIndex: isColumn ? '-1' : 'auto',
+				}}
+				onMouseUp={(e) => {
+					// should toggle menu if current drag menu open.
+					// return focus to editor so copying table selections whilst still works, i cannot call e.preventDefault in a mousemove event as this stops dragstart events from firing
+					// -> this is bad for a11y but is the current standard new copy/paste keyboard shortcuts should be introduced instead
+					editorView.focus();
+					if (isDragMenuOpen) {
+						toggleDragMenu && toggleDragMenu('mouse', e);
+					}
+				}}
+				onClick={onClick}
+			/>
+			<button
+				type="button"
+				id={isDragMenuTarget ? showDragMenuAnchorId : undefined}
+				// eslint-disable-next-line @atlaskit/ui-styling-standard/no-classname-prop -- Ignored via go/DSP-18766
+				className={classnames(ClassName.DRAG_HANDLE_BUTTON_CONTAINER, appearance, {
+					[ClassName.DRAG_HANDLE_DISABLED]: hasMergedCells,
+				})}
+				ref={dragHandleDivRef}
+				style={{
+					transform: isColumn ? 'none' : 'rotate(90deg)',
+					alignSelf: isColumn ? 'none' : 'center',
+				}}
+				data-testid="table-drag-handle-button"
+				aria-label={formatMessage(isRow ? messages.rowDragHandle : messages.columnDragHandle)}
+				aria-expanded={isDragMenuOpen && isDragMenuTarget ? 'true' : 'false'}
+				aria-haspopup="menu"
+				onMouseOver={onMouseOver}
+				onMouseOut={onMouseOut}
+				onMouseUp={(e) => {
+					// return focus to editor so copying table selections whilst still works, i cannot call e.preventDefault in a mousemove event as this stops dragstart events from firing
+					// -> this is bad for a11y but is the current standard new copy/paste keyboard shortcuts should be introduced instead
+					editorView.focus();
+					toggleDragMenu && toggleDragMenu('mouse', e);
+				}}
+				onClick={onClick}
+				onKeyDown={(e) => {
+					if (e.key === 'Enter' || e.key === ' ') {
+						toggleDragMenu && toggleDragMenu('keyboard');
+					}
+				}}
+			>
+				{appearance !== 'placeholder' ? (
+					//  cannot block pointer events in Firefox as it breaks Dragging functionality
+					browser.gecko ? (
+						<HandleIconComponent {...handleIconProps} />
+					) : (
+						// eslint-disable-next-line @atlaskit/ui-styling-standard/enforce-style-prop -- Ignored via go/DSP-18766
+						<span style={{ pointerEvents: 'none' }}>
+							<HandleIconComponent {...handleIconProps} />
+						</span>
+					)
+				) : null}
+			</button>
+			{previewContainer &&
+				previewWidth !== undefined &&
+				previewHeight !== undefined &&
+				ReactDOM.createPortal(
+					<DragPreview direction={direction} width={previewWidth} height={previewHeight} />,
+					previewContainer,
+				)}
+		</>
+	);
 };
 
 export const DragHandle = injectIntl(DragHandleComponent);