Feat(canvas): Add temporary bbox move with C key hold#9104
Conversation
Good catch. I was only clearing the temporary bbox hotkey state on I fixed this by explicitly stopping any active bbox interaction before clearing temporary tool overrides. That should prevent the bbox from getting stuck half-active after alt-tab. |
# Conflicts: # invokeai/frontend/web/src/features/controlLayers/konva/CanvasTool/CanvasToolModule.ts # invokeai/frontend/web/src/features/ui/layouts/DockviewCanvasHeaderActions.tsx
Summary
This PR adds bbox hotkey behavior for canvas. A short tap on
Cselects the bbox tool, and holdingCtemporarily activates bbox until the key is released.This PR also moves temporary bbox hotkey handling into the shared canvas tool state so it works correctly with the existing temporary Space and Alt tool overrides. Persistent tool changes now update the base tool consistently, and focused tests were added for tap, hold, and nested temporary hotkey flows.
As a result it touches more files than it was originally expected.
Related Issues / Discussions
N/A
QA Instructions
Cquickly. Verify that bbox becomes the selected tool.C. Verify that bbox is active only whileCis held and that releasingCreturns to the previous tool.C, then hold Space, releaseC, then release Space. Verify that the active tool returns to the original base tool and does not stay on bbox.Merge Plan
No special merge plan.
Checklist
What's Newcopy (if doing a release after this PR)