HTML comments breaks morph about morphdom HOT 3 CLOSED

Hey @weslleyaraujo, thanks for the feedback!

This is the expected behavior, but morphdom should probably report an error to the user. What's happening behind the scenes is that morphdom is parsing the HTML provided as the second argument using createContextualFragment(html) and then it is picking only the first child as the target node:

If the parsing of the contextual fragment results in multiple child nodes then that is really bad input for the current implementation of morphdom.

The fact that there is a comment node doesn't really matter in this case. While it might solve your issue to skip over the comment nodes, I am not sure that is the right solution.

Hopefully you can update your code accordingly? If you have any thoughts or concerns please share, but for now I am going to go ahead and close the issue. While we could throw an error if there are multiple child nodes, that could potentially be a breaking change and we would need to up the major version and I want to avoid doing that for now.

from morphdom.

Thanks for the insights!

I think that some kind of warning/error would be very useful in this case,
the example above was just a test that I was doing to check... but back in a real world project, I took quite long to find out what was happening.

I think this might be something to add on the roadmap for the next major release,

thanks once again!

from morphdom.

he example above was just a test that I was doing to check... but back in a real world project, I took quite long to find out what was happening.

@weslleyaraujo @patrick-steele-idem Do you remember how to solve it? Same this is happening to me and I don't know how to solve it.

from morphdom.