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.

Sign up for GitHub

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

Jump to bottom

add a findPath DOM helper #2717

Closed
ianstormtaylor opened this issue Apr 27, 2019 · 0 comments · Fixed by #2747
Closed

add a findPath DOM helper #2717

ianstormtaylor opened this issue Apr 27, 2019 · 0 comments · Fixed by #2747
Labels
♥ help improvement ⚑ perf

Comments

@ianstormtaylor
Copy link
Owner

Do you want to request a feature or report a bug?

Improvement.

What's the current behavior?

Right now findNode is used, and then since it's easiest we end up using node.key for subsequent lookups.

What's the expected behavior?

If there was a findPath DOM helper instead, all of the subsequent lookups could use paths also instead of keys.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
♥ help improvement ⚑ perf
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Introduce annotations ianstormtaylor/slate
1 participant
@ianstormtaylor