sveltejs / cli

The Svelte CLI
MIT License
258 stars 11 forks source link

Following HTML5 specification for meta and link in app.html #318

Closed shinokada closed 2 days ago

shinokada commented 3 days ago

Describe the bug

According to HTML5 specification, meta and link do not have the trailing slash.

So shouldn't it be

<meta charset="utf-8">
<link rel="icon" href="%sveltekit.assets%/favicon.png">
<meta name="viewport" content="width=device-width, initial-scale=1">

rather than:

<meta charset="utf-8" />
<link rel="icon" href="%sveltekit.assets%/favicon.png" />
<meta name="viewport" content="width=device-width, initial-scale=1" />

Reproduction

Check src/app.html.

Logs

No response

System Info

n/a

Severity

annoyance

rChaoz commented 2 days ago

Also according to the spec, self-closing tags can close with either /> or > (there is no difference).

benmccann commented 2 days ago

Closing per the last comment