From fe4f1ec4eada14b4da42a48b15c83082c112306d Mon Sep 17 00:00:00 2001
From: Pierre Equoy <pierre.equoy@protonmail.com>
Date: Mon, 17 Jan 2022 16:06:39 +0800
Subject: [PATCH 1/4] Uniformize headers in simple theme

In the simple theme, some templates are using `h1`, others are using
`h2` for the main title of the page (other than the one in the header).
This commit changes that so all of the pages are using `h1`.
---
 pelican/themes/simple/templates/article.html  | 4 ++--
 pelican/themes/simple/templates/author.html   | 2 +-
 pelican/themes/simple/templates/category.html | 2 +-
 pelican/themes/simple/templates/tag.html      | 2 +-
 4 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/pelican/themes/simple/templates/article.html b/pelican/themes/simple/templates/article.html
index c8c9a4f7..f85c7410 100644
--- a/pelican/themes/simple/templates/article.html
+++ b/pelican/themes/simple/templates/article.html
@@ -24,9 +24,9 @@
 {% block content %}
 <section id="content" class="body">
   <header>
-    <h2 class="entry-title">
+    <h1 class="entry-title">
       <a href="{{ SITEURL }}/{{ article.url }}" rel="bookmark"
-         title="Permalink to {{ article.title|striptags }}">{{ article.title }}</a></h2>
+         title="Permalink to {{ article.title|striptags }}">{{ article.title }}</a></h1>
  {% import 'translations.html' as translations with context %}
  {{ translations.translations_for(article) }}
   </header>
diff --git a/pelican/themes/simple/templates/author.html b/pelican/themes/simple/templates/author.html
index a1901946..79d22c7d 100644
--- a/pelican/themes/simple/templates/author.html
+++ b/pelican/themes/simple/templates/author.html
@@ -3,6 +3,6 @@
 {% block title %}{{ SITENAME }} - Articles by {{ author }}{% endblock %}
 
 {% block content_title %}
-<h2>Articles by {{ author }}</h2>
+<h1>Articles by {{ author }}</h1>
 {% endblock %}
 
diff --git a/pelican/themes/simple/templates/category.html b/pelican/themes/simple/templates/category.html
index 14d7ff09..d73f6e31 100644
--- a/pelican/themes/simple/templates/category.html
+++ b/pelican/themes/simple/templates/category.html
@@ -3,6 +3,6 @@
 {% block title %}{{ SITENAME }} - {{ category }} category{% endblock %}
 
 {% block content_title %}
-<h2>Articles in the {{ category }} category</h2>
+<h1>Articles in the {{ category }} category</h1>
 {% endblock %}
 
diff --git a/pelican/themes/simple/templates/tag.html b/pelican/themes/simple/templates/tag.html
index 9c958030..93878134 100644
--- a/pelican/themes/simple/templates/tag.html
+++ b/pelican/themes/simple/templates/tag.html
@@ -3,5 +3,5 @@
 {% block title %}{{ SITENAME }} - {{ tag }} tag{% endblock %}
 
 {% block content_title %}
-<h2>Articles tagged with {{ tag }}</h2>
+<h1>Articles tagged with {{ tag }}</h1>
 {% endblock %}

From 7b9a859e5e30180808794748e47e529485544b8d Mon Sep 17 00:00:00 2001
From: Pierre Equoy <pierre.equoy@protonmail.com>
Date: Mon, 17 Jan 2022 16:11:06 +0800
Subject: [PATCH 2/4] Use <main> and <article> tags in simple theme

Add a <main> tag to surround all the content blocks, so that it's easier
to target the main part of a page (be it an article, the list of posts
or the different categories) using CSS.

Because of this, the <section> part of the article.html template is made
redundant, so it is removed.

Finally, the generic <div> is replaced by an <article> tag to surround
the article's content.
---
 pelican/themes/simple/templates/article.html | 6 ++----
 pelican/themes/simple/templates/base.html    | 2 ++
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/pelican/themes/simple/templates/article.html b/pelican/themes/simple/templates/article.html
index f85c7410..6dd0d967 100644
--- a/pelican/themes/simple/templates/article.html
+++ b/pelican/themes/simple/templates/article.html
@@ -22,7 +22,6 @@
 {% endblock %}
 
 {% block content %}
-<section id="content" class="body">
   <header>
     <h1 class="entry-title">
       <a href="{{ SITEURL }}/{{ article.url }}" rel="bookmark"
