refactor: polish base templates

- use intermediate variable for complex condition
- replace .Site with global function site

layouts/_default/single.html

@@ -1,11 +1,10 @@
 {{ define "main" }}
   {{ .Scratch.Set "scope" "single" }}
-
-
   <article>
-    {{ if .Params.showHero | default (.Site.Params.article.showHero | default false) }}
+    {{/* Hero */}}
+    {{ if .Params.showHero | default (site.Params.article.showHero | default false) }}
       {{ $heroStyle := .Params.heroStyle }}
-      {{ if not $heroStyle }}{{ $heroStyle = .Site.Params.article.heroStyle }}{{ end }}
+      {{ if not $heroStyle }}{{ $heroStyle = site.Params.article.heroStyle }}{{ end }}
       {{ $heroStyle := print "hero/" $heroStyle ".html" }}
       {{ if templates.Exists ( printf "partials/%s" $heroStyle ) }}
         {{ partial $heroStyle . }}
@@ -14,9 +13,9 @@
       {{ end }}
     {{ end }}
 
-
+    {{/* Header */}}
     <header id="single_header" class="mt-5 max-w-prose">
-      {{ if .Params.showBreadcrumbs | default (.Site.Params.article.showBreadcrumbs | default false) }}
+      {{ if .Params.showBreadcrumbs | default (site.Params.article.showBreadcrumbs | default false) }}
         {{ partial "breadcrumbs.html" . }}
       {{ end }}
       <h1 class="mt-0 text-4xl font-extrabold text-neutral-900 dark:text-neutral">
@@ -25,33 +24,24 @@
       <div class="mt-1 mb-6 text-base text-neutral-500 dark:text-neutral-400 print:hidden">
         {{ partial "article-meta/basic.html" (dict "context" . "scope" "single") }}
       </div>
-
-      {{ if not (.Params.showAuthorBottom | default ( .Site.Params.article.showAuthorBottom | default false)) }}
+      {{ if not (.Params.showAuthorBottom | default (site.Params.article.showAuthorBottom | default false)) }}
         {{ template "SingleAuthor" . }}
       {{ end }}
-
     </header>
 
+    {{/* Body */}}
     <section class="flex flex-col max-w-full mt-0 prose dark:prose-invert lg:flex-row">
-      {{ if or (and (.Params.showTableOfContents | default (.Site.Params.article.showTableOfContents | default false)) (in
-        .TableOfContents "<ul")) (.Site.Params.article.showRelatedPosts | default false)
-      }}
+      {{ $enableToc := .Params.showTableOfContents | default (site.Params.list.showTableOfContents | default false) }}
+      {{ $showToc := and $enableToc (in .TableOfContents "<ul") }}
+      {{ $showRelated := site.Params.article.showRelatedPosts | default false }}
+      {{ $topClass := cond (hasPrefix site.Params.header.layout "fixed") "lg:top-[140px]" "lg:top-10" }}
+      {{ if or $showToc $showRelated }}
         <div class="order-first lg:ml-auto px-0 lg:order-last ltr:lg:pl-8 rtl:lg:pr-8">
-          <div
-            class="toc ltr:pl-5 rtl:pr-5 print:hidden lg:sticky {{ if hasPrefix .Site.Params.header.layout "fixed" -}}
-              lg:top-[140px]
-            {{ else }}
-              lg:top-10
-            {{ end }}
-            ">
-            {{ if and (.Params.showTableOfContents | default (.Site.Params.article.showTableOfContents | default false)) (in
-              .TableOfContents "<ul")
-            }}
+          <div class="toc ltr:pl-5 rtl:pr-5 print:hidden lg:sticky {{ $topClass }}">
+            {{ if $showToc }}
               {{ partial "toc.html" . }}
             {{ end }}
-            {{ if .Site.Params.article.showRelatedPosts |
-              default false
-            }}
+            {{ if $showRelated }}
               sd
             {{ end }}
           </div>
@@ -61,28 +51,24 @@
 
       <div class="min-w-0 min-h-0 max-w-fit">
         {{ partial "series/series.html" . }}
-
-
         <div class="article-content max-w-prose mb-20">
           {{ .Content }}
