From 09652440b84a77b4a3e6aee41d80a9343d2babb9 Mon Sep 17 00:00:00 2001
From: perennial <mail@perennialte.ch>
Date: Tue, 8 Oct 2024 15:15:32 +1100
Subject: [PATCH] only display adjacent works in NovelNav

---
 assets/views/blocks/novelnav.jet.html | 45 +++++++++++++++++----------
 assets/views/novel.jet.html           | 10 ++++--
 2 files changed, 37 insertions(+), 18 deletions(-)

diff --git a/assets/views/blocks/novelnav.jet.html b/assets/views/blocks/novelnav.jet.html
index fc8fec0..f3ee626 100644
--- a/assets/views/blocks/novelnav.jet.html
+++ b/assets/views/blocks/novelnav.jet.html
@@ -1,22 +1,35 @@
 {{- block NovelNav(baseURL, paths, names, activeState="") }}
-  <div class="row nav nav-underline justify-content-center bg-charcoal-surface2 rounded py-4 mx-4">
+  <!-- NOTE: Border or no border? cant decide -->
+  <div class="row nav nav-underline justify-content-center border border-1 rounded py-4 mx-4">
+      {{- activeIndex := -1 }}
       {{- range i, k := paths }}
-      <div class="col-12 col-lg-3">
-          {{ if k == activeState }}
-          <span class="nav-link text-center
-              {{ activeState == k ? "active border-bottom border-4 border-custom-color fw-bold" : "text-body-secondary fw-semibold" }}"
-              {{ activeState == k ? `aria-current="page"` : "" }}
-              >
-              {{ names[i] }}
-          </span>
-          {{- else }}
-          <a class="nav-link text-center
-              {{ activeState == k ? "active border-bottom border-4 border-custom-color fw-bold" : "text-body-secondary fw-semibold" }}"
-              {{ activeState == k ? `aria-current="page"` : "" }}
-              href="{{ baseURL }}{{k}}">
-              {{ names[i] }}
-          </a>
+          {{- if k == activeState }}
+              {{- activeIndex = i }}
           {{- end }}
+      {{- end }}
+
+      {{- if activeIndex > 0 }}
+      <div class="col-12 col-lg-3">
+          <a class="nav-link text-center text-body-secondary fw-semibold"
+             href="{{ baseURL }}{{paths[activeIndex-1]}}">
+             {{ names[activeIndex-1] }}
+          </a>
+      </div>
+      {{- end }}
+
+      <div class="col-12 col-lg-3">
+          <span class="nav-link text-center active border-bottom border-4 border-custom-color fw-bold"
+                aria-current="page">
+              {{ names[activeIndex] }}
+          </span>
+      </div>
+
+      {{- if activeIndex < len(paths) - 1 }}
+      <div class="col-12 col-lg-3">
+          <a class="nav-link text-center text-body-secondary fw-semibold"
+             href="{{ baseURL }}{{paths[activeIndex+1]}}">
+             {{ names[activeIndex+1] }}
+          </a>
       </div>
       {{- end }}
   </div>
diff --git a/assets/views/novel.jet.html b/assets/views/novel.jet.html
index 03b5a61..333584e 100644
--- a/assets/views/novel.jet.html
+++ b/assets/views/novel.jet.html
@@ -210,7 +210,7 @@
           </div>
 
           <!-- Series navigation, if present -->
-          <!-- NOTE: col element unconditionally for spacing -->
+          <!-- NOTE: col element rendered unconditionally for spacing -->
           <div class="col-12 my-4">
           {{- if .Novel.SeriesNavData }}
             <h2 class="text-center mb-2">
@@ -219,8 +219,14 @@
               </a>
             </h2>
             <h3 class="text-body-secondary text-center mb-4">
+              <!-- NOTE: the logic below is necessary because some series consist of only a single work, and "1 works" would be incorrect -->
               <a href="/novel/series/{{ .Novel.SeriesNavData.SeriesID }}" class="text-body-secondary text-decoration-none">
-                <i class="bi bi-stack me-2"></i>{{ len(.NovelSeriesContentTitles) }} works in this series
+                <i class="bi bi-stack me-2"></i>{{ len(.NovelSeriesContentTitles) }}
+                {{- if len(.NovelSeriesContentTitles) == 1 }}
+                  work in this series
+                {{- else }}
+                  works in this series
+                {{- end }}
               </a>
             </h3>