Updated condition requiring tagline or description

jekyll / jekyll-seo-tag

A Jekyll plugin to add metadata tags for search engines and social networks to better index and display your site's content.

https://jekyll.github.io/jekyll-seo-tag

MIT License

1.66k stars 294 forks source link

Updated condition requiring tagline or description #472

Open uy-rrodriguez opened 2 years ago

uy-rrodriguez commented 2 years ago

This PR updates the logic generating titles.

The title property was updated in #356 to prefer tagline instead of description. However, the condition on the if just above was not updated, so if site.description is not defined, then tagline won't be used.

imustafin commented 4 months ago

I faced a problem with a website having tagline but no description. It might be confusing that adding a description makes tagline to appear somewhere.

Just wanted to make the same PR now. What is the status of this PR?

I guess @mattr- is active in this repo. Is it possible to merge this? Maybe @uy-rrodriguez could also update the branch, if not then I can make a similar PR.