@@ -60,8 +59,7 @@
     </div>
     {% endif %}
   </footer><!-- /.post-info -->
-  <div class="entry-content">
+  <article>
     {{ article.content }}
-  </div><!-- /.entry-content -->
-</section>
+  </article>
 {% endblock %}
diff --git a/pelican/themes/simple/templates/base.html b/pelican/themes/simple/templates/base.html
index 2c17dbfb..b1ea57da 100644
--- a/pelican/themes/simple/templates/base.html
+++ b/pelican/themes/simple/templates/base.html
@@ -51,8 +51,10 @@
           {% endfor %}
         {% endif %}
         </ul></nav><!-- /#menu -->
+        <main>
         {% block content %}
         {% endblock %}
+        </main>
         <footer id="contentinfo" class="body">
                 <address id="about" class="vcard body">
                 Proudly powered by <a href="https://getpelican.com/">Pelican</a>,

From 4794752dd9ecd4f80eac52f84bd15abbf5bb9d4b Mon Sep 17 00:00:00 2001
From: Pierre Equoy <pierre.equoy@protonmail.com>
Date: Mon, 17 Jan 2022 16:14:19 +0800
Subject: [PATCH 3/4] Add a viewport meta tag in simple theme for better mobile
 support

See MDN article about this:

https://developer.mozilla.org/en-US/docs/Web/HTML/Viewport_meta_tag
---
 pelican/themes/simple/templates/base.html | 1 +
 1 file changed, 1 insertion(+)

diff --git a/pelican/themes/simple/templates/base.html b/pelican/themes/simple/templates/base.html
index b1ea57da..6ac81e26 100644
--- a/pelican/themes/simple/templates/base.html
+++ b/pelican/themes/simple/templates/base.html
@@ -5,6 +5,7 @@
         <title>{% block title %}{{ SITENAME }}{% endblock title %}</title>
         <meta charset="utf-8" />
         <meta name="generator" content="Pelican" />
+        <meta name="viewport" content="width=device-width, initial-scale=1.0">
         {% if FEED_ALL_ATOM %}
         <link href="{{ FEED_DOMAIN }}/{% if FEED_ALL_ATOM_URL %}{{ FEED_ALL_ATOM_URL }}{% else %}{{ FEED_ALL_ATOM }}{% endif %}" type="application/atom+xml" rel="alternate" title="{{ SITENAME }} Full Atom Feed" />
         {% endif %}

From 16b8a03ad92c32788cb4ceb2d8d4e0a6f670c0a1 Mon Sep 17 00:00:00 2001
From: Pierre Equoy <pierre.equoy@protonmail.com>
Date: Sat, 22 Jan 2022 21:31:37 +0800
Subject: [PATCH 4/4] Remove unnecessary ids and classes in simple theme

All of the modified HTML tags can be accessed in CSS without the need
for a dedicated id or an additional class.
---
 pelican/themes/simple/templates/base.html | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/pelican/themes/simple/templates/base.html b/pelican/themes/simple/templates/base.html
index 6ac81e26..1d8ae843 100644
--- a/pelican/themes/simple/templates/base.html
+++ b/pelican/themes/simple/templates/base.html
@@ -33,11 +33,11 @@
         {% endblock head %}
 </head>
 
-<body id="index" class="home">
-        <header id="banner" class="body">
+<body>
+        <header>
                 <h1><a href="{{ SITEURL }}/">{{ SITENAME }}{% if SITESUBTITLE %} <strong>{{ SITESUBTITLE }}</strong>{% endif %}</a></h1>
-        </header><!-- /#banner -->
-        <nav id="menu"><ul>
+        </header>
+        <nav><ul>
         {% for title, link in MENUITEMS %}
             <li><a href="{{ link }}">{{ title }}</a></li>
         {% endfor %}
@@ -51,16 +51,16 @@
             <li{% if cat == category %} class="active"{% endif %}><a href="{{ SITEURL }}/{{ cat.url }}">{{ cat }}</a></li>
           {% endfor %}
         {% endif %}
-        </ul></nav><!-- /#menu -->
+        </ul></nav>
         <main>
         {% block content %}
         {% endblock %}
         </main>
-        <footer id="contentinfo" class="body">
+        <footer>
                 <address id="about" class="vcard body">
                 Proudly powered by <a href="https://getpelican.com/">Pelican</a>,
                 which takes great advantage of <a href="https://www.python.org/">Python</a>.
                 </address><!-- /#about -->
-        </footer><!-- /#contentinfo -->
+        </footer>
 </body>
 </html>