marvin/site-gateway
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix Create button drifting out of sync when switching admin tabs (v0.16.14)

Browse Source
This commit is contained in:
marvin
2026-09-18 19:25:28 -04:00
parent 9aa111b7fe
commit e35cbfa2ab
5 changed files with 10 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/public/features.js
+4 -2
View File
@@ -211,8 +211,10 @@ document.querySelector("#access-list").addEventListener("click", event => { if (
// --- Administration: tab switching between admin panel sections ----------------
document.querySelector(".admin-tabs").addEventListener("click", event => { const button = event.target.closest("[data-admin-tab]"); if (!button) return; document.querySelectorAll("[data-admin-tab]").forEach(item => item.classList.toggle("tab-active", item === button)); document.querySelectorAll("[data-admin-panel]").forEach(panel => panel.classList.toggle("hidden", panel.dataset.adminPanel !== button.dataset.adminTab)); document.querySelector("#open-create").classList.toggle("hidden", button.dataset.adminTab !== "users"); });
document.querySelector(".admin-tabs").addEventListener("click", event => { const button = event.target.closest("[data-admin-tab]"); if (!button) return; state.adminTab = button.dataset.adminTab; history.replaceState(null, "", `${location.pathname}${location.search}#administration/${state.adminTab}`); });
// A single handler updates state and defers to the real render() for tab-active classes, panel
// visibility, and the shared Create button's visibility/text -- render() is the one place that
// logic lives, so this never drifts out of sync with it the way two separate DOM-poking handlers did.
document.querySelector(".admin-tabs").addEventListener("click", event => { const button = event.target.closest("[data-admin-tab]"); if (!button) return; state.adminTab = button.dataset.adminTab; render(); });
if (state.adminTab && state.view === "administration") document.querySelector(`[data-admin-tab="${state.adminTab}"]`)?.click();