From bc6508c0d906db9e45caeb55792f0a28fd909ec1 Mon Sep 17 00:00:00 2001
From: Jeremy Stretch <jstretch@netboxlabs.com>
Date: Thu, 11 Jan 2024 10:36:15 -0500
Subject: [PATCH] Clean up nav menu styling

---
 netbox/project-static/dist/netbox.css         | Bin 543941 -> 544150 bytes
 .../styles/transitional/_navigation.scss      |  12 +++++++++---
 .../utilities/templates/navigation/menu.html  |   6 +++---
 3 files changed, 12 insertions(+), 6 deletions(-)

diff --git a/netbox/project-static/dist/netbox.css b/netbox/project-static/dist/netbox.css
index 1ba0252274e709f9c974328404a1b69bbd4cbbe0..8c28a6c30162fb661ef169be0e2895c654a28df3 100644
GIT binary patch
delta 76
zcmX>)Q*qjC#fBEf7N!>F7M2#)7Pc1lEgTO$7^|lz{$&xHKF5<)e7fHk79-Y-{Ib-d
d>4}q=gr}!_a)h&4K?MG<V3XYb&x7MQGXN(C93KDx

delta 36
scmbO>Tk+^j#fBEf7N!>F7M2#)7Pc1lEgTO$rpu-<%5FdJ$#I++0Q6`K#sB~S

diff --git a/netbox/project-static/styles/transitional/_navigation.scss b/netbox/project-static/styles/transitional/_navigation.scss
index e676d7c98..0d2571cfc 100644
--- a/netbox/project-static/styles/transitional/_navigation.scss
+++ b/netbox/project-static/styles/transitional/_navigation.scss
@@ -3,10 +3,16 @@
   .navbar-collapse {
     .dropdown-menu {
       .dropdown-item {
+        a {
+          color: inherit;
+        }
 
-        // Remove underline from nav menu items
-        a:hover {
-          text-decoration: none;
+        // Adjust hover color & style for menu items
+        &:hover {
+          background-color: $gray-700;
+          a {
+            text-decoration: none;
+          }
         }
 
         // Style active menu item
diff --git a/netbox/utilities/templates/navigation/menu.html b/netbox/utilities/templates/navigation/menu.html
index 776006443..18be3c717 100644
--- a/netbox/utilities/templates/navigation/menu.html
+++ b/netbox/utilities/templates/navigation/menu.html
@@ -19,12 +19,12 @@
         <div class="dropdown-menu-columns">
           <div class="dropdown-menu-column pb-2">
             {% for group, items in groups %}
-              <div class="dropdown-item text-uppercase fw-bold fs-5 ps-3 pt-3 pb-1">
+              <div class="text-uppercase fw-bold fs-5 ps-3 pt-3 pb-1">
                 {{ group.label }}
               </div>
               {% for item, buttons in items %}
-                <div class="dropdown-item d-flex justify-content-between ps-3 py-1">
-                  <a href="{% url item.link %}" class="d-inline-flex flex-fill">{{ item.link_text }}</a>
+                <div class="dropdown-item d-flex justify-content-between ps-3 py-0">
+                  <a href="{% url item.link %}" class="d-inline-flex flex-fill py-1">{{ item.link_text }}</a>
                   {% if buttons %}
                     <div class="btn-group ms-1">
                       {% for button in buttons %}