From 24351cdeb178d81a712913c7f05b38a871277cd6 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Knut=20H=C3=BChne?= <knut@viderum.com>
Date: Mon, 4 Jul 2016 14:24:59 +0200
Subject: [PATCH] Always show sidebar navigation

This change always includes the side navigation on
the left. Previously it was hidden if there was
additional content supplied by a custom sidebar
extension. This is now shown additionally above
the navigation itself.
---
 static/scss/_page.scss          | 14 +++++++++++++-
 templates/cms_twocolumn.html    |  6 ++++--
 templates/includes/sidebar.html |  2 --
 3 files changed, 17 insertions(+), 5 deletions(-)

diff --git a/static/scss/_page.scss b/static/scss/_page.scss
index 994342b6..a3f23c23 100644
--- a/static/scss/_page.scss
+++ b/static/scss/_page.scss
@@ -43,12 +43,24 @@
       nav {
         display: none;
 
+        #side-nav {
+          width: 100%;
+        }
+
+        #outer-nav{
+          overflow: hidden;
+          height: 100%;
+        }
+
         @media (min-width: $screen-md-min) {
           display: block;
-          height: 90vh;
+          height: 60vh;
+
 
           &.mm-menu {
             background-color: $body-bg;
+            height: 30vh;
+
 
             .mm-prev:before {
               left: 8px;
diff --git a/templates/cms_twocolumn.html b/templates/cms_twocolumn.html
index 277fc8ec..bfbb7213 100644
--- a/templates/cms_twocolumn.html
+++ b/templates/cms_twocolumn.html
@@ -3,10 +3,12 @@
 
 {% block body %}
 <div>
-  <nav id="side-nav">
+  <nav id='outer-nav'>
     {% static_placeholder "sitewide sidebar" %}
-    {% include 'includes/sidebar.html' %}
     {% placeholder "sidebar" inherit %}
+    <nav id="side-nav">
+      {% include 'includes/sidebar.html' %}
+    </nav>
   </nav>
   <article>
     {% placeholder "main" %}
diff --git a/templates/includes/sidebar.html b/templates/includes/sidebar.html
index 77e97e00..657d2218 100644
--- a/templates/includes/sidebar.html
+++ b/templates/includes/sidebar.html
@@ -1,7 +1,5 @@
 {% load menu_tags %}
 {% load i18n %}
-{% if request.current_page.get_descendant_count > 0 and not request.current_page.sidebarextension %}
 <ul>
   {% include 'includes/nav.html' %}
 </ul>
-{% endif %}