From 24e029783ad7f32630fad9aac266276e167bf52c Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Miroslav=20Bajto=C5=A1?= <mbajtoss@gmail.com>
Date: Thu, 18 Jun 2020 12:31:07 +0200
Subject: [PATCH] feat: support sidebar groups with no page link
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Allow side bar entries with *some children* to have *no page* associated
with them.

Signed-off-by: Miroslav Bajtoš <mbajtoss@gmail.com>
---
 _includes/sidebar_menu_list_items.liquid | 6 +++++-
 css/customstyles.css                     | 6 +++++-
 js/customscripts.js                      | 7 +++++--
 3 files changed, 15 insertions(+), 4 deletions(-)

diff --git a/_includes/sidebar_menu_list_items.liquid b/_includes/sidebar_menu_list_items.liquid
index bd1eeac80..613bcd44b 100644
--- a/_includes/sidebar_menu_list_items.liquid
+++ b/_includes/sidebar_menu_list_items.liquid
@@ -4,7 +4,11 @@
   {% if item.output contains "web" %}
   <li class="{% if item.children.size > 0 %} tree-parent {%else%} leaf-node {% endif %} {% if pageurl.last == item.url %}active{% endif %}">
     {% if item.children.size > 0 %} <a class="show-hide" href="#"></span> {% endif %}
-    <a class="page-link" href="{{item.url}}">{{ item.title }}</a>
+    {% if item.url %}
+      <a class="page-link" href="{{item.url}}">{{ item.title }}</a>
+    {% else %}
+      <a class="section-name" href="#">{{ item.title }}</a>
+    {% endif %}
       {% if item.children.size > 0 %}
       <ul class="nav-list">
           {% include sidebar_menu_list_items.liquid menuitems=item.children %}
diff --git a/css/customstyles.css b/css/customstyles.css
index c088efc54..cdb4ddc51 100644
--- a/css/customstyles.css
+++ b/css/customstyles.css
@@ -737,17 +737,19 @@ ul#mysidebar, #mysidebar ul{
     display: block;
 }
 
+#mysidebar .section-name,
 #mysidebar .page-link {
   padding: 3px 5px;
   margin-left: 1em;
 }
 
+#mysidebar > li > .section-name,
 #mysidebar > li > .page-link {
   text-transform: uppercase;
   font-size: 11px;
   font-weight: 500;
 }
-  
+
 #mysidebar .leaf-node>.page-link::before{
   content: "▪";
   float:left;
@@ -773,6 +775,8 @@ ul#mysidebar, #mysidebar ul{
   text-decoration: none;
 }
 
+#mysidebar li.active>a.section-name,
+#mysidebar a.section-name:hover,
 #mysidebar li.active>a.page-link,
 #mysidebar a.page-link:hover{
   background: #e3eeff;
diff --git a/js/customscripts.js b/js/customscripts.js
index 6f07f347a..338a010d6 100644
--- a/js/customscripts.js
+++ b/js/customscripts.js
@@ -71,7 +71,10 @@ function setupSidebarTreeNav(){
     //add expanded class to active tree parents
     $('.tree-parent.active').addClass('expanded');
 
-    $('a.show-hide').click(function (e) {
+    $('a.show-hide').click(toggleSectionChildren);
+    $('a.section-name').click(toggleSectionChildren);
+
+    function toggleSectionChildren(e) {
       console.log('clicked', this);
       $(this)
         .blur()
@@ -79,5 +82,5 @@ function setupSidebarTreeNav(){
         .children('ul.nav-list').toggle(200);
       return false;
       // $(this).parent().children('ul.nav-list').toggle(200);
-    });
+    }
 }