Improve Brush Graph Test Canvas#86
Conversation
There was a problem hiding this comment.
Code Review
This pull request introduces a resizable and draggable preview pane in the Brush Graph screen, allowing users to dynamically adjust its height. It also implements masking logic to exclude inspector and notification pane bounds from the drawing surface, updates the color picker to a dropdown menu, and cleans up unused parameters and imports. The review feedback highlights several optimization opportunities, including avoiding coroutine churn during drag gestures, preventing static list re-allocations inside the dropdown menu, and bypassing empty mask path creation. Additionally, minor maintainability improvements are suggested, such as removing an unused local variable and simplifying an unused callback.
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.
Add a few quality of life fixes to the test canvas in brush graph that have been on my plate for awhile.
maskPathparameter onInProgressStrokescomposable viaDrawingSurfaceso that we can hide strokes drawn on the non-expanded parts of test canvas. While we're at it, fix a long standing bug that allowed wet/in progress strokes to be visible atop the inspector/notification pane if it was expanded over the test canvas.