-
Notifications
You must be signed in to change notification settings - Fork 46.9k
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
Wishlist: ability to disable the synthetic event system #1269
Comments
Not quite sure what you mean. If you don't attach any events to your elements then there shouldn't be any overhead from the events system. |
@jaredly, are you seeing issues? The Synthetic event system may actually help you get responsiveness out of underpowered machines. Attaching event listeners to actual DOM nodes is expensive. |
Yeah, I'm seeing a lag with my <input>s. I know I can just hack around it with componentDidMount, but I wondered if it would be possible to just switch to native event handling. |
@jaredly You're seeing lag that goes away when you use native event handling? I'd be interested in seeing a test case that shows that. |
step 1) use a really underpowered netbook :) |
The problem is probably exacerbated by the "keydown -> change state -> re-render -> update dom" loop for bound input elements. |
Is this just an input related problem? Or do you see issues with click handlers (I suspect you don't)? |
I've only noticed it on the inputs |
I don't have any onMouseMove handlers at the moment, but I wouldn't be surprised if it had issues as well |
click handlers don't generate enough events to bottleneck |
@jaredly It sounds like your problem is large parts or the entire app re-rendering (without any shouldComponentUpdate) and it's simply too costly and ends up slowing down the input. |
That's what I thought, but having a single input on a page with nothing
|
No plans to do this. |
I am trying to build a side nav that can be dragged in from the left. I listen to the The touchmove event is fired at a very high speed, though I throttle these events using |
Would there be huge problems caused by optionally disabling the synthetic event system? I'd like the option to do so in order to get more responsiveness out of underpowered machines.
The text was updated successfully, but these errors were encountered: