Seating: a cancelled drag no longer strands a half-drawn zone or calibration line

[JFrusher]

Closes #55

What a user would notice

When a zone or calibration drag was interrupted (a system gesture, an interrupted touch, a release over browser UI), the dashed draft stayed on the canvas, still following the pointer. Now it disappears and nothing is added.

Cause, with evidence

startZoneDraw and startCalibrate in RoomCanvas.tsx listened for pointermove and pointerup only. The new test reproduced both halves of the problem on main:

• after pointercancel the .draftZone element was still rendered;
• the listeners stayed on window, so the next release anywhere committed a zone nobody drew. That leak even made the third test count two zones on main.

Fix

A shared stop() removes all three listeners and clears the draft. pointerup commits and then stops, as before. pointercancel only stops.

The other fixed handlers (TableHandles, TableNode, pan) treat cancel like release. That's right for them, because they apply changes live. Here a release commits (adds the zone, opens the calibrate dialog), so a cancel must not.

Also deletes the stale TODO in useCanvasPanZoom.ts, whose fix was already sitting beneath it.

Checks

• New RoomCanvas.test.tsx: cancel drops the zone, cancel drops the line and opens no dialog, and an ordinary release still adds a zone. All 3 failed on main and pass now.
• Seating unit tests, typecheck: green.
• e2e (seating, guests, header, tour, persistence, axe on every page) on a build combining this batch: 61/61, including "a zone is drawn on the floor, then moved".

🤖 Generated with Claude Code

https://claude.ai/code/session_01VhFp1PtVBwgmxST9ZrRTEs

---

Generated by Claude Code

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated
knotwork-suite	Ready	Preview	Oct 4, 2026 7:47pm UTC