From e85173f55a0c3318933cb24db80a7bf359d5cfa1 Mon Sep 17 00:00:00 2001
From: =?utf8?q?Vladim=C3=ADr=20Vondru=C5=A1?= <mosra@centrum.cz>
Date: Sun, 10 Dec 2017 00:02:48 +0100
Subject: [PATCH] theme: simpler markup in minimal cases of articles.

---
 pelican-theme/templates/archives.html       | 10 ++++++----
 pelican-theme/templates/article.html        | 15 ++++++++++++---
 pelican-theme/templates/article_header.html |  5 +++--
 pelican-theme/templates/index.html          | 10 ++++++----
 4 files changed, 27 insertions(+), 13 deletions(-)

diff --git a/pelican-theme/templates/archives.html b/pelican-theme/templates/archives.html
index 2ad7421b..7ae981db 100644
--- a/pelican-theme/templates/archives.html
+++ b/pelican-theme/templates/archives.html
@@ -27,9 +27,11 @@
       <article>
         {% macro header() %}{% include "article_header.html" %}{% endmacro %}
         {{ header()|indent(8) }}
-        {% if loop.first and (not articles_page or articles_page.number == 1) and
-              ((article.collapse_first is defined and article.collapse_first != 'True') or
-               (article.collapse_first is not defined and not M_COLLAPSE_FIRST_ARTICLE)) %}
+        {% set show_content =
+            article.content and loop.first and (not articles_page or articles_page.number == 1) and
+            ((article.collapse_first is defined and article.collapse_first != 'True') or
+             (article.collapse_first is not defined and not M_COLLAPSE_FIRST_ARTICLE)) %}
+        {% if show_content %}
         <div class="m-clearfix-l"></div>
 <!-- content -->
 {{ article.content|trim }}
@@ -39,7 +41,7 @@
           {% macro footer() %}{% include "article_footer.html" %}{% endmacro %}
           {{ footer()|indent(10) }}
         </footer>
-        {% if not loop.first or (articles_page and articles_page.number != 1) %}
+        {% if not show_content %}
         <div class="m-clearfix-l"></div>
         {% endif %}
       </article>
diff --git a/pelican-theme/templates/article.html b/pelican-theme/templates/article.html
index a044ab34..03bcc708 100644
--- a/pelican-theme/templates/article.html
+++ b/pelican-theme/templates/article.html
@@ -12,8 +12,10 @@
   <meta name="twitter:title" content="{{ article.title }}" />
   <meta property="og:url" content="{{ article.url|format_siteurl }}" />
   <meta name="twitter:url" content="{{ article.url|format_siteurl }}" />
+  {% if article.summary %}
   <meta property="og:description" content="{{ article.summary|striptags }}" />
   <meta name="twitter:description" content="{{ article.summary|striptags }}" />
+  {% endif %}
   {% if article.cover %}
   <meta property="og:image" content="{{ article.cover|expand_link(article) }}" />
   <meta name="twitter:card" content="summary_large_image" />
@@ -32,8 +34,10 @@
         <div id="m-jumbo-cover">
           <div class="m-container">
             <div class="m-row">
-              <div class="m-col-t-6 m-col-s-5 m-push-s-1 m-text-left">{{ article.locale_date }}</div>
+              <div class="{% if article.authors %}m-col-t-6 m-col-s-5{% else %}m-col-t-12 m-col-s-10{% endif %} m-push-s-1 m-text-left">{{ article.locale_date }}</div>
+              {% if article.authors %}
               <div class="m-col-t-6 m-col-s-5 m-push-s-1 m-text-right">{% for author in article.authors %}<a href="{{ author.url|format_siteurl }}">{{ author }}</a>{% endfor %}</div>
+              {% endif %}
             </div>
             <div class="m-row">
               <div class="m-col-t-12 m-col-s-10 m-push-s-1 m-col-m-8 m-push-m-2">
@@ -47,8 +51,9 @@
           </div>
         </div>
       </div>
