From 8990aa978cd713d08c83830a3c2b020e1f5f056a Mon Sep 17 00:00:00 2001
From: rzen <rzen@nhl.com>
Date: Tue, 2 Jun 2020 21:03:13 -0400
Subject: [PATCH] refactor how global menu is compiled; also add an option to
 hide home item

---
 _includes/header.html | 19 ++++++++++++++-----
 1 file changed, 14 insertions(+), 5 deletions(-)

diff --git a/_includes/header.html b/_includes/header.html
index 2c97b13..c476f71 100644
--- a/_includes/header.html
+++ b/_includes/header.html
@@ -1,16 +1,25 @@
 <header>
 	<nav>
 		<ul>
-			<li><a href="{{ site.baseurl | append: '/' }}">{{ site.nav.home | default: "Home" }}</a>
-			{% if jekyll.environment == 'local' %}
-				<li><a href="{{ site.baseurl | append: '/private/' }}">Private</a>
+			{% if site.home.hide_home %}
+				<!-- home menu hidden per _config.yaml nav.hide_home_link -->
+			{% else %}
+				<li><a href="{{ '/' | relative_url }}">{{ site.nav.home | default: "Home" }}</a>
 			{% endif %}
-			{% for page in site.pages | sort: 'sort_order', 'title' %}
+
+			{% assign sorted_pages=(site.pages | sort: "sort_order") %}
+			
+			{% for page in sorted_pages %}
 				{% if page.categories contains 'global' %}
-					<li><a href="{{ site.baseurl | append: page.url }}">{{ page.title }}</a>
+					<li><a href="{{ page.url | relative_url }}">{{ page.title }}</a>
 				{% endif %}
 			{% endfor %}
+			
 			{% include additional_glboal_nav.html %}
+			
+			{% if jekyll.environment == 'local' %}
+				<li><a href="{{ '/private/' | relative_url }}">Private</a>
+			{% endif %}
 		</ul>
 	</nav>
 </header>
\ No newline at end of file