From 43850693acf1216caa5a3f0090fd815f0a7fe490 Mon Sep 17 00:00:00 2001
From: nmemmert <67961292+nmemmert@users.noreply.github.com>
Date: Mon, 19 Oct 2026 13:39:09 -0400
Subject: [PATCH] Fix Settings nav and scope admin sidebar active-class toggle

- Include .admin-sidebar-footer .nav-pill in click listener so Settings is reachable
- Update showPanel to toggle active class on both .admin-nav and footer pills

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 public/admin.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/public/admin.js b/public/admin.js
index 9bdc1de..b0e26c9 100644
--- a/public/admin.js
+++ b/public/admin.js
@@ -110,7 +110,7 @@ if (parentPin) unlock();
 
 // ---------- navigation ----------
 
-document.querySelectorAll('.admin-nav .nav-pill').forEach((pill) =>
+document.querySelectorAll('.admin-nav .nav-pill, .admin-sidebar-footer .nav-pill').forEach((pill) =>
   pill.addEventListener('click', () => showPanel(pill.dataset.panel))
 );
 document.querySelectorAll('[data-nav]').forEach((btn) =>
@@ -126,7 +126,7 @@ const PANEL_TITLES = {
 
 function showPanel(name) {
   document.querySelectorAll('.panel').forEach((p) => (p.hidden = p.id !== `panel-${name}`));
-  document.querySelectorAll('.admin-nav .nav-pill').forEach((p) => p.classList.toggle('active', p.dataset.panel === name));
+  document.querySelectorAll('.admin-nav .nav-pill, .admin-sidebar-footer .nav-pill').forEach((p) => p.classList.toggle('active', p.dataset.panel === name));
   const titleEl = document.getElementById('admin-page-title');
   if (titleEl) titleEl.textContent = PANEL_TITLES[name] || name;
   const loaders = {