From 1f6cc0cf422f1bc94df268057c2842db2ab5e57d Mon Sep 17 00:00:00 2001
From: Alan Orth <alan.orth@gmail.com>
Date: Sat, 7 Jan 2017 11:22:45 +0200
Subject: [PATCH] layouts/_default/single.html: Adjust Disqus logic

Bring the Disqus comments inside the <article> tag. The article has
a sizeable bottom margin seprating it from the blog footer, and if
Disqus comments are active they appear after this margin, and this
looks ugly.

Semantically, I think the comments should actually be inside the
article tag anyways (since they are directly related), as well as
further wrapped in article tags, but we don't control the code that
Disqus injects so we'll have to make do with this.

See: https://www.w3.org/TR/html5/sections.html#the-article-element
---
 layouts/_default/single.html | 16 ++++++++++++----
 1 file changed, 12 insertions(+), 4 deletions(-)

diff --git a/layouts/_default/single.html b/layouts/_default/single.html
index d1144fb..8c2eb13 100644
--- a/layouts/_default/single.html
+++ b/layouts/_default/single.html
@@ -19,14 +19,22 @@
     {{ end }}
   </ul>
   {{ end }}
+
+  {{ if or (not .Site.Params.sharingicons.hide) (.Site.DisqusShortname) }}
+  <hr>
+  <footer>
+
   {{ if not .Site.Params.sharingicons.hide }}
   {{ partial "sharing-icons.html" . }}
   {{ end }}
-</article> <!-- /.blog-post -->
 
-{{ if and (.Site.DisqusShortname) (ne .Params.comments false) }}
-{{ partial "disqus.html" . }}
-{{ end }}
+  {{ if and (.Site.DisqusShortname) (ne .Params.comments false) }}
+  {{ partial "disqus.html" . }}
+  {{ end }}
+  </footer>
+  {{ end }}
+
+</article> <!-- /.blog-post -->
 
 {{ end }}