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

fix: make pencil tool temporary, return to selector on ESC 

Drawing tools (pencil, rect, circle, line) now auto-clear after one
stroke, returning to the default selector tool. ESC clears any active
whiteboard tool. Switching to another toolbar tool also clears the
active whiteboard tool.

This fixes the resize issue — when a whiteboard tool was stuck active,
canvasContent had pointer-events:none which blocked all shape
interactions including resize handles.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
Jeff Emmett 2026-03-02 20:24:09 -08:00
parent cb828fc605
commit 7d8f64d17c
1 changed files with 11 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
website/canvas.html
View File

@ -3215,6 +3215,7 @@
let ghostEl = null;
function setPendingTool(tagName, props = {}) {
if (wbTool) setWbTool(null); // clear whiteboard tool when switching
pendingTool = { tagName, props };
canvas.style.cursor = "crosshair";
@ -3270,10 +3271,11 @@
}
});
// ESC clears pending tool
// ESC returns to default selector tool
document.addEventListener("keydown", (e) => {
if (e.key === "Escape" && pendingTool) {
clearPendingTool();
if (e.key === "Escape") {
if (wbTool) setWbTool(null);
if (pendingTool) clearPendingTool();
}
});
@ -3561,6 +3563,7 @@
const newArrowBtn = document.getElementById("new-arrow");
newArrowBtn.addEventListener("click", () => {
if (wbTool) setWbTool(null); // clear whiteboard tool
connectMode = !connectMode;
newArrowBtn.classList.toggle("active", connectMode);
canvas.classList.toggle("connect-mode", connectMode);
@ -3770,6 +3773,11 @@
wbPreviewEl = null;
wbCurrentPath = [];
// Drawing tools are temporary — return to selector after one stroke
if (wbTool && wbTool !== "eraser") {
setWbTool(null);
}
});
// Eraser: click on existing SVG strokes to delete them + remove from Automerge