From 7d056b5008c7a8d67cda981c8daa7304ce00065e Mon Sep 17 00:00:00 2001
From: Paul van Tilburg <paul@luon.net>
Date: Mon, 1 Jan 2018 21:59:27 +0100
Subject: [PATCH] Hide the navbar collapse on select via JS instead of HTML

Appearantly, if there is a data-toggle, the href/link part isn't
done anymore.
---
 static/js/wishlists.js    | 2 ++
 templates/index.html.tera | 4 +---
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/static/js/wishlists.js b/static/js/wishlists.js
index 0f7f3c5..def7137 100644
--- a/static/js/wishlists.js
+++ b/static/js/wishlists.js
@@ -47,6 +47,8 @@ function selectList(listId) {
       listElem.show(200);
     });
   }
+
+  $("#navbarNav").collapse('hide');
 }
 
 function updateList(list) {
diff --git a/templates/index.html.tera b/templates/index.html.tera
index 378513a..1372ac4 100644
--- a/templates/index.html.tera
+++ b/templates/index.html.tera
@@ -5,9 +5,7 @@
   <ul class="navbar-nav mr-auto">
   {% for list in lists %}
     <li class="nav-item">
-      <a class="nav-link list-select" data-list-id="{{ list.id }}"
-         data-toggle="collapse" data-target="#navbarNav"
-         href="#{{ list.id }}">{{ list.name }}</a>
+      <a class="nav-link list-select" data-list-id="{{ list.id }}" href="#{{ list.id }}">{{ list.name }}</a>
     </li>
   {% endfor %}
   </ul>