[PR 3] Removed edge-cases of GridSelection

packages/lexical-clipboard/flow/LexicalClipboard.js.flow

@@ -11,7 +11,6 @@ import type {
   BaseSelection,
   LexicalEditor,
   INTERNAL_PointSelection,
-  GridSelection,
   RangeSelection,
   LexicalNode,
 } from 'lexical';

packages/lexical-clipboard/src/clipboard.ts

@@ -18,6 +18,7 @@ import {
   $createTabNode,
   $getRoot,
   $getSelection,
+  $INTERNAL_isPointSelection,
   $isElementNode,
   $isRangeSelection,
   $isTextNode,
@@ -30,13 +31,11 @@ import {
   DEPRECATED_$isGridCellNode,
   DEPRECATED_$isGridNode,
   DEPRECATED_$isGridRowNode,
-  DEPRECATED_$isGridSelection,
   DEPRECATED_GridNode,
-  GridSelection,
+  INTERNAL_PointSelection,
   isSelectionWithinEditor,
   LexicalEditor,
   LexicalNode,
-  RangeSelection,
   SELECTION_CHANGE_COMMAND,
   SerializedTextNode,
 } from 'lexical';
@@ -205,17 +204,17 @@ export function $insertGeneratedNodes(
   nodes: Array<LexicalNode>,
   selection: BaseSelection,
 ): void {
-  const isGridSelection = DEPRECATED_$isGridSelection(selection);
+  const isPointSelection = $INTERNAL_isPointSelection(selection);
   const isRangeSelection = $isRangeSelection(selection);
   const isSelectionInsideOfGrid =
-    isGridSelection ||
     (isRangeSelection &&
       $findMatchingParent(selection.anchor.getNode(), (n) =>
         DEPRECATED_$isGridCellNode(n),
       ) !== null &&
       $findMatchingParent(selection.focus.getNode(), (n) =>
         DEPRECATED_$isGridCellNode(n),
-      ) !== null);
+      ) !== null) ||
+    (isPointSelection && !isRangeSelection);
 
   if (
     isSelectionInsideOfGrid &&
@@ -232,7 +231,7 @@ export function $insertGeneratedNodes(
 
 function $mergeGridNodesStrategy(
   nodes: LexicalNode[],
-  selection: RangeSelection | GridSelection,
+  selection: INTERNAL_PointSelection,
   isFromLexical: boolean,
   editor: LexicalEditor,
 ) {

packages/lexical-playground/__tests__/e2e/CopyAndPaste/html/TablesHTMLCopyAndPaste.spec.mjs

@@ -242,6 +242,10 @@ test.describe('HTML Tables CopyAndPaste', () => {
 
   test('Copy + paste - Merge Grids', async ({page, isPlainText, isCollab}) => {
     test.skip(isPlainText);
+    test.fixme(
+      isCollab,
+      'Table selection styles are not properly selected/deselected',
+    );
 
     await focusEditor(page);
     await insertTable(page, 4, 4);

packages/lexical-selection/src/lexical-node.ts

@@ -19,7 +19,6 @@ import {
   $setSelection,
   BaseSelection,
   DEPRECATED_$isGridCellNode,
-  DEPRECATED_$isGridSelection,
   ElementNode,
   INTERNAL_PointSelection,
   LexicalEditor,
@@ -322,7 +321,7 @@ export function $patchStyleText(
   const selectedNodes = selection.getNodes();
   const selectedNodesLength = selectedNodes.length;
 
-  if (DEPRECATED_$isGridSelection(selection)) {
+  if (!$isRangeSelection(selection)) {
     const cellSelection = $createRangeSelection();
     const cellSelectionAnchor = cellSelection.anchor;
     const cellSelectionFocus = cellSelection.focus;

packages/lexical/src/LexicalSelection.ts

@@ -545,14 +545,6 @@ export class GridSelection extends INTERNAL_PointSelection {
     this.gridKey = gridKey;
   }
 
-  getCachedNodes(): LexicalNode[] | null {
-    return this._cachedNodes;
-  }
-
-  setCachedNodes(nodes: LexicalNode[] | null): void {
-    this._cachedNodes = nodes;
-  }
-
   is(selection: null | BaseSelection): boolean {
     if (!DEPRECATED_$isGridSelection(selection)) {
       return false;
@@ -837,14 +829,6 @@ export class RangeSelection extends INTERNAL_PointSelection {
     this.style = style;
   }
 
-  getCachedNodes(): LexicalNode[] | null {
-    return this._cachedNodes;
-  }
-
-  setCachedNodes(nodes: LexicalNode[] | null): void {
-    this._cachedNodes = nodes;
-  }
-
   /**
    * Used to check if the provided selections is equal to this one by value,
    * inluding anchor, focus, format, and style properties.