Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replaced addTransform with registerNodeTransform in transforms doc #2882

Merged
merged 1 commit into from
Aug 24, 2022

Conversation

kevinansfield
Copy link
Collaborator

addTransform was renamed a couple of times (first to addNodeTransform then registerNodeTransform) but the doc page on transforms still referenced the original naming.

`addTransform` was renamed a couple of times (first to `addNodeTransform` then `registerNodeTransform`) but the doc page on transforms still referenced the original naming.
@vercel
Copy link

vercel bot commented Aug 24, 2022

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

Name Status Preview Updated
lexical ✅ Ready (Inspect) Visit Preview Aug 24, 2022 at 9:34AM (UTC)
lexical-playground ✅ Ready (Inspect) Visit Preview Aug 24, 2022 at 9:34AM (UTC)

@facebook-github-bot facebook-github-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 Aug 24, 2022
@kevinansfield
Copy link
Collaborator Author

Something for a separate issue - it's not entirely clear what "rules 2 and 1" refers to in the "You move a node via replace, rules 2 and 1 are applied." text.

Copy link
Collaborator

@trueadm trueadm left a comment

Choose a reason for hiding this comment

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

Thanks

@@ -107,7 +107,7 @@ You remove a node, its parent is marked dirty, also the node's immediate sibling
You move a node via `replace`, rules 2 and 1 are applied.
Copy link
Collaborator

Choose a reason for hiding this comment

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

I don't even know what this is too, must have been a typo. We can probably remove this line.

Copy link
Contributor

Choose a reason for hiding this comment

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

Probably refers to the previous two lines?

@acywatson
Copy link
Contributor

Thank you!

@acywatson acywatson merged commit e690aa2 into facebook:main Aug 24, 2022
thegreatercurve pushed a commit that referenced this pull request Nov 25, 2022
#2882)

`addTransform` was renamed a couple of times (first to `addNodeTransform` then `registerNodeTransform`) but the doc page on transforms still referenced the original naming.
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants