Plugin is incorrectly rewriting empty non-void HTML tags in svelte files

[milne-dev]

Within .svelte files, this plugin rewrites empty <div></div> to <div /> which is an HTML error according to the whatwg spec:

non-void-html-element-start-tag-with-trailing-solidus

This error occurs if the parser encounters a start tag for an element that is not in the list of void elements or is not a part of foreign content (i.e., not an SVG or MathML element) that has a U+002F (/) code point right before the closing U+003E (>) code point.

The list of valid void elements:
area, base, br, col, embed, hr, img, input, link, meta, source, track, wbr

[dummdidumm]

We will adjust the options in the next major version to make this possible, though we're not sure yet what the default should be. Closely related to #238 / #348.

[milne-dev]

Please consider making the defaults align with the HTML standard as well as the behavior of prettier core.

Further, my initial motivation to report this issue was because I found it frustrating to write layout structure in HTML only to have several elements being rewritten as <tag /> before I gave them content.

[dummdidumm]

Version 3 of this plugin will leave tags as-is, so writing <div></div> won't be reformatted to <div /> anymore. We keep formatting <input> as <input /> to align with Prettier.

[danielniccoli]

I'm not in favour of aligning with Prettier in case of void elements. They're explicitly mentioned as not self-closing. Prettier takes an opinionated stance on the matter and imposes its choice on the developer. I wouldn't argue, if it were up to the developer, but the HTML specs are very clear on void elements.

For now, I can enable svelteStrictMode, but as I understand it's deprecated.

[danielniccoli]

Related sveltejs/svelte#11052

[mustafa0x]

Related: https://github.com/awmottaz/prettier-plugin-void-html

Doesn't work with .svelte files thought.

[shadow-identity]

Rich mentioned that the latest version on prettier-plugin-svelte should prevent prettier from braking html by converting html tags into void tags. But as I can see, 3.2.3 still replaces <br> with <br />. Is it planned to fix it?

[dummdidumm]

No this will stay as is for the foreseeable future to align with Prettier's HTML formatting. <br> is a void tag, and it's fine to write <br /> as well (browsers handle it). What is not fine, and which was changed (and accounted for in the prettier plugin) is writing <div /> instead of <div></div>.