__layout.svelte is ugly... So is %sveltekit.head% #5816
Replies: 4 comments 9 replies
-
|
I could be wrong, but I think svelte uses the same syntax. If this changes it would probably have a ripple effect. |
Beta Was this translation helpful? Give feedback.
-
|
Well Svelte doesn't have control over const app = new App({
target: document.querySelector("#app")
}); |
Beta Was this translation helpful? Give feedback.
-
|
Honestly, |
Beta Was this translation helpful? Give feedback.
-
Svelte has |
Beta Was this translation helpful? Give feedback.
-
|
Yes, svelte does already have a |
Beta Was this translation helpful? Give feedback.
-
Can you share link to where that is documented, because now you mention routes and that sounds like SvelteKit as Svelte doesn't have routes neither. (Unless I'm missing something) |
Beta Was this translation helpful? Give feedback.
-
|
Sure thing: https://svelte.dev/docs#template-syntax-svelte-head (same link as attached to my previous reply). I mentioned routes thinking of SvelteKit, you are right on that. But this special element is from Svelte (SvelteKit routes are essentially components with extended functionalities for ssr and other kit features) |
Beta Was this translation helpful? Give feedback.
-
|
Also I would most likely to get rid of whole |
Beta Was this translation helpful? Give feedback.
-
Why get rid of the whole app.html? I find it useful to add extra javascript and css links outside of the layout file. |
Beta Was this translation helpful? Give feedback.
-
|
How can I get rid of it. |
Beta Was this translation helpful? Give feedback.
-
They are just markups help the SSR server know where to place their rendering result. They're not parsed into XML tree so using a tag for either head or body is not useful. Also tag variant would confuse the rendering server. Ex, |
Beta Was this translation helpful? Give feedback.
-
|
I personally find it somewhat confusing that app.html is not parsed as HTML and is thus not preprocessed and minified during SSR, so maybe this is worth thinking through more. |
Beta Was this translation helpful? Give feedback.
-
It's actually been preprocess by ViteJS during both SSR and CSR including static assets handling. The markups |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
-
As Rich would say, we are a compiler, we can do whatever we want.
So following the line of
__layout.svelte is uglyso are%sveltekit.head%and%sveltekit.body%Why not change them to something like
<sveltekit:head />and<sveltekit:body />or anything that makes them look less out of place.Beta Was this translation helpful? Give feedback.
All reactions