From 1fefdd07f96b3ab6580fbbb9eb7cea73352d2470 Mon Sep 17 00:00:00 2001
From: Nicolas MASSE <nicolas.masse@itix.fr>
Date: Fri, 15 Jan 2021 09:03:00 +0100
Subject: [PATCH] Improve Web Vitals: SEO and Accessibility

---
 assets/css/itix.css        |  2 +-
 i18n/en.yaml               |  2 ++
 i18n/fr.yaml               |  2 ++
 layouts/partials/head.html | 20 ++++++++++++++++----
 layouts/partials/menu.html |  6 ++++--
 5 files changed, 25 insertions(+), 7 deletions(-)

diff --git a/assets/css/itix.css b/assets/css/itix.css
index 4548401..5119d78 100644
--- a/assets/css/itix.css
+++ b/assets/css/itix.css
@@ -2,7 +2,7 @@
   /*
    * Theme colors 
    */
-  --main-color: #1975d2; /* Adjusted to ensure a 4.5:1 contrast ratio for accessibility */
+  --main-color: #1870c9; /* Adjusted to ensure a 4.5:1 contrast ratio for accessibility */
   --main-background-color: #fcfcfc;
   --alternate-text-color: #145ca4;
   --main-text-color: #092949; /* 50% darker than main-color */
diff --git a/i18n/en.yaml b/i18n/en.yaml
index 225c4c4..304acf5 100644
--- a/i18n/en.yaml
+++ b/i18n/en.yaml
@@ -34,3 +34,5 @@
   translation: Speaking
 - id: type_page
   translation: Misc.
+- id: show_menu
+  translation: Show the menu
diff --git a/i18n/fr.yaml b/i18n/fr.yaml
index 7788d5a..686e7bf 100644
--- a/i18n/fr.yaml
+++ b/i18n/fr.yaml
@@ -34,3 +34,5 @@
   translation: Conférence
 - id: type_page
   translation: Divers
+- id: show_menu
+  translation: Afficher le menu
diff --git a/layouts/partials/head.html b/layouts/partials/head.html
index d9992d0..64474bd 100644
--- a/layouts/partials/head.html
+++ b/layouts/partials/head.html
@@ -26,10 +26,22 @@
 <link rel="preload" href="/icons/en.svg" as="image" type="image/svg+xml">
 <link rel="preload" href="/icons/fr.svg" as="image" type="image/svg+xml">
 
-{{ if .IsHome -}}
-<meta name="description" content="{{ .Site.Params.description }}">
-{{- else -}}
-<meta name="description" content="{{ .Params.description }}">
+<!--
+  -- Ensure the <meta name=description> tag is never empty since it is used
+  -- in the SEO score.
+  -->
+{{- if .IsHome }}
+{{- with .Site.Params.description }}
+<meta name="description" content="{{ . }}">
+{{- else }}
+<meta name="description" content="{{ .Title }}">
+{{- end }}
+{{- else }}
+{{- with .Params.description }}
+<meta name="description" content="{{ . }}">
+{{- else }}
+<meta name="description" content="{{ .Title }}">
+{{- end }}
 {{- end }}
 
 <!-- Tell IE to use the Edge engine -->
diff --git a/layouts/partials/menu.html b/layouts/partials/menu.html
index ffdb318..ec6f2f4 100644
--- a/layouts/partials/menu.html
+++ b/layouts/partials/menu.html
@@ -4,8 +4,10 @@
       -- A fake / hidden checkbox is used as click reciever,
       -- so you can use the :checked selector on it.
       -->
-    <input type="checkbox" class="hamburger-menu" />
-    
+    <input type="checkbox" class="hamburger-menu" id="show-menu" />
+    <!-- Add a label for accessibility -->
+    <label class="hamburger-menu" for="show-menu" style="display: none;" >{{ i18n "show_menu" }}</label>
+
     <!-- Some spans to draw a hamburger. -->
     <div class="hamburger-menu"></div>
     <div class="hamburger-menu"></div>