cmd/ctrl + scroll now pans the canvas up/down instead of zooming at the
cursor. The catch: a trackpad pinch reports as a wheel with ctrlKey set,
indistinguishable at the event level from a real Ctrl+wheel. Gate on the
physically-held key (cmdRef, tracked via keydown): a real modifier down
pans; a pinch has ctrlKey without any keydown, so it still zooms at the
cursor. Plain wheel continues to zoom at the viewport center.
The webview-forwarded cmd/ctrl+wheel path (canvas-wheel-zoom) stays
cursor-zoom — that channel can't tell a pinch from a held key, and
keeping it zoom preserves pinch-to-zoom while hovering a browser card.
A mouse-wheel notch arrives as deltaY 100 where a trackpad sends ~1-10;
piped through the zoom curve at default sensitivity that's a ~24% jump
per notch, and macOS wheel acceleration stacks them. Clamp the per-event
zoom delta to +/-24 so each notch is a small predictable step. No-op for
trackpads (their deltas are already under the cap), so the continuous
pinch/scroll curve is unchanged.
Plain vertical wheel/two-finger scroll over the canvas now zooms
(anchored at the viewport center, reusing the zoomIn/zoomOut anchor)
instead of panning the y-axis. Horizontal-dominant scroll still pans X,
gated on the dominant axis so a sideways swipe's vertical jitter can't
also zoom. cmd/ctrl+wheel and trackpad pinch still zoom at the cursor.
flushWheel now folds a same-frame pan into the zoom branch, since a
vertical zoom and a horizontal pan can now be accumulated in one RAF
tick; dropping the pan would swallow the gesture.
Import already handled workflow bundles, but nothing told the UI. Unlike a
dashboard there is no route to navigate to, so the hub -- which only fetches on
mount -- kept showing a stale list and the imported workflow looked like it had
vanished until a remount.
Refetch on a workflow-root import. Import drops dashboard_id and /list keeps
unassigned workflows for every dashboard, so it shows up wherever the user is.
Puts Share beside the existing trash icon on each rail row, so a workflow can be
shared without opening it first. Trash keeps its current always-visible styling.
The button is faded out rather than unmounted when the row loses hover:
ShareButton owns the modal's open state, so unmounting on hover-out would slam
the dialog shut the instant the pointer left the row to reach it. Keeping it
mounted also stops the row reflowing as the icon appears.
Adds the Share affordance to the Workflows hub, left of the close button. It
only appears in detail view, so it always has an unambiguous target; goHome
leaves selectedId set, hence the explicit mode check.
Everything under it already existed: WorkflowExportable is registered and
ShareKind already had 'workflow'. The hub simply never rendered a ShareButton.
The title bar moves from the card into the content. The card is at 281 of the
linter's 300-line cap, so lifting nav state up into it would have breached the
cap; pushing the bar down to the state instead keeps both files well under, and
splits on a better seam anyway (card = drag/resize geometry, content = the app
and its chrome). The card hands its drag handlers down via CardHeader.
The share dialog portals to the body but its events still bubble the React tree,
so the wrapper stops click/pointerdown; without it, dragging the card follows a
click inside the modal.
Verified in the running app: share icon absent on Home, present in detail and
left of the X, and the modal builds a real workflow bundle.
The module was written when the workflow store lived only on eric/workflow, so
every store touch went through p_store()/p_model() try/except-returning-None.
Both resolve on this branch (the round-trip test already asserts as much), so
those guards had become dead code that would swallow a genuine ImportError in
storage.py and surface it to the user as "this build doesn't support workflows
yet; please update OpenSwarm".
Imports storage/Workflow at module level, drops the None branches, and rewrites
the stale docstring. The safety contract is unchanged and still covered by
test_workflow_sanitize_disables_schedule_and_strips_pii: an imported workflow
never auto-runs, and the sharer's phone number never rides along.
ciregenz