From b1fa203e3a91398f9dbc3010fefe8777a777d440 Mon Sep 17 00:00:00 2001
From: S Nitesh Singh <niteshseram@gmail.com>
Date: Mon, 19 Jul 2021 11:13:18 +0530
Subject: [PATCH] fixing the huge white space in sidebar

---
 layouts/partials/sidebar-tree.html | 108 ++++++++++++++---------------
 1 file changed, 54 insertions(+), 54 deletions(-)

diff --git a/layouts/partials/sidebar-tree.html b/layouts/partials/sidebar-tree.html
index 5e909e778f9..f1a3c9926a2 100644
--- a/layouts/partials/sidebar-tree.html
+++ b/layouts/partials/sidebar-tree.html
@@ -1,14 +1,14 @@
 {{/* We cache this partial for bigger sites and set the active class client side. */}}
-{{ $sidebarCacheLimit := cond (isset .Site.Params.ui "sidebar_cache_limit") .Site.Params.ui.sidebar_cache_limit 2000 }}
-{{ $shouldDelayActive := ge (len .Site.Pages) $sidebarCacheLimit }}
+{{ $sidebarCacheLimit := cond (isset .Site.Params.ui "sidebar_cache_limit") .Site.Params.ui.sidebar_cache_limit 2000 -}}
+{{ $shouldDelayActive := ge (len .Site.Pages) $sidebarCacheLimit -}}
 <div id="td-sidebar-menu" class="td-sidebar__inner{{ if $shouldDelayActive }} d-none{{ end }}">
-  {{ if not .Site.Params.ui.sidebar_search_disable }}
+  {{ if not .Site.Params.ui.sidebar_search_disable -}}
   <form class="td-sidebar__search d-flex align-items-center">
     {{ partial "search-input.html" . }}
     <button class="btn btn-link td-sidebar__toggle d-md-none p-0 ml-3 fas fa-bars" type="button" data-toggle="collapse" data-target="#td-section-nav" aria-controls="td-docs-nav" aria-expanded="false" aria-label="Toggle section navigation">
     </button>
   </form>
-  {{ else }}
+  {{ else -}}
   <div id="content-mobile">
   <form class="td-sidebar__search d-flex align-items-center">
     {{ partial "search-input.html" . }}
@@ -17,7 +17,7 @@
   </form>
   </div>
   <div id="content-desktop"></div>
-  {{ end }}
+  {{ end -}}
   <nav class="collapse td-sidebar-nav{{ if .Site.Params.ui.sidebar_menu_foldable }} foldable-nav{{ end }}" id="td-section-nav">
     <!-- {{ if  (gt (len .Site.Home.Translations) 0) }}
     <div class="nav-item dropdown d-block d-lg-none">
@@ -25,70 +25,70 @@
     </div>
     {{ end }} -->
     <!-- {{ $navRoot := cond (and (ne .Params.toc_root true) (eq .Site.Home.Type "docs")) .Site.Home .FirstSection }} -->
-    {{ $ulNr := 0 }}
-    {{ $ulShow := cond (isset .Site.Params.ui "ul_show") .Site.Params.ui.ul_show 1 }}
-    {{ $sidebarMenuTruncate := cond (isset .Site.Params.ui "sidebar_menu_truncate") .Site.Params.ui.sidebar_menu_truncate 50 }}
-    {{ $currentLang := .Site.Language }}
+    {{ $ulNr := 0 -}}
+    {{ $ulShow := cond (isset .Site.Params.ui "ul_show") .Site.Params.ui.ul_show 1 -}}
+    {{ $sidebarMenuTruncate := cond (isset .Site.Params.ui "sidebar_menu_truncate") .Site.Params.ui.sidebar_menu_truncate 50 -}}
+    {{ $currentLang := .Site.Language -}}
     <ul class="td-sidebar-nav__section pr-md-3 ul-{{ $ulNr }}">
       {{ template "section-tree-nav-section" (dict "page" . "section" .FirstSection "shouldDelayActive" $shouldDelayActive "sidebarMenuTruncate" $sidebarMenuTruncate "ulNr" $ulNr "ulShow" (add $ulShow 1) "currentLang" $currentLang)  }}
     </ul>
   </nav>
 </div>
