From 5f63e7d18e156d22e0828e0461d2b8cdb2d7cd4f Mon Sep 17 00:00:00 2001 From: Kenneth Reitz Date: Sat, 29 Nov 2025 19:00:09 -0500 Subject: [PATCH] Fix sidebar collapse state persistence across page views MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The sidebar now correctly remembers its collapsed/expanded state. Previously, the logic only set checked=true for 'true' but didn't handle the 'false' case to collapse the sidebar. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude --- kjvstudy_org/templates/base.html | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/kjvstudy_org/templates/base.html b/kjvstudy_org/templates/base.html index 80407ca..5a8837b 100644 --- a/kjvstudy_org/templates/base.html +++ b/kjvstudy_org/templates/base.html @@ -1353,12 +1353,16 @@ // Sidebar collapse state persistence (function() { var toggle = document.getElementById('sidebar-toggle'); - var expanded = localStorage.getItem('sidebarExpanded') === 'true'; - if (expanded) { + var savedState = localStorage.getItem('sidebarExpanded'); + // If user has explicitly collapsed the sidebar, respect that + if (savedState === 'false') { + toggle.checked = false; + } else if (savedState === 'true') { toggle.checked = true; } + // Default is checked (expanded) if no saved state toggle.addEventListener('change', function() { - localStorage.setItem('sidebarExpanded', toggle.checked); + localStorage.setItem('sidebarExpanded', toggle.checked ? 'true' : 'false'); }); })();