Comments (6)
Wow! Great find. Your description makes some sense to me; useEffect
definitely fires after rendering is done. I can see how the race condition occurs; that's a tricky one!
I can think of two potential fixes:
1. Use useLayoutEffect
instead to fire synchronously after render
However, this might have negative effects; per the React docs:
Updates scheduled inside useLayoutEffect will be flushed synchronously, before the browser has a chance to paint.
This means that when Concurrent React lands, we might be over-scheduling the state changes to re-render the component.
This actually begs the question how re-renders are being scheduled now while we're using useEffect
. 🤔
2. Manually manage the subscription via refs
Something like:
(let [[v u] (<-state @a)
atom-identity (<-ref nil)
sub (<-ref (gensym "<-"))]
(when (and (not= a @atom-identity)
(when (not (nil? @atom-identity))
(remove-watch a @sub))
(add-watch a @sub (fn [_ _ _ v'] (u v'))))
v)
This completely removes the act of subscribing from React's lifecycle. But once again I'm not sure how this interacts with React's scheduler.
I might reach out to the React team to see the best way to handle this case. I imagine this would come up anytime you're subscribing to some external event bus.
from hx.
Looks like there is already an issue in the React repo: facebook/react#14988
gaeron posted this:
As for fixing the original issue, the strategy is to read the mutable value in useEffect. If it changed since the value captured during render, setState.
Which seems like an easy fix.
from hx.
from hx.
Would you be able to verify that 6d3b7cb fixes the issue?
from hx.
I’ll have to test it on Monday, but I applied pretty much the same code (sans comparison) in a local version and that did the trick. The comparison should be ok, since there can’t be a second thread mutating the atom behind the scenes.
Thanks!
from hx.
Cool. Closing this for now. Re-open if you still run into issues.
from hx.
Related Issues (20)
- Destructuring defnc :keys [class] has weird behaviour HOT 1
- Improved react dev-tools support for hooks HOT 3
- CLJS data structures are compared by ref in <-effect deps HOT 11
- Semantics and API of <-state HOT 16
- class & className issue when wrapping defnc component with pure react component HOT 3
- Make it fast
- Experience Report HOT 7
- Reducers dont support Multimethods HOT 4
- Document hiccup parser API
- hx/f exceptions handlint HOT 4
- any HOT 1
- Export hooks under Clojure-style names HOT 2
- `defcomponent` API is very confusing HOT 1
- How-to contains misleading memo use HOT 7
- Push missing tags + maybe a 0.5.4 release HOT 2
- Docstring support for defnc (and defcomponent) HOT 1
- track dependencies to selectively rerender with useMemo HOT 3
- Passing down original js props in render props does not work with js->clj
- hx.hooks/useMemo deps as CLJS collection? HOT 1
- Typo in package 0.5.3 (maybe broken) HOT 3
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from hx.