Fix the Artboard tool resizing and nudging locked or hidden artboards

[Keavon]

No description provided.

[cubic-dev-ai]

All reported issues were addressed across 1 file

<sub>Reply with feedback, questions, or to request a fix.

Re-trigger cubic</sub>

[cubic-dev-ai]

1 issue found across 1 file (changes from recent commits).

Confidence score: 4/5

• In artboard_tool.rs, the test’s preliminary mouse move redraws overlays and clears the stale cage before pointer-down, so it bypasses the resize guard it’s meant to cover. Send pointer-down at the handle without that preliminary move.

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. When an issue isn't valid or won't be fixed in this PR, reply in its thread with the reason and then resolve the thread. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="editor/src/messages/tool/tool_messages/artboard_tool.rs">

<violation number="1" location="editor/src/messages/tool/tool_messages/artboard_tool.rs:1185">
P2: This `drag` redraws overlays before pointer-down, clearing the stale cage and bypassing the resize guard this test is meant to cover. Send pointer-down at the handle without the helper’s preliminary mouse move.</violation>
</file>

<sub>Reply with feedback, questions, or to request a fix.

Re-trigger cubic</sub>

[cubic-dev-ai]

P2: This drag redraws overlays before pointer-down, clearing the stale cage and bypassing the resize guard this test is meant to cover. Send pointer-down at the handle without the helper’s preliminary mouse move.

Prompt for AI agents

Check if this issue is valid — if so, understand the root cause and fix it. When an issue isn't valid or won't be fixed in this PR, reply in its thread with the reason and then resolve the thread. At editor/src/messages/tool/tool_messages/artboard_tool.rs, line 1185:

<comment>This `drag` redraws overlays before pointer-down, clearing the stale cage and bypassing the resize guard this test is meant to cover. Send pointer-down at the handle without the helper’s preliminary mouse move.</comment>

<file context>
@@ -1161,4 +1161,28 @@ mod test_artboard {
+		editor.press(Key::ArrowRight, ModifierKeys::SHIFT).await;
+		assert_bounds(&editor, artboard, [DVec2::new(100., 100.), DVec2::new(200., 160.)]);
+
+		drag(&mut editor, DVec2::new(150., 160.), DVec2::new(150., 220.), ModifierKeys::empty()).await;
+		assert_bounds(&editor, artboard, [DVec2::new(100., 100.), DVec2::new(200., 160.)]);
+	}
</file context>

Suggested change

	drag(&mut editor, DVec2::new(150., 160.), DVec2::new(150., 220.), ModifierKeys::empty()).await;
	editor.left_mousedown(150., 160., ModifierKeys::empty()).await;
	move_to(&mut editor, DVec2::new(150., 220.), ModifierKeys::empty()).await;
	editor.left_mouseup(150., 220., ModifierKeys::empty()).await;