<meta name="description or keywords"> are always used site variables

[fantasticmao]

Hello, I found that the meta tag <meta name="description"> and <meta name="keywords"> use the Hugo site variables on every page. I checked the source code, it is because of the following code snippet.

hugo-coder/layouts/_default/baseof.html

Lines 9 to 10 in acb4bf6

	{{ with .Site.Params.description }}<meta name="description" content="{{ . }}">{{ end }}
	{{ with .Site.Params.keywords }}<meta name="keywords" content="{{ . }}">{{ end }}

Hmmm, I think this is actually not SEO friendly.

[clement-pannetier]

Hi @fantasticmao, that's true. I think that this feature has not been implemented by @luizdepra.
So you would want keywords and description for each posts for example ?

[clement-pannetier]

After some tests locally, you can change the description and keywords of your posts with the front matter attributes but it only affects the open graph and twitter cards meta tags, and only for the description since there are no meta for keywords.

[fantasticmao]

Hi @clement-pannetier, that is what I really need.

Actually, I have implemented it by the custom layouts/_default/baseof.html file, you can check the following code snippet:

https://github.com/FantasticMao/blog/blob/e55d6e9b9c94e4b2854cf4f7a7d6d88693e7c6e7/layouts/_default/baseof.html#L10-L36

I wonder if my implementation is correct?

[clement-pannetier]

Hi @clement-pannetier, that is what I really need.

Actually, I have implemented it by the custom layouts/_default/baseof.html file, you can check the following code snippet:

https://github.com/FantasticMao/blog/blob/e55d6e9b9c94e4b2854cf4f7a7d6d88693e7c6e7/layouts/_default/baseof.html#L10-L36

I wonder if my implementation is correct?

Yes it seems correct, but the code is a bit redundant. I've implemented my own version thanks to the internal templates hugo already provides here

<meta property="description" content="{{ with .Description }}{{ . }}{{ else }}{{if .IsPage}}{{ .Summary }}{{ else }}{{ with .Site.Params.description }}{{ . }}{{ end }}{{ end }}{{ end }}" />

<meta property="keywords" content="{{ with .Keywords }}{{ range . }}{{ . }}{{ end }}{{ else }}{{ with .Site.Params.keywords }}{{ . }}{{ end }}{{ end }}" />

[fantasticmao]

Okay, thanks for your suggestions !! :)

[clement-pannetier]

Okay, thanks for your suggestions !! :)

@fantasticmao, If you want you can do a PR 👍

[fantasticmao]

Ok, and I found that thing is not so simple @clement-pannetier.

Keywords in the front matter may be an array type, so I need to use the Hugo function (delimit) to process it, as described in the Stack Overflow question: https://stackoverflow.com/questions/55712785/how-to-convert-a-list-of-values-into-comma-separated-string-in-hugo-template

[clement-pannetier]

Ok, and I found that thing is not so simple @clement-pannetier.

Keywords in the front matter may be an array type, so I need to use the Hugo function (delimit) to process it, as described in the Stack Overflow question: https://stackoverflow.com/questions/55712785/how-to-convert-a-list-of-values-into-comma-separated-string-in-hugo-template

I understand, I did like that because in the config.toml file, the keywords parameter is a string.

Create a pull request with your code, @luizdepra will review it later 😄

[fantasticmao]

OK, I will!