Skip to content

Commit

Permalink
Always show sidebar navigation
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
Knut Hühne committed Jul 4, 2016
1 parent fa7f0c9 commit 24351cd
Show file tree
Hide file tree
Showing 3 changed files with 17 additions and 5 deletions.
14 changes: 13 additions & 1 deletion static/scss/_page.scss
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand Down
6 changes: 4 additions & 2 deletions templates/cms_twocolumn.html
Original file line number Diff line number Diff line change
Expand Up @@ -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" %}
Expand Down
2 changes: 0 additions & 2 deletions templates/includes/sidebar.html
Original file line number Diff line number Diff line change
@@ -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 %}

0 comments on commit 24351cd

Please sign in to comment.