-{{ define "section-tree-nav-section" }}
-  {{ $s := .section }}
-  {{ $p := .page }}
-  {{ $shouldDelayActive := .shouldDelayActive }}
-  {{ $sidebarMenuTruncate := .sidebarMenuTruncate }}
-  {{ $treeRoot := cond (eq .ulNr 0) true false }}
-  {{ $ulNr := .ulNr }}
-  {{ $ulShow := .ulShow }}
-  {{ $active := and (not $shouldDelayActive) (eq $s $p) }}
-  {{ $activePath := and (not $shouldDelayActive) ($p.IsDescendant $s) }}
-  {{ $show := cond (or (lt $ulNr $ulShow) $activePath (and (not $shouldDelayActive) (eq $s.Parent $p.Parent)) (and (not $shouldDelayActive) (eq $s.Parent $p)) (and (not $shouldDelayActive) ($p.IsDescendant $s.Parent))) true false }}
-  {{ $mid := printf "m-%s" ($s.RelPermalink | anchorize) }}
-  {{ $pages_tmp := where (union $s.Pages $s.Sections).ByWeight ".Params.toc_hide" "!=" true }}
-  {{ $pages := $pages_tmp | first $sidebarMenuTruncate }}
-  {{ $withChild := gt (len $pages) 0 }}
-  {{ $manualLink := cond (isset $s.Params "manuallink") $s.Params.manualLink ( cond (isset $s.Params "manuallinkrelref") (relref $s $s.Params.manualLinkRelref) $s.RelPermalink) }}
-  {{ $manualLinkTitle := cond (isset $s.Params "manuallinktitle") $s.Params.manualLinkTitle $s.Title }}
+{{ define "section-tree-nav-section" -}}
+  {{ $s := .section -}}
+  {{ $p := .page -}}
+  {{ $shouldDelayActive := .shouldDelayActive -}}
+  {{ $sidebarMenuTruncate := .sidebarMenuTruncate -}}
+  {{ $treeRoot := cond (eq .ulNr 0) true false -}}
+  {{ $ulNr := .ulNr -}}
+  {{ $ulShow := .ulShow -}}
+  {{ $active := and (not $shouldDelayActive) (eq $s $p) -}}
+  {{ $activePath := and (not $shouldDelayActive) ($p.IsDescendant $s) -}}
+  {{ $show := cond (or (lt $ulNr $ulShow) $activePath (and (not $shouldDelayActive) (eq $s.Parent $p.Parent)) (and (not $shouldDelayActive) (eq $s.Parent $p)) (and (not $shouldDelayActive) ($p.IsDescendant $s.Parent))) true false -}}
+  {{ $mid := printf "m-%s" ($s.RelPermalink | anchorize) -}}
+  {{ $pages_tmp := where (union $s.Pages $s.Sections).ByWeight ".Params.toc_hide" "!=" true -}}
+  {{ $pages := $pages_tmp | first $sidebarMenuTruncate -}}
+  {{ $withChild := gt (len $pages) 0 -}}
+  {{ $manualLink := cond (isset $s.Params "manuallink") $s.Params.manualLink ( cond (isset $s.Params "manuallinkrelref") (relref $s $s.Params.manualLinkRelref) $s.RelPermalink) -}}
+  {{ $manualLinkTitle := cond (isset $s.Params "manuallinktitle") $s.Params.manualLinkTitle $s.Title -}}
 
   <li class="td-sidebar-nav__section-title td-sidebar-nav__section{{ if $withChild }} with-child{{ else }} without-child{{ end }}{{ if $activePath }} active-path{{ end }}{{ if (not (or $show $p.Site.Params.ui.sidebar_menu_foldable )) }} collapse{{ end }}" id="{{ $mid }}-li">
-    {{ if (and $p.Site.Params.ui.sidebar_menu_foldable (ge $ulNr 1)) }}
+    {{ if (and $p.Site.Params.ui.sidebar_menu_foldable (ge $ulNr 1)) -}}
       <input type="checkbox" id="{{ $mid }}-check"{{ if $activePath}} checked{{ end }}/>
       <label for="{{ $mid }}-check"><a href="{{ $manualLink }}"{{ if ne $s.LinkTitle $manualLinkTitle }} title="{{ $manualLinkTitle }}"{{ end }}{{ with $s.Params.manualLinkTarget }} target="{{ . }}"{{ if eq . "_blank" }} rel="noopener"{{ end }}{{ end }} class="align-left pl-0 {{ if $active}} active{{ end }} td-sidebar-link{{ if $s.IsPage }} td-sidebar-link__page{{ else }} td-sidebar-link__section{{ end }}{{ if $treeRoot }} tree-root{{ end }}" id="{{ $mid }}">{{ with $s.Params.Icon}}<i class="{{ . }}"></i>{{ end }}<span class="{{ if $active }}td-sidebar-nav-active-item{{ end }}">{{ $s.LinkTitle }}</span></a></label>