-      {% if (article.hide_summary is defined and article.hide_summary != 'True') or
-            (article.hide_summary is not defined and not M_HIDE_ARTICLE_SUMMARY) %}
+      {% if article.summary and
+            ((article.hide_summary is defined and article.hide_summary != 'True') or
+             (article.hide_summary is not defined and not M_HIDE_ARTICLE_SUMMARY)) %}
       <div class="m-container">
         <div class="m-row">
           <div class="m-col-m-10 m-push-m-1 m-nopady">
@@ -58,6 +63,7 @@
       </div>
       {% endif %}
     </header>
+    {% if article.content %}
     <div class="m-container m-container-inflatable">
       <div class="m-row">
         <div class="m-col-m-10 m-push-m-1 m-nopady">
@@ -67,6 +73,7 @@
         </div>
       </div>
     </div>
+    {% endif %}
     <footer class="m-container">
       <div class="m-row">
         <div class="m-col-m-10 m-push-m-1 m-nopadb">
@@ -81,9 +88,11 @@
       {% macro header() %}{% include "article_header.html" %}{% endmacro %}
       {{ header()|indent(6) }}
       <div class="m-clearfix-l"></div>
+      {% if article.content %}
 <!-- content -->
 {{ article.content|trim }}
 <!-- /content -->
+      {% endif %}
       <footer>
         {% macro footer() %}{% include "article_footer.html" %}{% endmacro %}
         {{ footer()|indent(8) }}
diff --git a/pelican-theme/templates/article_header.html b/pelican-theme/templates/article_header.html
index 5804e8fe..9e4a74db 100644
--- a/pelican-theme/templates/article_header.html
+++ b/pelican-theme/templates/article_header.html
@@ -11,14 +11,15 @@
       expanded only if it's first on the first page only on the index/archive
       page and only if it doesn't say it should be collapsed)
     - the article is expanded but doesn't say the summary should be hidden #}
-  {% if (articles_page and
+  {% if article.summary and
+        ((articles_page and
          (page_name not in ['index', 'archives'] or
           not loop.first or
           articles_page.number != 1 or
           (article.collapse_first is defined and article.collapse_first == 'True') or
           (article.collapse_first is not defined and M_COLLAPSE_FIRST_ARTICLE))) or
         (article.hide_summary is defined and article.hide_summary != 'True') or
-        (article.hide_summary is not defined and not M_HIDE_ARTICLE_SUMMARY) %}
+        (article.hide_summary is not defined and not M_HIDE_ARTICLE_SUMMARY)) %}
   {{ article.summary|trim|indent(2) }}
   {% endif %}
 </header>
diff --git a/pelican-theme/templates/index.html b/pelican-theme/templates/index.html
index 74a702f3..a9e2d454 100644
--- a/pelican-theme/templates/index.html
+++ b/pelican-theme/templates/index.html
@@ -22,9 +22,11 @@
       <article>
         {% macro header() %}{% include "article_header.html" %}{% endmacro %}
         {{ header()|indent(8) }}
-        {% if loop.first and articles_page.number == 1 and
-              ((article.collapse_first is defined and article.collapse_first != 'True') or
-               (article.collapse_first is not defined and not M_COLLAPSE_FIRST_ARTICLE)) %}
+        {% set show_content =
+            article.content and loop.first and articles_page.number == 1 and
+            ((article.collapse_first is defined and article.collapse_first != 'True') or
+             (article.collapse_first is not defined and not M_COLLAPSE_FIRST_ARTICLE)) %}
+        {% if show_content %}
         <div class="m-clearfix-l"></div>
 <!-- content -->
 {{ article.content|trim }}
@@ -34,7 +36,7 @@
           {% macro footer() %}{% include "article_footer.html" %}{% endmacro %}
           {{ footer()|indent(10) }}
         </footer>
-        {% if not loop.first or articles_page.number != 1 %}
+        {% if not show_content %}
         <div class="m-clearfix-l"></div>
         {% endif %}
       </article>
-- 
2.30.2