nodesInitialized will fire before node measurements are exposed since xyflow 12-13 about react-flow HOT 6 CLOSED

Thanks :) I can confirm that behavior is consistent between 12.0.0-next.12 and 12.0.0-next.17. Appreciate the look into this.

from react-flow.

Thanks for the super detailed explanation here :) This should be fixed in 12.0.0-next.16.

from react-flow.

Hey, I've tested my example on v12-next.16, and I'm not able to get nodesInitialized to run consistently on the demo. I've added an alert statement whenever nodesInitialized changes for debugging purposes. The nodes useEffect will run only once (not twice for population & measurement), and the nodesInitialized useEffect seems to only fire when I initialize / hot reload the page. However, on my school project, I'm able to get nodesInitialized to fire, and I'm not really sure why it's inconsistent here.

This is on v12-next.16:
https://codesandbox.io/p/sandbox/reactflow-measurement-bug-v12-next-12-forked-yw88qs

Thanks for the response and maintaining this package.

from react-flow.

It seems we need to re-open this :/ We will check it next week. Thanks for the codesandbox!

from react-flow.

Thanks! Take care.

from react-flow.

This is fixed in 12.0.0-next.17. Thanks again @collinsleewyatt !

from react-flow.