From 9d7075798892106fffbb385a5f5a728c0a6c1266 Mon Sep 17 00:00:00 2001 From: Jason McBrayer Date: Thu, 6 Dec 2018 10:14:25 -0500 Subject: [PATCH] Fix issue #40 (menu covers posting box in mobile with no JS) Don't pin the menu to the top by default -- add those classes from JavaScript. This is clearly the best thing on mobile, but it means that on desktop, without JavaScript, the menu will not be pinned to the top, which used to work. --- brutaldon/static/js/brutaldon-enhancements.js | 4 ++++ brutaldon/templates/base.html | 5 ++--- 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/brutaldon/static/js/brutaldon-enhancements.js b/brutaldon/static/js/brutaldon-enhancements.js index 1bf83c4..299ac15 100644 --- a/brutaldon/static/js/brutaldon-enhancements.js +++ b/brutaldon/static/js/brutaldon-enhancements.js @@ -23,6 +23,10 @@ function menuPrepare() { var menu = document.querySelector('#navMenu'); menu.classList.remove('is-active'); + // Pin the navbar to the top + document.querySelector('body').classList.toggle("has-navbar-fixed-top"); + document.querySelector('nav.navbar').classList.toggle("is-fixed-top"); + // Add the burger var brand = document.querySelector('.navbar-brand'); var burger = document.createElement('a'); diff --git a/brutaldon/templates/base.html b/brutaldon/templates/base.html index b0f0f8a..49bc665 100644 --- a/brutaldon/templates/base.html +++ b/brutaldon/templates/base.html @@ -51,12 +51,11 @@ {% endif %} - +
 
{% block navbar %} -