-    {{ else }}
+    {{ else -}}
       {{ if not $treeRoot }}
       <a href="{{ $manualLink }}"{{ if ne $s.LinkTitle $manualLinkTitle }} title="{{ $manualLinkTitle }}"{{ end }}{{ with $s.Params.manualLinkTarget }} target="{{ . }}"{{ if eq . "_blank" }} rel="noopener"{{ end }}{{ end }} class="align-left pl-0{{ if $active}} active{{ end }} td-sidebar-link{{ if $s.IsPage }} td-sidebar-link__page{{ else }} td-sidebar-link__section{{ end }}" id="{{ $mid }}">{{ with $s.Params.Icon}}<i class="{{ . }}"></i>{{ end }}<span class="{{ if $active }}td-sidebar-nav-active-item{{ end }}">{{ $s.LinkTitle }}</span></a>
-      {{ end }}
-    {{ end }}
-    {{if $withChild }}
-      {{ $ulNr := add $ulNr 1 }}
+      {{ end -}}
+    {{ end -}}
+    {{ if $withChild -}}
+      {{ $ulNr := add $ulNr 1 -}}
       <ul class="ul-{{ $ulNr }}{{ if (gt $ulNr 1)}} foldable{{end}}">
-        {{ $pages := where (union $s.Pages $s.Sections).ByWeight ".Params.toc_hide" "!=" true }}
-        {{ with site.Params.language_alternatives }}
+        {{ $pages := where (union $s.Pages $s.Sections).ByWeight ".Params.toc_hide" "!=" true -}}
+        {{ with site.Params.language_alternatives -}}
           {{ range . }}
-            {{ with (where $.section.Translations ".Lang" . ) }}
-              {{ $p := index . 0 }}
-              {{ $pages =  $pages | lang.Merge (union $p.Pages $p.Sections) }}
-            {{ end }}
-          {{ end }}
-        {{ end }}
-        {{ $pages := $pages | first 50 }}
-        {{ range $pages }}
-          {{ if (not (and (eq $s $p.Site.Home) (eq .Params.toc_root true)) ) }}
-            {{ $mid := printf "m-%s" (.RelPermalink | anchorize) }}
-            {{ $active := eq . $p }}
-            {{ $isForeignLanguage := (ne (string .Lang) (string $.currentLang)) }}
-            {{ if (and $isForeignLanguage  ($p.IsDescendant $s)) }}
+            {{ with (where $.section.Translations ".Lang" . ) -}}
+              {{ $p := index . 0 -}}
+              {{ $pages =  $pages | lang.Merge (union $p.Pages $p.Sections) -}}
+            {{ end -}}
+          {{ end -}}
+        {{ end -}}
+        {{ $pages := $pages | first 50 -}}
+        {{ range $pages -}}
+          {{ if (not (and (eq $s $p.Site.Home) (eq .Params.toc_root true)) ) -}}
+            {{ $mid := printf "m-%s" (.RelPermalink | anchorize) -}}
+            {{ $active := eq . $p -}}
+            {{ $isForeignLanguage := (ne (string .Lang) (string $.currentLang)) -}}
+            {{ if (and $isForeignLanguage  ($p.IsDescendant $s)) -}}
               <a class="td-sidebar-link td-sidebar-link__page {{ if and (not $shouldDelayActive) $active }} active{{ end }}" id="{{ $mid }}" {{ if $isForeignLanguage }}target="_blank"{{ end }}  href="{{ .RelPermalink }}">
-                {{ .LinkTitle }}{{ if $isForeignLanguage }} <small>({{ .Lang | upper }})</small>{{ end }}
+                {{ .LinkTitle }}{{ if $isForeignLanguage }} <small>({{ .Lang | upper }})</small>{{ end -}}
               </a>
-            {{ else }}
+            {{ else -}}
               {{ template "section-tree-nav-section" (dict "page" $p "section" . "currentLang" $.currentLang "shouldDelayActive" $shouldDelayActive "sidebarMenuTruncate" $sidebarMenuTruncate "ulNr" $ulNr "ulShow" $ulShow) }}
-            {{ end }}
-          {{ end }}
-        {{ end }}
+            {{- end }}
+          {{- end }}
+        {{- end }}
       </ul>
-    {{ end }}
+    {{- end }}
   </li>
-{{ end }}
+{{- end }}