jeffemmett
/
rspace-online
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

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 <noreply@anthropic.com>
This commit is contained in:
Jeff Emmett 2026-03-21 20:39:32 -07:00
parent 355b53ee17
commit d5e822ec7c
1 changed files with 2 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
website/canvas.html
View File

@ -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");