Skip to content

Conversation

@ivailop7
Copy link
Collaborator

Reverts #7984

@vercel
Copy link

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
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
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
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.

5 participants