search

pawroman / zola-theme-terminimal

A simple, minimal retro theme for Zola
https://pawroman.github.io/zola-theme-terminimal/
Other
346 stars 76 forks source link

tag page: add / to "all tags" link #60

Closed heitorPB closed 3 weeks ago

heitorPB commented 4 months ago

This is to improve Google indexing pages, for the /tags/ page. In GitHub Pages, the URL /tags does not exist per se, but is a redirect to /tags/. Google Crawler gets confused with 301 HTTP status and does not index it.

Purely for SEO purposes.

pawroman commented 3 weeks ago

Hmm, should we add a trailing slash to all URLs then? I see similar behavior on GitLab pages. If this is added anyway, we can avoid one round trip by adding / to all the links in the top navigation pane by default.

heitorPB commented 3 weeks ago

That could lead to double trailing slashes?

I only found this issue on the tags page, did you find it in other places as well?

pawroman commented 3 weeks ago

Huh. Looks like I misunderstood the problem.

Are we talking about this link specifically? See screenshot below and the source URL.

https://pawroman.github.io/zola-theme-terminimal/tags/javascript/ image

heitorPB commented 3 weeks ago

Yes, that link.

Is there a missing / in other links as well? Not sure I got your point.

pawroman commented 3 weeks ago

For a moment I thought that github/gitlab were doing some URL manipulation when the website got deployed, but then I realized that all the URLs come from the theme.

So your fix makes sense, to make things consistent.