From 0b9cd3b98b39b770e24896923527c861fc7c7f49 Mon Sep 17 00:00:00 2001
From: sea-kelp <66500457+sea-kelp@users.noreply.github.com>
Date: Mon, 1 Jul 2024 04:30:07 +0000
Subject: [PATCH] Fix nav bar links on mobile (#1109)

---
 OpenOversight/app/templates/base.html | 33 ++++++++++++++-------------
 1 file changed, 17 insertions(+), 16 deletions(-)

diff --git a/OpenOversight/app/templates/base.html b/OpenOversight/app/templates/base.html
index 9fa84a465..218e9a818 100644
--- a/OpenOversight/app/templates/base.html
+++ b/OpenOversight/app/templates/base.html
@@ -58,7 +58,7 @@
           <span class="navbar-toggler-icon"></span>
         </button>
         <div id="navbar" class="collapse navbar-collapse">
-          <ul class="navbar-nav">
+          <ul class="navbar-nav me-auto">
             <li class="nav-item">
               <a class="nav-link" href="{{ url_for("main.browse") }}">Browse</a>
             </li>
@@ -80,30 +80,31 @@
               </li>
             {% endif %}
           </ul>
-          <ul class="navbar-nav flex-row flex-wrap ms-md-auto">
+          <ul class="navbar-nav">
             {% if current_user and current_user.is_authenticated %}
               <li class="dropdown nav-item">
-                <button class="btn dropdown-toggle"
-                        data-bs-toggle="dropdown"
-                        aria-expanded="false">
+                <a class="nav-link dropdown-toggle"
+                   data-bs-toggle="dropdown"
+                   aria-expanded="false">
                   Account <b class="caret"></b>
-                </button>
+                </a>
                 <ul class="dropdown-menu dropdown-menu-end">
-                  <li class="nav-item">
-                    <a class="nav-link"
+                  <li>
+                    <a class="dropdown-item"
                        href="{{ url_for('main.profile', username=current_user.username) }}">Profile</a>
                   </li>
-                  <li class="nav-item">
-                    <a class="nav-link" href="{{ url_for("auth.change_password") }}">Change Password</a>
+                  <li>
+                    <a class="dropdown-item" href="{{ url_for("auth.change_password") }}">Change Password</a>
                   </li>
-                  <li class="nav-item">
-                    <a class="nav-link" href="{{ url_for("auth.change_email_request") }}">Change Email</a>
+                  <li>
+                    <a class="dropdown-item"
+                       href="{{ url_for("auth.change_email_request") }}">Change Email</a>
                   </li>
-                  <li class="nav-item">
-                    <a class="nav-link" href="{{ url_for("auth.change_dept") }}">Change Default Department</a>
+                  <li>
+                    <a class="dropdown-item" href="{{ url_for("auth.change_dept") }}">Change Default Department</a>
                   </li>
-                  <li class="nav-item">
-                    <a class="nav-link" href="{{ url_for("auth.logout") }}">Log Out</a>
+                  <li>
+                    <a class="dropdown-item" href="{{ url_for("auth.logout") }}">Log Out</a>
                   </li>
                 </ul>
               </li>