-          {{ $defaultReplyByEmail := .Site.Params.replyByEmail }}
+          {{ $defaultReplyByEmail := site.Params.replyByEmail }}
           {{ $replyByEmail := default $defaultReplyByEmail .Params.replyByEmail }}
           {{ if $replyByEmail }}
             <strong class="block mt-8">
               <a
                 target="_blank"
                 class="m-1 rounded bg-neutral-300 p-1.5 text-neutral-700 hover:bg-primary-500 hover:text-neutral dark:bg-neutral-700 dark:text-neutral-300 dark:hover:bg-primary-400 dark:hover:text-neutral-800"
-                href="mailto:{{ .Site.Params.Author.email }}?subject={{ replace (printf "Reply to %s" .Title) "\"" "'" }}">
+                href="mailto:{{ site.Params.Author.email }}?subject={{ replace (printf "Reply to %s" .Title) "\"" "'" }}">
                 Reply by Email
               </a>
             </strong>
           {{ end }}
         </div>
-
-        {{ if (.Params.showAuthorBottom | default ( .Site.Params.article.showAuthorBottom | default false)) }}
+        {{ if (.Params.showAuthorBottom | default (site.Params.article.showAuthorBottom | default false)) }}
           {{ template "SingleAuthor" . }}
         {{ end }}
-
         {{ partial "series/series-closed.html" . }}
         {{ partial "sharing-links.html" . }}
         {{ partial "related.html" . }}
@@ -96,7 +82,7 @@
           {{ $path = replace $path $lang ".md" }}
         {{ end }}
         {{ $jsPage := resources.Get "js/page.js" }}
-        {{ $jsPage = $jsPage | resources.Minify | resources.Fingerprint ($.Site.Params.fingerprintAlgorithm | default "sha512") }}
+        {{ $jsPage = $jsPage | resources.Minify | resources.Fingerprint (site.Params.fingerprintAlgorithm | default "sha512") }}
         <script
           type="text/javascript"
           src="{{ $jsPage.RelPermalink }}"
@@ -104,11 +90,12 @@
           data-oid="views_{{ $path }}"
           data-oid-likes="likes_{{ $path }}"></script>
       {{ end }}
-
     </section>
+
+    {{/* Footer */}}
     <footer class="pt-8 max-w-prose print:hidden">
       {{ partial "article-pagination.html" . }}
-      {{ if .Params.showComments | default (.Site.Params.article.showComments | default false) }}
+      {{ if .Params.showComments | default (site.Params.article.showComments | default false) }}
         {{ if templates.Exists "partials/comments.html" }}
           <div class="pt-3">
             <hr class="border-dotted border-neutral-300 dark:border-neutral-600">
@@ -125,18 +112,18 @@
 {{ end }}
 
 {{ define "SingleAuthor" }}
-  {{ $authorsData := .Site.Data.authors }}
-  {{ $taxonomies := .Site.Taxonomies.authors }}
-  {{ $baseURL := .Site.BaseURL }}
+  {{ $authorsData := site.Data.authors }}
+  {{ $taxonomies := site.Taxonomies.authors }}
+  {{ $baseURL := site.BaseURL }}
   {{ $taxonomyLink := 0 }}
   {{ $showAuthor := 0 }}
-  {{ $isAuthorBottom := (.Params.showAuthorBottom | default ( .Site.Params.article.showAuthorBottom | default false)) }}
+  {{ $isAuthorBottom := (.Params.showAuthorBottom | default ( site.Params.article.showAuthorBottom | default false)) }}
 
   {{ if not (strings.HasSuffix $baseURL "/") }}
     {{ $baseURL = delimit (slice $baseURL "/") "" }}
   {{ end }}
 
-  {{ if .Params.showAuthor | default (.Site.Params.article.showAuthor | default true) }}
+  {{ if .Params.showAuthor | default (site.Params.article.showAuthor | default true) }}
     {{ $showAuthor = 1 }}
     {{ partial "author.html" . }}
   {{ end }}
@@ -151,13 +138,12 @@
       {{ end }}
 
       {{ $finalLink := $taxonomyLink }}
-      {{ $currentLang := $.Site.Language.Lang }}
-      {{ if eq $.Site.LanguagePrefix "" }}
+      {{ $currentLang := site.Language.Lang }}
+      {{ if eq site.LanguagePrefix "" }}
         {{ $finalLink = printf "%sauthors/%s/" $baseURL $author }}
       {{ else }}
         {{ $finalLink = printf "%s%s/authors/%s/" $baseURL $currentLang $author }}
       {{ end }}
-
       {{ partial "author-extra.html" (dict "context" . "data" $authorData "link" $finalLink) }}
     {{- end -}}
   {{ end }}