make the closing "/" character optional for the void HTML elements

[PK1A]

Fixes #70
Closes #74

[marclaval]

There is a
opening element in the template

[PK1A]

Yes, the error message is not the best after this change :-/
Let me see if I can quickly fix it.

[PK1A]

How about this message: "An orphan end element </br> is not needed, <br> is a void HTML element"?

[benouat]

I would suggest:
</whatevervoidelem> and/or <whatevervoidelem/> is not needed here. As a void HTML element just <whateverelem> is fine

in which we replace dynamically whateverelem

[olaf-k]

My 2 cents here:

• <br/> is not necessary as <br> is perfectly fine, but
• </br> is not just "not needed", it is wrong.

Only the second point should be raised (as an error.)

[piuccio]

<br> is a void element and has no closing tag </br>
or just
<br> has no closing tag </br>

[benouat]

I would still enforce the void element as a warning. Just to make sure that people that does not know will learn something.

My concern here is quite simple. Let's consider something that does not know void elements syntax (or even just void elements period):

• he might write <br></br>
• we might display an error such as <br> is a void element and has no closing tag </br>

What would be the first reaction ? I am pretty sure it might be this one

• he will change the code to <br/>

People probably doesn't know void elements, if if you suggest that the closing tag is not mandatory, they will self close if...

Which is totally fine, as @olaf-k mentioned. This is perfectly valid (even in html5, or x-html5 I would say..).
So for me, we should take the opportunity, via a warning message, to educate our user with a nice message that a void html element such as <br> or <input> is fine! <br/> or <input/> is not needed