[Feature Request] Allow substance components to return components as their root? about substance

[obuchtala]

merged #1196 with a fix for detecting invalid render() implementation.

[e-e-e]

You don't need to apologise - I am really enjoying using substance and am happy to help contribute by finding any issues.
Thanks so much for making such a great open source library.

[obuchtala]

That would be a wonderful contribution!

[obuchtala]

ATM, this must be a regular element. But please feel free to turn this issue into a feature request. This was definitely something we were also wanting several times.

[e-e-e]

Ah cool - good to know its a know constraint! Thank you @oliver----. I will modify the title of the issue to make it into a feature request.

In the mean time - it could be beneficial to catch this and warn the user. Might save some folk some time in debugging. I definitely just spent way too long diagnosing this.

[obuchtala]

Yeah. Sorry about the debugging experience.
Will do what you suggested!

[obuchtala]

At least I found that we have a check in the code already ;)
So you saw an exception "Component.render must return VirtualHTMLElement", didn't you?
Should improve the error message. And documentation-wise we are not at all in a good place.

[e-e-e]

Actually no - I did not see that error at all. But perhaps that is a version issue - and I should update to the latest.

Its tough - without docs - I have just been reading your code. I should probably be helping also by contributing back documentation as I go.

[obuchtala]

Re: exception. I can confirm that the test I have for that is not sufficient. So this slipped through.

[obuchtala]

Returning $$(Component) is on the way...
15375f3

[michael]

Working now with preview.75.

[e-e-e]

Amazing - thanks guys.