From b348fabfd247fb35f9ff97eb797ee477d3daf445 Mon Sep 17 00:00:00 2001 From: Keavon Chambers Date: Mon, 28 Jul 2025 17:03:48 -0700 Subject: [PATCH] Remove the unsupported Brush tool warning dialog now that raster performance is improved --- .../node_graph/document_node_definitions.rs | 4 ++-- .../messages/tool/tool_messages/brush_tool.rs | 18 ------------------ 2 files changed, 2 insertions(+), 20 deletions(-) diff --git a/editor/src/messages/portfolio/document/node_graph/document_node_definitions.rs b/editor/src/messages/portfolio/document/node_graph/document_node_definitions.rs index d24ebaa2..080353b7 100644 --- a/editor/src/messages/portfolio/document/node_graph/document_node_definitions.rs +++ b/editor/src/messages/portfolio/document/node_graph/document_node_definitions.rs @@ -630,8 +630,8 @@ fn static_nodes() -> Vec { NodeInput::value(TaggedValue::VectorData(VectorDataTable::default()), true), NodeInput::value( TaggedValue::Footprint(Footprint { - transform: DAffine2::from_scale_angle_translation(DVec2::new(100., 100.), 0., DVec2::new(0., 0.)), - resolution: UVec2::new(100, 100), + transform: DAffine2::from_scale_angle_translation(DVec2::new(1000., 1000.), 0., DVec2::new(0., 0.)), + resolution: UVec2::new(1000, 1000), ..Default::default() }), false, diff --git a/editor/src/messages/tool/tool_messages/brush_tool.rs b/editor/src/messages/tool/tool_messages/brush_tool.rs index b17a2b17..c63bdb3e 100644 --- a/editor/src/messages/tool/tool_messages/brush_tool.rs +++ b/editor/src/messages/tool/tool_messages/brush_tool.rs @@ -28,7 +28,6 @@ pub struct BrushTool { } pub struct BrushOptions { - legacy_warning_was_shown: bool, diameter: f64, hardness: f64, flow: f64, @@ -41,7 +40,6 @@ pub struct BrushOptions { impl Default for BrushOptions { fn default() -> Self { Self { - legacy_warning_was_shown: false, diameter: DEFAULT_BRUSH_SIZE, hardness: 0., flow: 100., @@ -79,7 +77,6 @@ pub enum BrushToolMessageOptionsUpdate { Hardness(f64), Spacing(f64), WorkingColors(Option, Option), - NoDisplayLegacyWarning, } #[derive(Clone, Copy, Debug, Default, PartialEq, Eq)] @@ -224,7 +221,6 @@ impl<'a> MessageHandler> for Brus self.options.color.primary_working_color = primary; self.options.color.secondary_working_color = secondary; } - BrushToolMessageOptionsUpdate::NoDisplayLegacyWarning => self.options.legacy_warning_was_shown = true, } self.send_layout(responses, LayoutTarget::ToolOptions); @@ -322,20 +318,6 @@ impl Fsm for BrushToolFsmState { document, global_tool_data, input, .. } = tool_action_data; - if !tool_options.legacy_warning_was_shown { - responses.add(DialogMessage::DisplayDialogError { - title: "Unsupported tool".into(), - description: " - The current Brush tool is a legacy feature with\n\ - significant quality and performance limitations.\n\ - It will be replaced soon by a new implementation.\n\ - " - .trim() - .into(), - }); - responses.add(BrushToolMessage::UpdateOptions(BrushToolMessageOptionsUpdate::NoDisplayLegacyWarning)); - } - let ToolMessage::Brush(event) = event else { return self }; match (self, event) { (BrushToolFsmState::Ready, BrushToolMessage::DragStart) => {