@dfosco/storyboard-react 3.11.0-beta.6 → 3.11.0-beta.8

package/package.json

@@ -1,10 +1,10 @@
 {
   "name": "@dfosco/storyboard-react",
-  "version": "3.11.0-beta.6",
+  "version": "3.11.0-beta.8",
   "type": "module",
   "dependencies": {
-    "@dfosco/storyboard-core": "3.11.0-beta.6",
-    "@dfosco/tiny-canvas": "3.11.0-beta.6",
+    "@dfosco/storyboard-core": "3.11.0-beta.8",
+    "@dfosco/tiny-canvas": "3.11.0-beta.8",
     "@neodrag/react": "^2.3.1",
     "glob": "^11.0.0",
     "jsonc-parser": "^3.3.1"

package/src/Viewfinder.jsx

@@ -45,9 +45,11 @@ export default function Viewfinder({ pageModules = {}, basePath, title = 'Storyb
         showThumbnails,
         hideDefaultFlow: shouldHideDefault,
       })
-      // Reveal after CSS has been processed to prevent FOUC
-      requestAnimationFrame(() => {
-        if (containerRef.current) containerRef.current.style.opacity = '1'
+      // Wait for styles to be fully loaded before revealing
+      handleRef.current.ready.then(() => {
+        requestAnimationFrame(() => {
+          if (containerRef.current) containerRef.current.style.opacity = '1'
+        })
       })
     })
 

package/src/canvas/CanvasPage.bridge.test.jsx

@@ -65,6 +65,7 @@ vi.mock('./widgets/widgetProps.js', () => ({
 
 vi.mock('./widgets/widgetConfig.js', () => ({
   getFeatures: () => [],
+  isResizable: () => false,
   schemas: {},
   getMenuWidgetTypes: () => [],
 }))

package/src/canvas/CanvasPage.jsx

@@ -7,7 +7,7 @@ import { shouldPreventCanvasTextSelection } from './textSelection.js'
 import { getCanvasThemeVars, getCanvasPrimerAttrs } from './canvasTheme.js'
 import { getWidgetComponent } from './widgets/index.js'
 import { schemas, getDefaults } from './widgets/widgetProps.js'
-import { getFeatures } from './widgets/widgetConfig.js'
+import { getFeatures, isResizable } from './widgets/widgetConfig.js'
 import { isFigmaUrl, sanitizeFigmaUrl } from './widgets/figmaUrl.js'
 import WidgetChrome from './widgets/WidgetChrome.jsx'
 import ComponentWidget from './widgets/ComponentWidget.jsx'
@@ -209,8 +209,9 @@ function WidgetRenderer({ widget, onUpdate, widgetRef }) {
     console.warn(`[canvas] Unknown widget type: ${widget.type}`)
     return null
   }
+  const resizable = isResizable(widget.type) && !!onUpdate
   // Only pass ref to forwardRef-wrapped components (e.g. PrototypeEmbed)
-  const elementProps = { id: widget.id, props: widget.props, onUpdate }
+  const elementProps = { id: widget.id, props: widget.props, onUpdate, resizable }
   if (Component.$$typeof === Symbol.for('react.forward_ref')) {
     elementProps.ref = widgetRef
   }
@@ -224,6 +225,7 @@ function WidgetRenderer({ widget, onUpdate, widgetRef }) {
 function ChromeWrappedWidget({
   widget,
   selected,
+  multiSelected,
   onSelect,
   onDeselect,
   onUpdate,
@@ -248,6 +250,7 @@ function ChromeWrappedWidget({
       widgetType={widget.type}
       features={features}
       selected={selected}
+      multiSelected={multiSelected}
       widgetProps={widget.props}
       widgetRef={widgetRef}
       onSelect={onSelect}
@@ -278,7 +281,7 @@ export default function CanvasPage({ name }) {
   // Local mutable copy of widgets for instant UI updates
   const [localWidgets, setLocalWidgets] = useState(canvas?.widgets ?? null)
   const [trackedCanvas, setTrackedCanvas] = useState(canvas)
-  const [selectedWidgetId, setSelectedWidgetId] = useState(null)
+  const [selectedWidgetIds, setSelectedWidgetIds] = useState(() => new Set())
   const initialViewport = loadViewportState(name)
   const [zoom, setZoom] = useState(initialViewport?.zoom ?? 100)
   const zoomRef = useRef(initialViewport?.zoom ?? 100)
@@ -307,6 +310,92 @@ export default function CanvasPage({ name }) {
     return writeQueueRef.current
   }
 
+  // Ref for selectedWidgetIds to avoid stale closures in callbacks
+  const selectedIdsRef = useRef(selectedWidgetIds)
+  useEffect(() => {
+    selectedIdsRef.current = selectedWidgetIds
+  }, [selectedWidgetIds])
+
+  const isMultiSelected = selectedWidgetIds.size > 1
+
+  /**
+   * Selection handler — shift+click toggles in/out of multi-select set,
+   * plain click single-selects (clears others).
+   */
+  const handleWidgetSelect = useCallback((widgetId, shiftKey) => {
+    if (shiftKey) {
+      setSelectedWidgetIds(prev => {
+        const next = new Set(prev)
+        if (next.has(widgetId)) {
+          next.delete(widgetId)
+        } else {
+          next.add(widgetId)
+        }
+        return next
+      })
+    } else {
+      setSelectedWidgetIds(new Set([widgetId]))
+    }
+  }, [])
+
+  // --- Multi-select live drag preview via imperative DOM transforms ---
+  // On drag start, snapshot ALL articles' translate values (same coord space).
+  // On each tick, read dragged article's current translate, compute delta
+  // from its snapshot, apply same delta to all peers.
+  const draggedArticleRef = useRef(null)
+  const draggedStartTranslate = useRef({ x: 0, y: 0 })
+  const peerSnapshots = useRef(new Map())
+
+  function parseTranslate(article) {
+    const raw = article?.style.translate || '0px 0px'
+    const parts = raw.match(/-?[\d.]+/g) || [0, 0]
+    return { x: parseFloat(parts[0]) || 0, y: parseFloat(parts[1]) || 0 }
+  }
+
+  const handleItemDragStart = useCallback((dragId) => {
+    const ids = selectedIdsRef.current
+    peerSnapshots.current.clear()
+    draggedArticleRef.current = null
+    if (ids.size <= 1 || !ids.has(dragId)) return
+
+    // Snapshot dragged widget's article translate
+    const draggedEl = document.getElementById(dragId)
+    const draggedArticle = draggedEl?.closest('article')
+    if (!draggedArticle) return
+    draggedArticleRef.current = draggedArticle
+    draggedStartTranslate.current = parseTranslate(draggedArticle)
+
+    // Snapshot each peer's article translate
+    for (const id of ids) {
+      if (id === dragId) continue
+      const widgetEl = document.getElementById(id)
+      const article = widgetEl?.closest('article')
+      if (!article) continue
+      peerSnapshots.current.set(id, {
+        article,
+        ...parseTranslate(article),
+      })
+    }
+  }, [])
+
+  const handleItemDrag = useCallback(() => {
+    if (!draggedArticleRef.current) return
+
+    // Read dragged article's CURRENT translate (set by neodrag)
+    const current = parseTranslate(draggedArticleRef.current)
+    const dx = current.x - draggedStartTranslate.current.x
+    const dy = current.y - draggedStartTranslate.current.y
+
+    for (const [, peer] of peerSnapshots.current) {
+      peer.article.style.translate = `${peer.x + dx}px ${peer.y + dy}px`
+    }
+  }, [])
+
+  const clearDragPreview = useCallback(() => {
+    peerSnapshots.current.clear()
+    draggedArticleRef.current = null
+  }, [])
+
   if (canvas !== trackedCanvas) {
     setTrackedCanvas(canvas)
     setLocalWidgets(canvas?.widgets ?? null)
@@ -447,6 +536,44 @@ export default function CanvasPage({ name }) {
       return
     }
 
+    const ids = selectedIdsRef.current
+    // Multi-select move: apply same delta to all selected widgets
+    if (ids.size > 1 && ids.has(dragId)) {
+      clearDragPreview()
+      undoRedo.snapshot(stateRef.current, 'multi-move')
+      const currentWidgets = stateRef.current.widgets ?? []
+      const draggedWidget = currentWidgets.find(w => w.id === dragId)
+      if (!draggedWidget) return
+      const oldPos = draggedWidget.position || { x: 0, y: 0 }
+      const dx = rounded.x - oldPos.x
+      const dy = rounded.y - oldPos.y
+
+      debouncedSave.cancel()
+      setLocalWidgets((prev) => {
+        if (!prev) return prev
+        const next = prev.map((w) => {
+          if (w.id === dragId) return { ...w, position: rounded }
+          if (ids.has(w.id)) {
+            return {
+              ...w,
+              position: {
+                x: Math.max(0, roundPosition((w.position?.x ?? 0) + dx)),
+                y: Math.max(0, roundPosition((w.position?.y ?? 0) + dy)),
+              },
+            }
+          }
+          return w
+        })
+        queueWrite(() =>
+          updateCanvas(name, { widgets: next }).catch((err) =>
+            console.error('[canvas] Failed to save multi-move:', err)
+          )
+        )
+        return next
+      })
+      return
+    }
+
     undoRedo.snapshot(stateRef.current, 'move', dragId)
     setLocalWidgets((prev) => {
       if (!prev) return prev
@@ -460,7 +587,7 @@ export default function CanvasPage({ name }) {
       )
       return next
     })
-  }, [name, undoRedo])
+  }, [name, undoRedo, debouncedSave, clearDragPreview])
 
   useEffect(() => {
     zoomRef.current = zoom
@@ -780,22 +907,39 @@ export default function CanvasPage({ name }) {
 
   useEffect(() => {
     function handleKeyDown(e) {
-      if (!selectedWidgetId) return
+      if (selectedWidgetIds.size === 0) return
       const tag = e.target.tagName
       if (tag === 'INPUT' || tag === 'TEXTAREA' || e.target.isContentEditable) return
       if (e.key === 'Escape') {
         e.preventDefault()
-        setSelectedWidgetId(null)
+        setSelectedWidgetIds(new Set())
       }
       if (e.key === 'Delete' || e.key === 'Backspace') {
         e.preventDefault()
-        handleWidgetRemove(selectedWidgetId)
-        setSelectedWidgetId(null)
+        if (selectedWidgetIds.size > 1) {
+          // Multi-delete — snapshot once, remove all, persist via updateCanvas
+          undoRedo.snapshot(stateRef.current, 'multi-remove')
+          debouncedSave.cancel()
+          setLocalWidgets((prev) => {
+            if (!prev) return prev
+            const next = prev.filter(w => !selectedWidgetIds.has(w.id))
+            queueWrite(() =>
+              updateCanvas(name, { widgets: next }).catch(err =>
+                console.error('[canvas] Failed to save multi-delete:', err)
+              )
+            )
+            return next
+          })
+        } else {
+          const widgetId = [...selectedWidgetIds][0]
+          if (widgetId) handleWidgetRemove(widgetId)
+        }
+        setSelectedWidgetIds(new Set())
       }
     }
     document.addEventListener('keydown', handleKeyDown)
     return () => document.removeEventListener('keydown', handleKeyDown)
-  }, [selectedWidgetId, handleWidgetRemove])
+  }, [selectedWidgetIds, handleWidgetRemove, undoRedo, name, debouncedSave])
 
   // Paste handler — images become image widgets, same-origin URLs become prototypes,
   // other URLs become link previews, text becomes markdown
@@ -1120,6 +1264,7 @@ export default function CanvasPage({ name }) {
     colorMode: canvas.colorMode === 'auto'
       ? getToolbarColorMode(canvasTheme)
       : (canvas.colorMode ?? 'auto'),
+    locked: !isLocalDev,
   }
 
   const canvasThemeVars = getCanvasThemeVars(canvasTheme)
@@ -1146,20 +1291,22 @@ export default function CanvasPage({ name }) {
           id={`jsx-${exportName}`}
           data-tc-x={sourcePosition.x}
           data-tc-y={sourcePosition.y}
-          {...(isLocalDev ? { 'data-tc-handle': '.tc-drag-handle' } : {})}
+          {...(isLocalDev ? { 'data-tc-handle': '.tc-drag-handle, .tc-drag-surface' } : {})}
           {...canvasPrimerAttrs}
           style={canvasThemeVars}
           onClick={isLocalDev ? (e) => {
             e.stopPropagation()
-            setSelectedWidgetId(`jsx-${exportName}`)
+            if (!e.target.closest('.tc-drag-handle')) {
+              handleWidgetSelect(`jsx-${exportName}`, e.shiftKey)
+            }
           } : undefined}
         >
           <WidgetChrome
             widgetId={`jsx-${exportName}`}
             features={componentFeatures}
-            selected={selectedWidgetId === `jsx-${exportName}`}
-            onSelect={() => setSelectedWidgetId(`jsx-${exportName}`)}
-            onDeselect={() => setSelectedWidgetId(null)}
+            selected={selectedWidgetIds.has(`jsx-${exportName}`)}
+            onSelect={(shiftKey) => handleWidgetSelect(`jsx-${exportName}`, shiftKey)}
+            onDeselect={() => setSelectedWidgetIds(new Set())}
             readOnly={!isLocalDev}
           >
             <ComponentWidget
@@ -1167,6 +1314,7 @@ export default function CanvasPage({ name }) {
               width={sourceData.width}
               height={sourceData.height}
               onUpdate={isLocalDev ? (updates) => handleSourceUpdate(exportName, updates) : undefined}
+              resizable={isResizable('component') && isLocalDev}
             />
           </WidgetChrome>
         </div>
@@ -1182,24 +1330,27 @@ export default function CanvasPage({ name }) {
         id={widget.id}
         data-tc-x={widget?.position?.x ?? 0}
         data-tc-y={widget?.position?.y ?? 0}
-        {...(isLocalDev ? { 'data-tc-handle': '.tc-drag-handle' } : {})}
+        {...(isLocalDev ? { 'data-tc-handle': '.tc-drag-handle, .tc-drag-surface' } : {})}
         {...canvasPrimerAttrs}
         style={canvasThemeVars}
         onClick={isLocalDev ? (e) => {
           e.stopPropagation()
-          setSelectedWidgetId(widget.id)
+          if (!e.target.closest('.tc-drag-handle')) {
+            handleWidgetSelect(widget.id, e.shiftKey)
+          }
         } : undefined}
       >
         <ChromeWrappedWidget
           widget={widget}
-          selected={selectedWidgetId === widget.id}
-          onSelect={() => setSelectedWidgetId(widget.id)}
-          onDeselect={() => setSelectedWidgetId(null)}
+          selected={selectedWidgetIds.has(widget.id)}
+          multiSelected={isMultiSelected && selectedWidgetIds.has(widget.id)}
+          onSelect={(shiftKey) => handleWidgetSelect(widget.id, shiftKey)}
+          onDeselect={() => setSelectedWidgetIds(new Set())}
           onUpdate={isLocalDev ? handleWidgetUpdate : undefined}
           onCopy={isLocalDev ? handleWidgetCopy : undefined}
           onRemove={isLocalDev ? (id) => {
             handleWidgetRemove(id)
-            setSelectedWidgetId(null)
+            setSelectedWidgetIds(new Set())
           } : undefined}
           readOnly={!isLocalDev}
         />
@@ -1240,7 +1391,7 @@ export default function CanvasPage({ name }) {
           ...canvasThemeVars,
           ...(spaceHeld ? { cursor: panningActive ? 'grabbing' : 'grab' } : {}),
         }}
-        onClick={() => setSelectedWidgetId(null)}
+        onClick={() => setSelectedWidgetIds(new Set())}
         onMouseDown={handlePanStart}
       >
         <div
@@ -1255,7 +1406,7 @@ export default function CanvasPage({ name }) {
             ...(spaceHeld ? { pointerEvents: 'none' } : {}),
           }}
         >
-          <Canvas {...canvasProps} onDragEnd={isLocalDev ? handleItemDragEnd : undefined}>
+          <Canvas {...canvasProps} onDragStart={isLocalDev ? handleItemDragStart : undefined} onDrag={isLocalDev ? handleItemDrag : undefined} onDragEnd={isLocalDev ? handleItemDragEnd : undefined}>
             {allChildren}
           </Canvas>
         </div>

package/src/canvas/CanvasPage.multiselect.test.jsx

@@ -0,0 +1,260 @@
+import { fireEvent, render, screen, act } from '@testing-library/react'
+import CanvasPage from './CanvasPage.jsx'
+import { updateCanvas, removeWidget } from './canvasApi.js'
+
+const MOCK_UNDO_REDO = {
+  snapshot: vi.fn(),
+  undo: vi.fn(),
+  redo: vi.fn(),
+  reset: vi.fn(),
+  canUndo: false,
+  canRedo: false,
+}
+
+vi.mock('./useUndoRedo.js', () => ({
+  default: () => MOCK_UNDO_REDO,
+}))
+
+// Expose onDragEnd so tests can trigger drags with specific IDs
+let capturedOnDragEnd = null
+vi.mock('@dfosco/tiny-canvas', () => ({
+  Canvas: ({ children, onDragEnd }) => {
+    capturedOnDragEnd = onDragEnd
+    return <div data-testid="tiny-canvas">{children}</div>
+  },
+}))
+
+const mockCanvas = {
+  title: 'Multi-Select Test',
+  widgets: [
+    { id: 'w1', type: 'sticky-note', position: { x: 100, y: 100 }, props: {} },
+    { id: 'w2', type: 'sticky-note', position: { x: 300, y: 100 }, props: {} },
+    { id: 'w3', type: 'markdown', position: { x: 500, y: 200 }, props: {} },
+  ],
+  sources: [],
+  centered: false,
+  dotted: false,
+  grid: false,
+  gridSize: 18,
+  colorMode: 'auto',
+}
+
+vi.mock('./useCanvas.js', () => ({
+  useCanvas: () => ({
+    canvas: mockCanvas,
+    jsxExports: null,
+    loading: false,
+  }),
+}))
+
+vi.mock('./widgets/index.js', () => ({
+  getWidgetComponent: () => function MockWidget({ id }) {
+    return <div data-testid={`widget-content-${id}`}>widget</div>
+  },
+}))
+
+// WidgetChrome mock that exposes onSelect with shift parameter
+vi.mock('./widgets/WidgetChrome.jsx', () => ({
+  default: ({ children, onSelect, selected, multiSelected, widgetId }) => (
+    <div
+      data-testid={`chrome-${widgetId}`}
+      data-selected={selected || undefined}
+      data-multi-selected={multiSelected || undefined}
+    >
+      {children}
+      <button
+        className="tc-drag-handle"
+        data-testid={`select-${widgetId}`}
+        onClick={(e) => { e.stopPropagation(); onSelect?.(false) }}
+      >
+        select
+      </button>
+      <button
+        className="tc-drag-handle"
+        data-testid={`shift-select-${widgetId}`}
+        onClick={(e) => { e.stopPropagation(); onSelect?.(true) }}
+      >
+        shift-select
+      </button>
+    </div>
+  ),
+}))
+
+vi.mock('./widgets/widgetProps.js', () => ({
+  schemas: {},
+  getDefaults: () => ({}),
+}))
+
+vi.mock('./widgets/widgetConfig.js', () => ({
+  getFeatures: () => [],
+  isResizable: () => false,
+  schemas: {},
+  getMenuWidgetTypes: () => [],
+}))
+
+vi.mock('./widgets/figmaUrl.js', () => ({
+  isFigmaUrl: () => false,
+  sanitizeFigmaUrl: (url) => url,
+}))
+
+vi.mock('./canvasApi.js', () => ({
+  addWidget: vi.fn(),
+  updateCanvas: vi.fn(() => Promise.resolve({ success: true })),
+  removeWidget: vi.fn(() => Promise.resolve({ success: true })),
+  uploadImage: vi.fn(),
+}))
+
+describe('CanvasPage multi-select', () => {
+  beforeEach(() => {
+    delete window.__storyboardCanvasBridgeState
+    window.__SB_LOCAL_DEV__ = true
+    vi.clearAllMocks()
+    capturedOnDragEnd = null
+  })
+
+  afterEach(() => {
+    delete window.__SB_LOCAL_DEV__
+  })
+
+  it('shift+click on select handle adds widget to selection', async () => {
+    render(<CanvasPage name="test-canvas" />)
+
+    // Select first widget
+    fireEvent.click(screen.getByTestId('select-w1'))
+    await act(async () => { await new Promise((r) => setTimeout(r, 0)) })
+    expect(screen.getByTestId('chrome-w1').dataset.selected).toBeDefined()
+
+    // Shift+select second widget
+    fireEvent.click(screen.getByTestId('shift-select-w2'))
+    await act(async () => { await new Promise((r) => setTimeout(r, 0)) })
+    expect(screen.getByTestId('chrome-w1').dataset.selected).toBeDefined()
+    expect(screen.getByTestId('chrome-w2').dataset.selected).toBeDefined()
+  })
+
+  it('shift+click on already selected widget removes it from selection', async () => {
+    render(<CanvasPage name="test-canvas" />)
+
+    // Select both
+    fireEvent.click(screen.getByTestId('select-w1'))
+    fireEvent.click(screen.getByTestId('shift-select-w2'))
+    await act(async () => { await new Promise((r) => setTimeout(r, 0)) })
+    expect(screen.getByTestId('chrome-w1').dataset.selected).toBeDefined()
+    expect(screen.getByTestId('chrome-w2').dataset.selected).toBeDefined()
+
+    // Shift+click w1 again to remove it
+    fireEvent.click(screen.getByTestId('shift-select-w1'))
+    await act(async () => { await new Promise((r) => setTimeout(r, 0)) })
+    expect(screen.getByTestId('chrome-w1').dataset.selected).toBeUndefined()
+    expect(screen.getByTestId('chrome-w2').dataset.selected).toBeDefined()
+  })
+
+  it('normal click replaces multi-selection with single', async () => {
+    render(<CanvasPage name="test-canvas" />)
+
+    // Multi-select
+    fireEvent.click(screen.getByTestId('select-w1'))
+    fireEvent.click(screen.getByTestId('shift-select-w2'))
+    fireEvent.click(screen.getByTestId('shift-select-w3'))
+    await act(async () => { await new Promise((r) => setTimeout(r, 0)) })
+
+    // Normal click on w1 clears multi-select
+    fireEvent.click(screen.getByTestId('select-w1'))
+    await act(async () => { await new Promise((r) => setTimeout(r, 0)) })
+    expect(screen.getByTestId('chrome-w1').dataset.selected).toBeDefined()
+    expect(screen.getByTestId('chrome-w2').dataset.selected).toBeUndefined()
+    expect(screen.getByTestId('chrome-w3').dataset.selected).toBeUndefined()
+  })
+
+  it('sets multiSelected on all selected widgets when multiple are selected', async () => {
+    render(<CanvasPage name="test-canvas" />)
+
+    fireEvent.click(screen.getByTestId('select-w1'))
+    fireEvent.click(screen.getByTestId('shift-select-w2'))
+    await act(async () => { await new Promise((r) => setTimeout(r, 0)) })
+
+    expect(screen.getByTestId('chrome-w1').dataset.multiSelected).toBeDefined()
+    expect(screen.getByTestId('chrome-w2').dataset.multiSelected).toBeDefined()
+    // Unselected widget should not have multiSelected
+    expect(screen.getByTestId('chrome-w3').dataset.multiSelected).toBeUndefined()
+  })
+
+  it('Escape clears all selection', async () => {
+    render(<CanvasPage name="test-canvas" />)
+
+    fireEvent.click(screen.getByTestId('select-w1'))
+    fireEvent.click(screen.getByTestId('shift-select-w2'))
+    await act(async () => { await new Promise((r) => setTimeout(r, 0)) })
+
+    fireEvent.keyDown(document, { key: 'Escape' })
+    await act(async () => { await new Promise((r) => setTimeout(r, 0)) })
+
+    expect(screen.getByTestId('chrome-w1').dataset.selected).toBeUndefined()
+    expect(screen.getByTestId('chrome-w2').dataset.selected).toBeUndefined()
+  })
+
+  it('Delete removes all selected widgets and calls updateCanvas', async () => {
+    render(<CanvasPage name="test-canvas" />)
+
+    // Multi-select w1 and w2
+    fireEvent.click(screen.getByTestId('select-w1'))
+    fireEvent.click(screen.getByTestId('shift-select-w2'))
+    await act(async () => { await new Promise((r) => setTimeout(r, 0)) })
+
+    // Press Delete
+    fireEvent.keyDown(document, { key: 'Delete' })
+    await act(async () => { await new Promise((r) => setTimeout(r, 0)) })
+
+    // Should call updateCanvas with only w3 remaining
+    expect(updateCanvas).toHaveBeenCalledWith(
+      'test-canvas',
+      expect.objectContaining({
+        widgets: [expect.objectContaining({ id: 'w3' })],
+      })
+    )
+    // Should NOT use individual removeWidget API for multi-delete
+    expect(removeWidget).not.toHaveBeenCalled()
+    // Should snapshot for undo
+    expect(MOCK_UNDO_REDO.snapshot).toHaveBeenCalled()
+  })
+
+  it('single-select Delete uses removeWidget API', async () => {
+    render(<CanvasPage name="test-canvas" />)
+
+    fireEvent.click(screen.getByTestId('select-w1'))
+    fireEvent.keyDown(document, { key: 'Backspace' })
+    await act(async () => { await new Promise((r) => setTimeout(r, 0)) })
+
+    expect(removeWidget).toHaveBeenCalledWith('test-canvas', 'w1')
+  })
+
+  it('multi-select move applies delta to all selected widgets', async () => {
+    render(<CanvasPage name="test-canvas" />)
+
+    // Multi-select w1 (100,100) and w2 (300,100)
+    fireEvent.click(screen.getByTestId('select-w1'))
+    fireEvent.click(screen.getByTestId('shift-select-w2'))
+
+    // Wait for selectedIdsRef to sync
+    await act(async () => { await new Promise((r) => setTimeout(r, 0)) })
+
+    // Drag w1 to (150, 200) → delta is (+50, +100)
+    expect(capturedOnDragEnd).toBeTruthy()
+    act(() => {
+      capturedOnDragEnd('w1', { x: 150, y: 200 })
+    })
+    await act(async () => { await new Promise((r) => setTimeout(r, 0)) })
+
+    // w1 → (150, 200), w2 → (300+50, 100+100) = (350, 200)
+    expect(updateCanvas).toHaveBeenCalledWith(
+      'test-canvas',
+      expect.objectContaining({
+        widgets: expect.arrayContaining([
+          expect.objectContaining({ id: 'w1', position: { x: 150, y: 200 } }),
+          expect.objectContaining({ id: 'w2', position: { x: 350, y: 200 } }),
+          // w3 unchanged
+          expect.objectContaining({ id: 'w3', position: { x: 500, y: 200 } }),
+        ]),
+      })
+    )
+  })
+})

package/src/canvas/useCanvas.js

@@ -9,7 +9,8 @@ async function fetchCanvasFromServer(name) {
   try {
     const base = (import.meta.env?.BASE_URL || '/').replace(/\/$/, '')
     const res = await fetch(`${base}/_storyboard/canvas/read?name=${encodeURIComponent(name)}`)
-    if (res.ok) return res.json()
+    const contentType = res.headers.get('content-type') || ''
+    if (res.ok && contentType.includes('application/json')) return res.json()
   } catch { /* fall back to build-time data */ }
   return null
 }

package/src/canvas/widgets/ComponentWidget.jsx

@@ -11,7 +11,7 @@ import styles from './ComponentWidget.module.css'
  * Double-click the overlay to enter interactive mode (dropdowns, buttons work).
  * Click outside to exit interactive mode.
  */
-export default function ComponentWidget({ component: Component, width, height, onUpdate }) {
+export default function ComponentWidget({ component: Component, width, height, onUpdate, resizable }) {
   const containerRef = useRef(null)
   const [interactive, setInteractive] = useState(false)
 
@@ -51,12 +51,14 @@ export default function ComponentWidget({ component: Component, width, height, o
             onDoubleClick={enterInteractive}
           />
         )}
-        <ResizeHandle
-          targetRef={containerRef}
-          minWidth={100}
-          minHeight={60}
-          onResize={handleResize}
-        />
+        {resizable && (
+          <ResizeHandle
+            targetRef={containerRef}
+            minWidth={100}
+            minHeight={60}
+            onResize={handleResize}
+          />
+        )}
       </div>
     </WidgetWrapper>
   )