Skip to content

Revert "[lexical-react] [lexical-playground] Remove old ContextMenu, consolidate LexicalMenu render"#7997

Merged
thatmichael85 merged 1 commit intomainfrom
revert-7984-reusable_menu
Nov 20, 2025
Merged

Revert "[lexical-react] [lexical-playground] Remove old ContextMenu, consolidate LexicalMenu render"#7997
thatmichael85 merged 1 commit intomainfrom
revert-7984-reusable_menu

Conversation

@ivailop7
Copy link
Copy Markdown
Collaborator

Reverts #7984

@vercel
Copy link
Copy Markdown

vercel bot commented Nov 18, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
lexical Ready Ready Preview Comment Nov 18, 2025 10:09pm
lexical-playground Ready Ready Preview Comment Nov 18, 2025 10:09pm

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Nov 18, 2025
@ivailop7 ivailop7 added the extended-tests Run extended e2e tests on a PR label Nov 18, 2025
Copy link
Copy Markdown
Member

@zurfyx zurfyx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving to unblock but I would personally hold on merging until @jrfitzsimmons and @thatmichael85 have had a chance to review it and explore further adoption options. As we discussed, I genuinely think that this PR provides clear benefits for an existing anchoring problem (including externally in the community and internally at Meta), and just happens to be not so immediately compatible with the way we do component design systems.

@james-atticus
Copy link
Copy Markdown
Contributor

@zurfyx we'd be fine with #7995; we're not doing anything with our menus that can't be replicated with CSS for the container/div/ul/li elements.

@thatmichael85 thatmichael85 added this pull request to the merge queue Nov 20, 2025
Merged via the queue into main with commit 80fce01 Nov 20, 2025
43 checks passed
@etrepum etrepum mentioned this pull request Dec 10, 2025
@zurfyx zurfyx deleted the revert-7984-reusable_menu branch December 18, 2025 20:16
thatmichael85 added a commit that referenced this pull request Mar 9, 2026
… consolidate LexicalMenu render" (#7997)

This reverts commit 80fce01.
thatmichael85 added a commit that referenced this pull request Mar 9, 2026
… consolidate LexicalMenu render" (#7997)

This reverts commit 80fce01.
thatmichael85 added a commit that referenced this pull request Mar 9, 2026
Reapply "[lexical-react] [lexical-playground] Remove old ContextMenu,
consolidate LexicalMenu render" (#7997) with backward compatibility:

- Remove LexicalContextMenuPlugin (consolidated into LexicalNodeMenuPlugin)
- Add default menu rendering in LexicalMenu when menuRenderFn is not provided
- Make menuRenderFn optional on all plugin props for backward compatibility
- Playground plugins use built-in default menu renderer instead of custom ones
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. extended-tests Run extended e2e tests on a PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants