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

remove isNative? #963

Closed
ianstormtaylor opened this issue Aug 1, 2017 · 0 comments
Closed

remove isNative? #963

ianstormtaylor opened this issue Aug 1, 2017 · 0 comments

Comments

@ianstormtaylor
Copy link
Owner

I haven't looked into this deeply, but I remember having a conversation where it seemed like the performance benefits we were getting from isNative might not be as much as we thought, given that we do some other performance-enhancing things that Draft.js does not (and isNative was borrowed from Draft).

So we might be able to get rid of it, which would simplify a lot of other things.

If someone wanted to investigate this with a PR and check the performance after removing it, that would be amazing!

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

No branches or pull requests

1 participant