Skip to content

[lexical-text] Chore: Add deprecation notice to $findTextIntersectionFromCharacters#8364

Merged
etrepum merged 2 commits intofacebook:mainfrom
etrepum:claude/deprecate-find-text-intersection-07jvx
Apr 17, 2026
Merged

[lexical-text] Chore: Add deprecation notice to $findTextIntersectionFromCharacters#8364
etrepum merged 2 commits intofacebook:mainfrom
etrepum:claude/deprecate-find-text-intersection-07jvx

Conversation

@etrepum
Copy link
Copy Markdown
Collaborator

@etrepum etrepum commented Apr 17, 2026

Description

$findTextIntersectionFromCharacters doesn't work correctly and has no tests, usage, or documentation. Like the situation with @lexical/offset, it would be nice to have something like this, but it should be implemented correctly with an in-repo use case, tests, and documentation.

Test plan

N/A

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 17, 2026

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

Project Deployment Actions Updated (UTC)
lexical Ready Ready Preview, Comment Apr 17, 2026 6:48am
lexical-playground Ready Ready Preview, Comment Apr 17, 2026 6:48am

Request Review

@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 Apr 17, 2026
@etrepum etrepum added the extended-tests Run extended e2e tests on a PR label Apr 17, 2026
@etrepum
Copy link
Copy Markdown
Collaborator Author

etrepum commented Apr 17, 2026

Related #4103 (comment)

@etrepum etrepum added this pull request to the merge queue Apr 17, 2026
Merged via the queue into facebook:main with commit 4437e81 Apr 17, 2026
44 checks passed
@etrepum etrepum deleted the claude/deprecate-find-text-intersection-07jvx branch April 19, 2026 02:32
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.

3 participants