-
-
Notifications
You must be signed in to change notification settings - Fork 313
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
Performance bugs #276
Comments
You're right about the Since you obviously took more than a passing glance at the code, why didn't you bother to submit a PR? |
I have already re-coded as it should be, if you want you can take a look. |
I have the same issue. Mousemove event triggers angular change detection and therefore causes performance issues. |
Hey guys,
I wondered,why you are attaching a lot of events mousedown,when you can use only 1and your mousemove and mouseup events are working always even if there is no element attached. You need to attache mousemove and mouse ONLY after mousedown event and remove the events on mouse up.The plugin works good but the code dirty is dirty.
The text was updated successfully, but these errors were encountered: