Skip to content
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

Add missing head tags #994

Closed
fernandolucchesi opened this issue Apr 8, 2022 · 3 comments
Closed

Add missing head tags #994

fernandolucchesi opened this issue Apr 8, 2022 · 3 comments
Assignees
@fernandolucchesi
Labels
🌐 Next.js (web)

Comments

@fernandolucchesi
Copy link
Contributor

HREF tags:

Here's a quick writeup on hreflang:

The "rules" of hreflang:

All tags are bidirectional, meaning all pages will link to each other - the code example underneath will be installed on both language-versions

All pages should link to itself - line 2 in code example beneath

X-default is recommended: this is the default when no other language matches

Example:

On https://www.equinor.com/ and https://www.equinor.com/no.html

<head>
  <link rel="alternate" hreflang="en" href="https://www.equinor.com/"/>
  <link rel="alternate" hreflang="no" href="https://www.equinor.com/no.html"/>
  <link rel="alternate" hreflang="x-default" href="https://www.equinor.com/" />
</head>

Canonical tags

A canonical tag (aka "rel canonical") is a way of telling search engines that a specific URL represents the master copy of a page.

All pages should have the tag, linking to itself.

Example frontpage:
On https://www.equinor.com/ and https://www.equinor.com/no.html

<head>
  <link rel=“canonical” href=“https://www.equinor.com/” />
</head>
@fernandolucchesi fernandolucchesi self-assigned this Apr 19, 2022
fernandolucchesi added a commit that referenced this issue Apr 19, 2022
@fernandolucchesi
✨ Add missing head tags #994
319ebef
fernandolucchesi added a commit that referenced this issue Apr 20, 2022
@fernandolucchesi
🐛 Apply correct logic for head tags #994
724351a
@SvSven
Copy link
Contributor

SvSven commented Apr 20, 2022

I can see the rel="alternate" tags on Sanity pages that have a translation, and can see the canonical one on both Sanity pages with and without a translations.

Can't see them on legacy news pages - do we need them there too?

@wenche
Copy link
Contributor

wenche commented Apr 25, 2022

@fernandolucchesi Can't see them on legacy news pages - do we need them there too? ^^

@fernandolucchesi fernandolucchesi mentioned this issue Apr 27, 2022
Closed
@fernandolucchesi
Copy link
Contributor Author

Implementation for archive news will be handled on #1037

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@fernandolucchesi fernandolucchesi

Labels
🌐 Next.js (web)
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@wenche @SvSven @fernandolucchesi