peter/infix
1
0
Fork 0
mirror of https://github.com/kernelkit/infix.git synced 2026-07-22 01:13:00 +02:00
Code Issues Packages Projects Releases Wiki Activity

webui: sidebar, don't auto-navigate when expanding a section

Browse Source 
Clicking a section header (Status / Configure / Maintenance) used to
fold the previously-open section AND fire a click on the first page
link inside the just-opened section. That meant going from
"Status > Overview" to "Configure > Hardware" was three clicks worth
of page loads: Configure header expands and lands you on Configure >
Interfaces, then you click Hardware and wait again.

Toggle now just expands or collapses its own section. Sibling
sections stay open until a real page-link click, at which point
updateActiveNav() already closes the others to leave only the
section containing the active page open.

Configure's enter-POST (running → candidate snapshot) still fires
the first time the Configure accordion opens per page lifecycle, so
the candidate datastore is ready by the time the user clicks a
Configure page.

Signed-off-by: Joachim Wiberg <troglobit@gmail.com>
This commit is contained in:
Joachim Wiberg
2026-06-15 19:45:18 +02:00
parent 2c80c65962
commit 5570b13791
1 changed files with 10 additions and 21 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/webui/static/js/app.js
+10 -21
View File
@@ -996,7 +996,14 @@
}
});
// Mutual exclusion, persistence, and auto-navigation to first page link
// Persistence + Configure enter-hook.
//
// Toggling a section header just expands or collapses that section —
// it does NOT close sibling sections, and it does NOT auto-navigate
// to the first page in the section. This lets the user browse the
// available pages under a header without paying for a navigation
// they didn't ask for. The actual mutual-exclusion (other sections
// collapse) happens in updateActiveNav after a real page-link click.
groups.forEach(function(d) {
var label = d.querySelector(':scope > summary');
if (!label) return;
@@ -1010,26 +1017,8 @@
if (d.id !== 'nav-configure') {
localStorage.setItem(key, d.open ? 'open' : 'closed');
}
if (d.open) {
if (d.id === 'nav-configure') {
enterConfigure();
}
groups.forEach(function(other) {
if (other !== d && other.open) {
other.removeAttribute('open');
if (other.id !== 'nav-configure') {
var otherLabel = other.querySelector(':scope > summary');
if (otherLabel) {
localStorage.setItem('nav-top:' + otherLabel.textContent.trim(), 'closed');
}
}
}
});
// Navigate to the first page link if none in this section is active
if (!d.querySelector('.nav-link.active')) {
var first = d.querySelector('a.nav-link[hx-get]');
if (first) first.click();
}
if (d.open && d.id === 'nav-configure') {
enterConfigure();
}
});
});