From bdaadeefd3ccf7f2682cfb7c391aae72f460e75d Mon Sep 17 00:00:00 2001 From: Martin Taillefer Date: Fri, 23 Mar 2018 07:12:32 -0700 Subject: [PATCH] Improve sorting algorithm to use document title and not just document URL. (#1089) This makes it so documents in the same directory get sorted by document title instead of by the URL name (unless they have an order: directive, which takes precedence over alpha order) --- _includes/sort-hierarchy.html | 18 ++++++++++++++---- 1 file changed, 14 insertions(+), 4 deletions(-) diff --git a/_includes/sort-hierarchy.html b/_includes/sort-hierarchy.html index 8b93790d44..c67e8f7a38 100644 --- a/_includes/sort-hierarchy.html +++ b/_includes/sort-hierarchy.html @@ -63,15 +63,25 @@ Note: {% assign order = order | slice: 0, len | append: "#####" %} {% endif %} - {% assign a = a | append: "^" | append: order | append: "$" | append: doc.url | append: "$" | append: doc.title | append: "$" | append: doc.overview %} + + {% assign hackUrl = "" %} + {% for i in (1..count) %} + {% assign hackUrl = hackUrl | append: "/" | append: components[i] %} + {% endfor %} + {% assign hackUrl = hackUrl | append: "/" | append: doc.title %} + + {% assign a = a | append: "^" | append: order | append: "$" | append: hackUrl | append: "$" | append: doc.url | append: "$" | append: doc.title | + append: "$" | append: doc.overview %} {% endfor %} {% assign sorted = a | split: "^" | sort %} {% for s in sorted %} {% assign parts = s | split: "$" %} - {% assign urls = urls | append: "$" | append: parts[1] %} - {% assign titles = titles | append: "$" | append: parts[2] %} - {% assign overviews = overviews | append: "$" | append: parts[3] %} + {% assign urls = urls | append: "$" | append: parts[2] %} + {% assign titles = titles | append: "$" | append: parts[3] %} + {% assign overviews = overviews | append: "$" | append: parts[4] %} {% endfor %} {% assign urlslen = urls | size | minus: 2 %}