From c413c4e4fb689cc2a9d374ab9409a06a049c9a06 Mon Sep 17 00:00:00 2001
From: Sebastiaan van Stijn <github@gone.nl>
Date: Mon, 26 Oct 2020 18:55:10 +0100
Subject: [PATCH 1/2] _scss: remove unneeded $clear variable

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
---
 _scss/_base.scss      | 4 ++--
 _scss/_variables.scss | 1 -
 2 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/_scss/_base.scss b/_scss/_base.scss
index ab25d13e3c..0edb4c37a0 100755
--- a/_scss/_base.scss
+++ b/_scss/_base.scss
@@ -11,8 +11,8 @@ body {
     color: $body-text;
     font-family: $font;
     font-size: $body-text-size;
-    margin: $clear;
-    padding: $clear;
+    margin: 0;
+    padding: 0;
     overflow-x: hidden;
 }
 
diff --git a/_scss/_variables.scss b/_scss/_variables.scss
index 2fa14b3776..362e2fb1b0 100755
--- a/_scss/_variables.scss
+++ b/_scss/_variables.scss
@@ -11,7 +11,6 @@ $global-header-height:  440px;
 $bg-secondary-tabs:     rgba(0, 0, 0, 0.05);
 $white: #fff;
 $black: #000;
-$clear: 0;
 
 /*
  * standard mode

From 3c3b037d9a18fa562326e3353f498c8b101093a6 Mon Sep 17 00:00:00 2001
From: Sebastiaan van Stijn <github@gone.nl>
Date: Mon, 26 Oct 2020 18:56:17 +0100
Subject: [PATCH 2/2] js/docs.js: small cleanup in walkTree() function

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
---
 js/docs.js | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/js/docs.js b/js/docs.js
index 7c52025925..9e4baeafc2 100644
--- a/js/docs.js
+++ b/js/docs.js
@@ -28,17 +28,17 @@ function pageIsInSection(tree) {
 }
 
 function walkTree(tree) {
-    for (let j = 0; j < tree.length; j++) {
+    for (const page of tree) {
         totalTopics++;
-        if (tree[j].section) {
-            let sectionHasPath = pageIsInSection(tree[j].section);
+        if (page.section) {
+            let sectionHasPath = pageIsInSection(page.section);
             outputLetNav.push('<li><a onclick="navClicked(' + totalTopics + ')" data-target="#item' + totalTopics + '" data-toggle="collapse" data-parent="#stacked-menu"')
             if (sectionHasPath) {
                 outputLetNav.push('aria-expanded="true"')
             } else {
                 outputLetNav.push('class="collapsed" aria-expanded="false"')
             }
-            outputLetNav.push(">" + tree[j].sectiontitle + '<span class="caret arrow"></span></a>');
+            outputLetNav.push(">" + page.sectiontitle + '<span class="caret arrow"></span></a>');
             outputLetNav.push('<ul class="nav collapse');
             if (sectionHasPath) outputLetNav.push(" in");
             outputLetNav.push('" id="#item' + totalTopics + '" aria-expanded="');
@@ -48,15 +48,15 @@ function walkTree(tree) {
                 outputLetNav.push("false");
             }
             outputLetNav.push('">');
-            walkTree(tree[j].section);
+            walkTree(page.section);
             outputLetNav.push("</ul></li>");
         } else {
             // just a regular old topic; this is a leaf, not a branch; render a link!
-            outputLetNav.push('<li><a href="' + tree[j].path + '"')
-            if (tree[j].path === pageURL && !tree[j].nosync) {
+            outputLetNav.push('<li><a href="' + page.path + '"')
+            if (page.path === pageURL && !page.nosync) {
                 outputLetNav.push('class="active currentPage"')
             }
-            outputLetNav.push(">" + tree[j].title + "</a></li>")
+            outputLetNav.push(">" + page.title + "</a></li>")
         }
     }
 }