Elm.embed should accept a div with children.

[laszlopandy]

Currently if you try to embed Elm on a div with children it fails with:
Error: Elm.node must be given an empty DIV. No children allowed!

However there is a valid use case when this should be allowed. I am porting a mobile web app from React, which needs to start up as quickly as possible. My React version did something like this:

<div id='react_container'>
    <div class="spinner">
        <div class="bounce1"></div>
        <div class="bounce2"></div>
        <div class="bounce3"></div>
    </div>
</div>
<script src="myapp.js"></script>
<script>
React.renderComponent(MyApp(), document.getElementById('react_container'));
</script>

The spinner div has some CSS animation applied to it, and starts animating immediately while the scripts are loading. Once React has booted up, the first render will replace the children (including spinner) with the real app (or with the same spinner div if the app is still waiting for data). This is a very useful way to lower the perceived loading time.

[evancz]

Makes sense. Is the intended behavior that instantiating the component destroys all the existing children?

[evancz]

Makes sense. Is the intended behavior that instantiating the component destroys all the existing children?

[laszlopandy]

Yes, on the first render the component should delete all children.

On Sunday, December 14, 2014, Evan Czaplicki notifications@github.com
wrote:

Makes sense. Is the intended behavior that instantiating the component
destroys all the existing children?

—
Reply to this email directly or view it on GitHub
https://github.com/elm-lang/core/issues/47#issuecomment-66899024.

[laszlopandy]

Yes, on the first render the component should delete all children.

On Sunday, December 14, 2014, Evan Czaplicki notifications@github.com
wrote:

Makes sense. Is the intended behavior that instantiating the component
destroys all the existing children?

—
Reply to this email directly or view it on GitHub
https://github.com/elm-lang/core/issues/47#issuecomment-66899024.