Breaks When Using

FlorianWendelborn commented 7 years ago

I tried to modify hyperapp's counter example. When inserting a line-break   it broke. 🙁

Console

InvalidCharacterError (DOM Exception 5): The string contains invalid characters.

app({
    view: () => html`<div>a<br/>b</div>`
})

Corresponding issue on hyperapp's repo.
Related to substack/hyperx#34.

Note that I do realize that   works.   is more explicit though and as a React-user I'd like to keep this pattern.

Ramblurr commented 6 years ago

I also was caught by this bug.

josephg commented 6 years ago

Some notes:

Self-closing tags don't exist in the html spec.
Browsers do ignore the '/' and treat   and   as synonyms
Hyperx works fine if you put in a space -  
It should be easy to change the parser to support this. Its probably a ~2 line change to stop parsing the tag name when the parser sees a '/' character. Make a unit test, fix the code and submit a PR.
For now, the workaround is to write

yoshuawuyts commented 6 years ago

A PR to support   would be very welcome!