Skip to content

Linter Rule: Allow head-only elements on top level #641

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
marcoroth merged 1 commit into from
Oct 14, 2025
Merged

Linter Rule: Allow head-only elements on top level #641

marcoroth merged 1 commit into from
Oct 14, 2025

Conversation

@marcoroth
Copy link
Owner

@marcoroth marcoroth commented Oct 14, 2025 •
edited
Loading

This pull request updates the html-head-only-elements linter rule to allow head-only elements on the top-level, like:

<meta>
<link>
<base>
<title></title>
<style></style>

This is useful, when the content of <head> element in the layout is extracted to a partial, so that the partial itself doesn't contain the <head> element directly. This is now allowed.
@marcoroth marcoroth merged commit 59a4f5d into main Oct 14, 2025
7 checks passed
@marcoroth marcoroth deleted the html-head-only-improvements branch October 14, 2025 19:15
@asilano asilano mentioned this pull request Oct 21, 2025
Merged
asilano pushed a commit to fac/herb that referenced this pull request Oct 21, 2025
@marcoroth @asilano
Linter Rule: Allow head-only elements on top level (marcoroth#641)
fc4de2b 
This pull request updates the `html-head-only-elements` linter rule to
allow head-only elements on the top-level, like:

```html
<meta>
<link>
<base>
<title></title>
<style></style>
```

This is useful, when the content of `<head>` element in the layout is
extracted to a partial, so that the partial itself doesn't contain the
`<head>` element directly. This is now allowed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

linter linter-rule

2 participants

@marcoroth