[lexical-table] Stop selecting the whole table after pasting cells

[ivailop7]

Fixes #5656

Upon pasting a cells selection in a table when the target area is not a cells selection inside a table, it would select the whole table after the paste event. It worked correctly (not select the whole table) if the selected area to paste to was a cells selection.

Before:

before.mov

After:

after.mov

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
lexical	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Aug 22, 2024 4:46pm
lexical-playground	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Aug 22, 2024 4:46pm

[github-actions]

size-limit report 📦

Path	Size
lexical - cjs	29.38 KB (0%)
lexical - esm	29.24 KB (0%)
@lexical/rich-text - cjs	37.74 KB (0%)
@lexical/rich-text - esm	31.05 KB (0%)
@lexical/plain-text - cjs	36.41 KB (0%)
@lexical/plain-text - esm	28.42 KB (0%)
@lexical/react - cjs	39.64 KB (0%)
@lexical/react - esm	32.51 KB (0%)

[moughxyz]

I was actually just investigating an issue with whole table selection as well :) #6542

I'm wondering if it should be ok for a PR like the one that was merged here to go through without tests? The problem is that the change made here is now indefensible against future changes. No one will know whether it's safe to modify code in this area, and no one will know whether their changes break something here.

I mention this in my PR:

It's also my opinion that for large open source projects like this, tests should be the ultimate way of determining what stays and what goes. At some point there will no longer be anyone involved in the project who has all the historical context necessary to determine whether a change is harmful or beneficial, and its up to the tests to defend each change.