HEEx doesn't raise No space between attributes error

[bcardarella]

This works just fine:

~H"""
   <div style=""id="123"/>
"""

It will render as :

<div style="" id="123"/>

but the original template is invalid and shouldn't compile.

[josevalim]

Although not ambiguous here, I understand why HTML does it. We could error but perhaps it makes more sense to warn, given the formatter will automatically fix it for you anyway, and that would not be possible if erroring.

[bcardarella]

Now that LV is 1.0 I agree that this should not error. Considering it compiles anyway I didn't know if it was worth pursuing or not but figured I'd bring attention to it.

[bcardarella]

And Floki parses this syntax just fine without warning, which means that Mochi parses just fine. I think I'll close this as a non-issue.

I will have to account for this in LVN's template parser as we are erroring on it.

[josevalim]

Let's keep it open because I think warning is better. We don't want to have someone thinking this is fine and then emitting some hand written HTML somehow with this bug :)

[josevalim]

FWIW, browsers parse it as multiple attributes too, but they are also historically known to be very lenient.

[bcardarella]

yeah I sent it through the W3C validator which will be less forgiving