Skip to content

Add Gradient tool test for select and delete removes a stop #2571

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Apr 19, 2025

Conversation

Sidharth-Singh10
Copy link
Contributor

Closes part of #2488

@Keavon Keavon added the Testing Unit and integration tests label Apr 16, 2025
Comment on lines 571 to 583
async fn double_click_at(editor: &mut EditorTestUtils, position: DVec2) {
editor
.handle_message(InputPreprocessorMessage::DoubleClick {
editor_mouse_state: EditorMouseState {
editor_position: position,
mouse_keys: MouseKeys::LEFT,
scroll_delta: ScrollDelta::default(),
},
modifier_keys: ModifierKeys::empty(),
})
.await;
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good, however why is this not part of the EditorTestUtils like all the other testing utilities? It doesn't seem to be specific to the gradient tool.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

@Sidharth-Singh10 Sidharth-Singh10 force-pushed the gradient_test3 branch 2 times, most recently from 2047173 to 7b6f4f7 Compare April 18, 2025 23:06
@Sidharth-Singh10
Copy link
Contributor Author

resolved conflicts

@0HyperCube 0HyperCube enabled auto-merge (squash) April 19, 2025 16:42
auto-merge was automatically disabled April 19, 2025 18:45

Head branch was pushed to by a user without write access

@Sidharth-Singh10
Copy link
Contributor Author

@0HyperCube the auto-merge was disabled because there was a merge conflict again

@Keavon Keavon merged commit 6196dbf into GraphiteEditor:master Apr 19, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Testing Unit and integration tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants