createSignal Question

[jcuenod]

Hey Ryan (et. al.)
I'm still having trouble understanding why this doesn't work:

const Pane = props => {
    const [children, setChildren] = createSignal([])
    onMount(() => {
        const paneDetails = TreeModel.getPane(props.paneId)
        console.log(paneDetails.children, children())
        setChildren(paneDetails.children)
        console.log(paneDetails.children, children())
    }
    ...

The console logs are:

[1,2,3], []
[1,2,3], []

But, obviously, I'm expecting them to be:

[1,2,3], []
[1,2,3], [1,2,3]

[ryansolid]

Yeah it's because the change hasn't been committed yet. I made a change about 6 months ago to batch work in Effects. It wasn't the only solution but it was the easiest to ensure timing in some of the trickier use cases (concurrent rendering and transitions). Under a batch Solid has the same timing as React. So you won't see the update until later. Basically all updates are collected and applied later. I need to document this. And probably look at the necessity of the Effect batching (although I'm pretty sure it is necessary in those cases).