Open Wizek opened 6 years ago
@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 experimental-core
. It's not quite ready yet, but I think it will be fairly soon.
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!
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.
Glad to read even this brief reply. Looking forward to your more detailed feedback.
I'm not quite willing to break preventDefault
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 preventDefault
too. It's just a slightly different way to achieve the same behavior: [3]. And in all cases, I'm not even saying we'd have to apply this minor async-limitation to all code, just that it could make sense to make this configurable from within reflex-dom or jsaddle* somehow, so those app developers who suffer from UI slowness/stutters/freezes can try it out and see if it helps them.
hamishmack and I have been working on a major refactor of the jsaddle core to address these issues
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 JSA.call
a bit? I'm getting a requirement for ToJSVal (RawElement (DomBuilderSpace m))
since my element is created outside of the prerender
thunk in which I'm trying to use your code and I don't know how to satisfy this sensibly. Thanks a lot!
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.)