Fix programmatic clipboard copy on non-editable

[zurfyx]

Clipboard copy uses a double execCommand hack. It turns out that we weren't listening for the copy event on non-editable. This PR allowlists this event which will subsequently dispatch the command after, I think this list should be revised after, potentially adding focus and blur but will do for now.

Screen.Recording.2024-05-31.at.4.50.39.PM.mov

Closes #6231

[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	Jun 2, 2024 0:46am
lexical-playground	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jun 2, 2024 0:46am

[github-actions]

size-limit report 📦

Path	Size
lexical - cjs	23.97 KB (0%)
lexical - esm	23.96 KB (0%)
@lexical/rich-text - cjs	34.66 KB (0%)
@lexical/rich-text - esm	23.95 KB (0%)
@lexical/plain-text - cjs	34.65 KB (0%)
@lexical/plain-text - esm	225 B (0%)
@lexical/react - cjs	149.31 KB (0%)
@lexical/react - esm	82.64 KB (0%)

[etrepum]

Shouldn't cut also be conditional when isEditable?

[zurfyx]

@etrepum You're right 🤦‍♂️

[etrepum]

Looks like the current failure is prettier for packages/lexical/src/LexicalEvents.ts

[etrepum]

Do you think this would be easier to maintain as a table? e.g.

ts playground example

const READONLY_EVENTS = {
  copy: COPY_COMMAND,
  // more commands here
} as const;
const EDITABLE_EVENTS = {
  ...READONLY_EVENTS,
  cut: CUT_COMMAND,
  // more commands here
} as const;

export function dispatchEvent(editor: LexicalEditor, eventName: string, event: Event) {
  const eventMap = editor.isEditable() ? EDITABLE_EVENTS : READONLY_EVENTS; 
  const command = (eventMap as Readonly<Record<string, undefined | LexicalCommand<Event>>>)[eventName];
  if (command) {
    dispatchCommand(editor, command, event);
  }
}