From d5e822ec7c83027f94b4815232bf768e5a423f03 Mon Sep 17 00:00:00 2001 From: Jeff Emmett Date: Sat, 21 Mar 2026 20:39:32 -0700 Subject: [PATCH] fix(canvas): remove duplicate settings/history button handlers Canvas.html had its own addEventListener calls for settings-btn and history-btn, but the shell script already wires these. Both handlers fired on click, causing double-toggle (open then immediately close). Co-Authored-By: Claude Opus 4.6 --- website/canvas.html | 11 ++--------- 1 file changed, 2 insertions(+), 9 deletions(-) diff --git a/website/canvas.html b/website/canvas.html index b9b7135..c579acc 100644 --- a/website/canvas.html +++ b/website/canvas.html @@ -5639,15 +5639,8 @@ } }); - // ── Settings + History panel toggles ── - document.getElementById('settings-btn')?.addEventListener('click', () => { - const panel = document.querySelector('rstack-space-settings'); - if (panel) panel.toggle(); - }); - document.getElementById('history-btn')?.addEventListener('click', () => { - const panel = document.querySelector('rstack-history-panel'); - if (panel) panel.toggle(); - }); + // Settings + History panel toggles are wired by the shell script — + // do NOT duplicate here or double-toggle cancels the action. // Memory panel — browse and remember forgotten shapes const memoryPanel = document.getElementById("memory-panel");