The docs and the starter project seem to disagree about putting everything in html tags in layouts.

[MikeMcDonald83]

📚 Subject area/topic

layouts

📋 Page(s) affected (or suggested, for new content)

https://docs.astro.build/en/basics/layouts/

📋 Description of content that is out-of-date or incorrect

In the docs about layouts, it says "However, if a layout component does contain a page shell, its element must be the parent of all other elements in the component. All <style> or <script> elements must be enclosed by the tags."

In the barebones project after 'npm create astro', the Layout.astro file has the style tag outside of the html tag. I don't know if this is actually inconsistent. Maybe the style tag is being used in a different way in the starter. I just noticed it while I was learning Astro.

(If this issue should have gone elsewhere, my bad.)

🖥️ Reproduction in StackBlitz (if reporting incorrect content or code samples)

No response

[sarah11918]

Thanks for bringing this up and I'll look into it! I know at one point we had a technical restriction that caused us to add that line to docs, but the basics template is brand new for v5 and maybe that restriction doesn't exist anymore?

[sarah11918]

It does seem like this is not currently an issue, so I think the fix is to remove this sentence entirely from the Layouts page!

Will add help-wanted and good first issue and we would love a PR to remover this sentence/paragraph!

[jsparkdev]

This issue has been resolved in #10690.

[MikeMcDonald83]

Sorry if I was supposed to do something. I don't know much about PRs and stuff. Seems like it's resolved though. Thanks!

[sarah11918]

No need to apologize @MikeMcDonald83 ! You did the right thing by reporting it, then the community can work together on a fix... which we did! All went perfectly as it should. 💜

[jsparkdev]

Thank you all. This issue appears to be resolved, so I close it.