fix(canvas): let the spacebar type a space during inline editing

Pressing space mid-word did nothing. The edited element lives in a breakpoint iframe, which forwards its keystrokes to the parent document as a clone dispatched on `document` (so parent shortcuts like Cmd+K/Cmd+S work). The space-to-pan handler (useCanvas) listens on that document and guards only against `target.isContentEditable` — but the forwarded event's target IS the document, not the iframe's editing element, so the guard missed it, the handler called preventDefault(), and forwardKeyboard propagated that back to suppress the original spacebar. Net effect: every space was eaten. Guard the space-pan handler with the same activeInlineEdit check the other canvas shortcut handler uses: while a session is active the spacebar (and every non-Escape/Enter key) must reach the editing element untouched. Verified in a real browser: a Space keydown on the editing element went from defaultPrevented:true to defaultPrevented:false after the fix; contentEditable inserts the space normally (execCommand path already confirmed spaces work). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
CoreBunch · DavidBabinec · Jun 11, 2026 · Jun 10, 2026 · Jun 10, 2026 · Jun 10, 2026
commit 4fe6f20a2f4db34e74a831be600eaa2fb5d31ff9
diff --git a/src/__tests__/canvas/inlineTextEditingWiring.test.ts b/src/__tests__/canvas/inlineTextEditingWiring.test.ts
@@ -20,6 +20,7 @@ import { readFileSync } from 'fs'
 const CANVAS_ROOT = new URL('../../admin/pages/site/canvas/CanvasRoot.tsx', import.meta.url)
 const NODE_RENDERER = new URL('../../admin/pages/site/canvas/NodeRenderer.tsx', import.meta.url)
 const KEYBOARD_SHORTCUTS = new URL('../../admin/pages/site/canvas/useCanvasKeyboardShortcuts.ts', import.meta.url)
+const USE_CANVAS = new URL('../../admin/pages/site/hooks/useCanvas.ts', import.meta.url)
 const BREAKPOINT_FRAME = new URL('../../admin/pages/site/canvas/BreakpointFrame.tsx', import.meta.url)
 const CONTEXTS = new URL('../../admin/pages/site/canvas/CanvasContexts.ts', import.meta.url)
 
@@ -63,6 +64,14 @@ describe('inline text editing wiring (in-place contentEditable)', () => {
     expect(src).toContain('if (useEditorStore.getState().activeInlineEdit) return')
   })
 
+  it('space-to-pan bails while an inline edit is active so the spacebar types a space', () => {
+    // The edited element forwards its keystrokes to the parent document, where
+    // the space-pan handler would otherwise preventDefault the spacebar and the
+    // author could never type a space mid-word.
+    const src = readFileSync(USE_CANVAS, 'utf-8')
+    expect(src).toContain('if (useEditorStore.getState().activeInlineEdit) return')
+  })
+
   it('BreakpointFrame no longer mounts an inline-edit overlay', () => {
     const src = readFileSync(BREAKPOINT_FRAME, 'utf-8')
     expect(src).not.toContain('InlineTextEditOverlay')

diff --git a/src/admin/pages/site/hooks/useCanvas.ts b/src/admin/pages/site/hooks/useCanvas.ts
@@ -251,6 +251,14 @@ export function useCanvas({ canvasRootRef, transformLayerRef, enabled }: UseCanv
   useEffect(() => {
     function onKeyDown(e: KeyboardEvent) {
       if (e.code === 'Space' && !e.repeat) {
+        // Inline text editing owns the keyboard. The edited element lives in a
+        // breakpoint iframe and forwards its keystrokes here as a clone
+        // dispatched on `document`, so `e.target` is the document — the
+        // contentEditable check below can't see it. Without this guard the
+        // space-pan would preventDefault every spacebar press, so the author
+        // could never type a space. The editing element handles Escape/Enter
+        // itself; every other key (space included) must reach it untouched.
+        if (useEditorStore.getState().activeInlineEdit) return
         const target = e.target as HTMLElement
         // Don't intercept space in inputs/textareas
         if (