From 514d7a0e554c5063096ae306ef4798fcfc22e7e6 Mon Sep 17 00:00:00 2001
From: George Cushen <gcushen@users.noreply.github.com>
Date: Sat, 26 Sep 2020 17:56:25 +0100
Subject: [PATCH] fix: wrong sort order for book folder named `updates`

Use descending title name order in menu for book folder named `updates`.

Fixes `updates` sidebar sort order in Wowchemy docs site.

Fix #1839
---
 wowchemy/layouts/partials/book_sidebar.html | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/wowchemy/layouts/partials/book_sidebar.html b/wowchemy/layouts/partials/book_sidebar.html
index 968c5d68..ff1796d3 100644
--- a/wowchemy/layouts/partials/book_sidebar.html
+++ b/wowchemy/layouts/partials/book_sidebar.html
@@ -11,7 +11,7 @@
 <nav class="collapse docs-links" id="docs-nav">
   {{ $current_node := . }}
   {{ $menu_name := "" }}
-  {{ $order_by := cond (eq $menu_name "updates") "title_desc" site.Params.books.order_by }}
+  {{ $order_by := site.Params.books.order_by | default "weight" }}
   {{ $query := "" }}
   {{ $root_page := .GetPage "/_index.md" }}
   {{ $is_root := false}}
@@ -28,7 +28,12 @@
       {{- $query = .Site.Home.Sections.ByWeight -}}
     {{- end}}
   {{else}}
+    {{/* Get section name from the path. */}}
     {{ $menu_name = (path.Base (path.Split .FirstSection).Dir) }}
+
+    {{/* For any folder named `updates`, use descending title order (e.g. latest release note first). */}}
+    {{ $order_by = cond (eq $menu_name "updates") "title_desc" $order_by }}
+
     {{- if eq $order_by "title" -}}
       {{- $query = where .Site.Home.Sections.ByTitle "Section" $menu_name -}}
     {{- else if eq $order_by "title_desc" -}}