From f3fc219980e0d30104c08eaa0e0cef8994f80854 Mon Sep 17 00:00:00 2001
From: Christopher Charbonneau Wells <chris@chrxs.net>
Date: Mon, 21 Aug 2017 09:55:06 -0400
Subject: [PATCH] Add hrefs to menu links so they work on mobile.

---
 .../templates/babyblotter/nav-dropdown.html       | 15 ++++++++++-----
 core/templates/core/timer_nav.html                |  7 ++++---
 2 files changed, 14 insertions(+), 8 deletions(-)

diff --git a/babyblotter/templates/babyblotter/nav-dropdown.html b/babyblotter/templates/babyblotter/nav-dropdown.html
index 9cde96e8..a04f7fee 100644
--- a/babyblotter/templates/babyblotter/nav-dropdown.html
+++ b/babyblotter/templates/babyblotter/nav-dropdown.html
@@ -19,8 +19,9 @@
                 </li>
 
                 <li class="nav-item dropdown">
-                    <a class="nav-link dropdown-toggle"
-                       id="nav-activity-menu-link"
+                    <a id="nav-activity-menu-link"
+                       class="nav-link dropdown-toggle"
+                       href="#"
                        data-toggle="dropdown"
                        aria-haspopup="true"
                        aria-expanded="false"><i class="fa fa-child" aria-hidden="true"></i> Activities</a>
@@ -87,10 +88,14 @@
             {% if request.user %}
                 <ul class="navbar-nav ml-auto">
                     <li class="nav-item dropdown">
-                        <a class="nav-link dropdown-toggle" data-toggle="dropdown">
-                            <i class="fa fa-user" aria-hidden="true"></i> {{ request.user }}
+                        <a id="nav-user-menu-link"
+                           class="nav-link dropdown-toggle"
+                           href="#"
+                           data-toggle="dropdown"
+                           aria-haspopup="true"
+                           aria-expanded="false"><i class="fa fa-user" aria-hidden="true"></i> {{ request.user }}
                         </a>
-                        <div class="dropdown-menu">
+                        <div class="dropdown-menu" aria-labelledby="nav-user-menu-link">
                             <a href="{% url 'api:api-root' %}"
                                class="dropdown-item"
                                target="_blank">API Browser</a>
diff --git a/core/templates/core/timer_nav.html b/core/templates/core/timer_nav.html
index 26ffd44c..e868a7ea 100644
--- a/core/templates/core/timer_nav.html
+++ b/core/templates/core/timer_nav.html
@@ -1,10 +1,11 @@
 <li class="nav-item dropdown">
-    <a class="nav-link dropdown-toggle"
-       id="timer-dropdown-menu-link"
+    <a id="nav-timer-menu-link"
+       class="nav-link dropdown-toggle"
+       href="#"
        data-toggle="dropdown"
        aria-haspopup="true"
        aria-expanded="false"><i class="fa fa-clock-o" aria-hidden="true"></i> Timers</a>
-    <div class="dropdown-menu" aria-labelledby="timer-dropdown-menu-link">
+    <div class="dropdown-menu" aria-labelledby="nav-timer-menu-link">
         {% if perms.core.add_timer %}
             <a class="dropdown-item" href="{% url 'timer-add-quick' %}">
                 <i class="fa fa-plus" aria-hidden="true"></i> Create Timer