From 4cf5ac2c02c028a5b3cf96249982f6c987721e5d Mon Sep 17 00:00:00 2001
From: Jennifer Taylor <dragonminded@dragonminded.com>
Date: Fri, 8 May 2020 20:07:33 +0000
Subject: [PATCH] Fix visual bug where navigating to admin/arcades/ would
 inadvertently highlight the "Arcades" top level as well.

---
 bemani/frontend/templates/base.html | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/bemani/frontend/templates/base.html b/bemani/frontend/templates/base.html
index c001be3..a4968b8 100644
--- a/bemani/frontend/templates/base.html
+++ b/bemani/frontend/templates/base.html
@@ -110,18 +110,21 @@
         <div class="navigation">
             <ul class="navigation" id="nav">
                 {% for entry in navigation %}
-                <li class="{{ 'right' if entry.get('right_justify') else 'left' }} {{ 'selected' if entry['uri'] == current_path or (((entry.get('base_uri', '/nonexistent') or '/nonexistent') in current_path) and not (any(entry.get('entries'), 'uri', current_path))) else '' }}">
+                    <li class="{{
+                        'right' if entry.get('right_justify') else 'left' }}
+                        {{ 'selected' if entry['uri'] == current_path or ((current_path.startswith(entry.get('base_uri', '/nonexistent') or '/nonexistent')) and not (any(entry.get('entries'), 'uri', current_path))) else ''
+                    }}">
                         {% with uri = entry.get('uri') %}
                             {% with label = entry.get('label') %}
                                 {% if label %}
                                     {% if uri %}
                                     <a
-                                        class="{{ 'current' if (entry['uri'] == current_path) or (any(entry.get('entries'), 'uri', current_path)) or ((entry.get('base_uri', '/nonexistent') or '/nonexistent') in current_path) else ''}} {{entry['gamecode']}} border"
+                                        class="{{ 'current' if (entry['uri'] == current_path) or (any(entry.get('entries'), 'uri', current_path)) or (current_path.startswith(entry.get('base_uri', '/nonexistent') or '/nonexistent')) else ''}} {{entry['gamecode']}} border"
                                         href="{{ entry['uri'] }}"
                                     >{{ entry['label'] }}</a>
                                     {% else %}
                                     <a
-                                        class="cursor {{ 'current' if (entry['uri'] == current_path) or (any(entry.get('entries'), 'uri', current_path)) or ((entry.get('base_uri', '/nonexistent') or '/nonexistent') in current_path) else ''}} {{entry['gamecode']}} border"
+                                        class="cursor {{ 'current' if (entry['uri'] == current_path) or (any(entry.get('entries'), 'uri', current_path)) or (current_path.startswith(entry.get('base_uri', '/nonexistent') or '/nonexistent')) else ''}} {{entry['gamecode']}} border"
                                     >{{ entry['label'] }}</a>
                                     {% endif %}
                                 {% else %}