Client side script inside layout file makes html lang attribute disappear

[riccardolardi]

What version of astro are you using?

1.6.11

Are you using an SSR adapter? If so, which one?

None

What package manager are you using?

yarn

What operating system are you using?

Mac

Describe the Bug

It seems using a <script> inside the layout which declares the html element will make the lang attribute disappear.

Link to Minimal Reproducible Example

https://stackblitz.com/edit/github-xuenvq?file=src/layouts/Layout.astro

Participation

• I am willing to submit a pull request for this issue.

[deeprobin]

Can repro.

[JerryWu1234]

mark

[bluwy]

Seems like the entire html tag disappears entirely when checking the network request. And this seems similar to withastro/astro#5483. I'm not sure if adding a script before the html makes sense though, maybe it's nice if Astro could sort it down. Marking as a minor bug for now.

[JerryWu1234]

Because astro@compiler appears, an error that lost the tag

[jasikpark]

I'm not sure if adding a script before the html makes sense though, maybe it's nice if Astro could sort it down.

I think it makes sense for Astro to support it, since the default for components is top-level scripts and styles associated with the component. this is a weird edgecase, but isn't it the same sort of dx?

[Jarmos-san]

I've recently stumbled across this bug on my website as well and here's the source code particularly responsible for generating the base template. Is there any way I can debug and perhaps identify what the root cause of the issue is?

[skilar]

Looks like this is caused not just by the script tag but also including styles before the html tag. That should help your issue, @Jarmos-san.

[Jarmos-san]

@skilar that fixed the issue! Thanks for it. I'm curious, what was wrong with the code there?

[skilar]

I'm not sure what the issue is, @Jarmos-san, seems like there's still a bug in Astro.

[tvarwig]

Having same issue, astro 4.6.3