Remove JS redirect and improve SEO on multi-lang page

Author: thibaudgg

data/site.yml

@@ -14,3 +14,23 @@ author:
   name: 'Thibaud Guillaume-Gentil'
   email: 'thibaud@thibaud.gg'
   website: 'https://thibaud.gg'
+
+jsonld:
+  "@context": "http://schema.org"
+  "@type": "WebSite"
+  "url": "https://csa-admin.org"
+  "inLanguage": "en"
+  "potentialAction":
+    "@type": "ReadAction"
+    "target":
+      - "@type": "EntryPoint"
+        "urlTemplate": "https://csa-admin.org"
+        "inLanguage": "en"
+      - "@type": "EntryPoint"
+        "urlTemplate": "https://csa-admin.org/acp"
+        "inLanguage": "fr"
+      - "@type": "EntryPoint"
+        "urlTemplate": "https://csa-admin.org/solawi"
+        "inLanguage": "de"
+
+

source/layouts/layout.slim

@@ -8,40 +8,19 @@ html class='bg-white antialiased scroll-smooth' lang='fr'
     meta name='viewport' content='width=device-width, initial-scale=1, shrink-to-fit=no'
     meta name='Description' content = t('site.description')
     meta name='author' content = data.site.author.name
+    meta name="robots" content="index, follow"
 
-    == auto_display_favicon_tags
-    == stylesheet_link_tag 'site'
-
-    javascript:
-      // Redirect to the correct language landing page
-      javascript:
-        document.addEventListener('DOMContentLoaded', function() {
-          var languageRedirected = localStorage.getItem('languageRedirected')
-          if (languageRedirected) {
-            return
-          }
-
-          var userAgent = navigator.userAgent.toLowerCase();
-          if (/bot|crawl|spider|slurp|bing|msn|teoma|baidu|yandex|duckduckbot|facebot|ia_archiver/.test(userAgent)) {
-            return;
-          }
+    link rel="canonical" href="https://csa-admin.org"
+    link rel="alternate" hreflang="en" href="https://csa-admin.org"
+    link rel="alternate" hreflang="fr" href="https://csa-admin.org/acp"
+    link rel="alternate" hreflang="de" href="https://csa-admin.org/solawi"
+    link rel="alternate" hreflang="x-default" href="https://csa-admin.org"
 
-          localStorage.setItem('languageRedirected', true)
-          var browserLanguage = navigator.language ? navigator.language.slice(0, 2) : null
+    script type="application/ld+json"
+      == data.site.jsonld.to_json
 
-          if (!browserLanguage || window.location.pathname !== '/') {
-            return
-          }
-
-          switch (browserLanguage) {
-            case 'fr':
-              window.location.pathname = '/acp'
-              break
-            case 'de':
-              window.location.pathname = '/solawi'
-              break
-          }
-        })
+    == auto_display_favicon_tags
+    == stylesheet_link_tag 'site'
 
   body class='font-sans md:mx-auto md:max-w-7xl'
     .wrapper class="mx-4 md:mx-12"

source/sitemap.xml.builder

@@ -1,11 +1,27 @@
-xml.instruct!
-xml.urlset 'xmlns' => "http://www.sitemaps.org/schemas/sitemap/0.9" do
-  sitemap.resources.select { |page| page.path =~ /\.html/ }.each do |page|
-    path = page.destination_path == "index.html" ? "" : "/#{page.destination_path}"
-    xml.url do
-      xml.loc "https://csa-admin.org#{path}"
-      xml.changefreq "weekly"
-      xml.priority path.blank? ? "1.0" : "0.5"
-    end
+xml.instruct! :xml, version: "1.0", encoding: "UTF-8"
+xml.urlset("xmlns" => "http://www.sitemaps.org/schemas/sitemap/0.9", "xmlns:xhtml" => "http://www.w3.org/1999/xhtml") do
+  xml.url do
+    xml.loc "https://csa-admin.org"
+    xml.changefreq "weekly"
+    xml.priority "1.0"
+    xml.tag!("xhtml:link", rel: "alternate", hreflang: "fr", href: "https://csa-admin.org/acp")
+    xml.tag!("xhtml:link", rel: "alternate", hreflang: "de", href: "https://csa-admin.org/solawi")
+    xml.tag!("xhtml:link", rel: "alternate", hreflang: "x-default", href: "https://csa-admin.org")
+  end
+
+  xml.url do
+    xml.loc "https://csa-admin.org/acp"
+    xml.changefreq "weekly"
+    xml.priority "0.9"
+    xml.tag!("xhtml:link", rel: "alternate", hreflang: "en", href: "https://csa-admin.org")
+    xml.tag!("xhtml:link", rel: "alternate", hreflang: "de", href: "https://csa-admin.org/solawi")
+  end
+
+  xml.url do
+    xml.loc "https://csa-admin.org/solawi"
+    xml.changefreq "weekly"
+    xml.priority "0.9"
+    xml.tag!("xhtml:link", rel: "alternate", hreflang: "en", href: "https://csa-admin.org")
+    xml.tag!("xhtml:link", rel: "alternate", hreflang: "fr", href: "https://csa-admin.org/acp")
   end
 end