-
Notifications
You must be signed in to change notification settings - Fork 368
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
TextRange's findText() method is very slow #476
Comments
I have found that |
Since @timdown isn't supporting the repo. If anyone is interested to pair program this issue please comment below. |
I'm having a tidy-up and PR review and intend to release a new minor version early next week, after which it may make sense to find a new maintainer. I will look into some issues as well but I'm not anticipating being able to do much about this one: the TextRange module is just very slow. |
Hello @timdown ! Thanks for reaching out. If I were to optimize the findText() function in the TextRange module. Where would you recommend I start? |
As I understand it,
|
The findText() method of the TextRange module takes about 200ms to run and around 300ms on 6x throttle on chrome . This causes lag if the method is called done multiple times or even a few times.
The text was updated successfully, but these errors were encountered: