-
Notifications
You must be signed in to change notification settings - Fork 141
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
A solution for reflex-dom + jsaddle-warp UI slowness, stutters, freezes. #224
Comments
@Wizek I'm not quite willing to break preventDefault. However, @hamishmack and I have been working on a major refactor of the jsaddle core to address these issues. The relevant branches are named things like Apologies for the slow response, and I must admit it may be a little while before I'm able to read everything you've written here and give it the consideration it needs. Hopefully we can get everything in a state that works for you as well as all the other users of jsaddle soon! |
Glad to read even this brief reply. Looking forward to your more detailed feedback.
Maybe this will become clearer upon a more thorough reading, but I don't think I'm proposing that either. My apps usually rely on
Great! Maybe/hopefully that will also address all these problems I've experienced and I can switch over. But I'm not holding my breath if it'll still rely on sync XHRs, and not fully (or mostly) async WS messages as the POC above shows, as there seem to be some fundamental UX problems with the sync XHR approach. |
@ryantrinkle any progress here? |
@Wizek could you explain that gist where you add the event using |
Hello @ryantrinkle. I've thought to reproduce here most of the email that I've sent you a about 10 days ago since others might be affected by this issue as well, and might benefit from me mentioning it here. And @hamishmack might also be interested to be included since it's related to jsaddle{,-warp}. (Even though we might decide to make these changes entirely on the level of reflex-dom without any changes required from jsaddle.)
The text was updated successfully, but these errors were encountered: