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

getNodeKey with text nodes isn't supported? #239

Open
aleclarson opened this issue Apr 7, 2023 · 0 comments
Open

getNodeKey with text nodes isn't supported? #239

aleclarson opened this issue Apr 7, 2023 · 0 comments

Comments

@aleclarson
Copy link

This isn't a high priority bug, as I don't actually need getNodeKey to work with text nodes, but I noticed issues when trying it out, so it should probably be documented that getNodeKey should always return undefined for text nodes (or preferably, never call getNodeKey on text nodes in the first place). Or if the intention is to support this use case, a test should be added for it.

As I have a workaround, I don't have time to write a detailed report.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant