Add layer and artboard nudge resizing#4242
Conversation
There was a problem hiding this comment.
Code Review
This pull request introduces the ability to resize selected layers and artboards using arrow-key nudges combined with modifier keys (Alt to resize, and Control to resize from the opposite corner). It updates the input mappings, message definitions, and handlers across the artboard, shape, and select tools, and implements a new utility function nudge_resize_bounds to calculate the updated bounds and scale transformations. Since there are no review comments, I have no additional feedback to provide.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
There was a problem hiding this comment.
1 issue found and verified against the latest diff
Confidence score: 3/5
- In
editor/src/messages/portfolio/document/document_message_handler.rs, the resize path’s combined bounding box can be poisoned by a single non-finite layer bound, which can cascade into invalid scale values and broken resize behavior for users — add a finite/NaN guard (skip or sanitize non-finite bounds) before computing the combined box before merging.
Reply with feedback, questions, or to request a fix.
Re-trigger cubic
…ite bounds are considered
* Add layer and artboard nudge resizing * Improve bounding box validation for layer resizing to ensure only finite bounds are considered
No description provided.