Skip to content

Commit b348fab

Browse files
committed
Remove the unsupported Brush tool warning dialog now that raster performance is improved
1 parent 35ab266 commit b348fab

File tree

2 files changed

+2
-20
lines changed

2 files changed

+2
-20
lines changed

editor/src/messages/portfolio/document/node_graph/document_node_definitions.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -630,8 +630,8 @@ fn static_nodes() -> Vec<DocumentNodeDefinition> {
630630
NodeInput::value(TaggedValue::VectorData(VectorDataTable::default()), true),
631631
NodeInput::value(
632632
TaggedValue::Footprint(Footprint {
633-
transform: DAffine2::from_scale_angle_translation(DVec2::new(100., 100.), 0., DVec2::new(0., 0.)),
634-
resolution: UVec2::new(100, 100),
633+
transform: DAffine2::from_scale_angle_translation(DVec2::new(1000., 1000.), 0., DVec2::new(0., 0.)),
634+
resolution: UVec2::new(1000, 1000),
635635
..Default::default()
636636
}),
637637
false,

editor/src/messages/tool/tool_messages/brush_tool.rs

Lines changed: 0 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,6 @@ pub struct BrushTool {
2828
}
2929

3030
pub struct BrushOptions {
31-
legacy_warning_was_shown: bool,
3231
diameter: f64,
3332
hardness: f64,
3433
flow: f64,
@@ -41,7 +40,6 @@ pub struct BrushOptions {
4140
impl Default for BrushOptions {
4241
fn default() -> Self {
4342
Self {
44-
legacy_warning_was_shown: false,
4543
diameter: DEFAULT_BRUSH_SIZE,
4644
hardness: 0.,
4745
flow: 100.,
@@ -79,7 +77,6 @@ pub enum BrushToolMessageOptionsUpdate {
7977
Hardness(f64),
8078
Spacing(f64),
8179
WorkingColors(Option<Color>, Option<Color>),
82-
NoDisplayLegacyWarning,
8380
}
8481

8582
#[derive(Clone, Copy, Debug, Default, PartialEq, Eq)]
@@ -224,7 +221,6 @@ impl<'a> MessageHandler<ToolMessage, &mut ToolActionMessageContext<'a>> for Brus
224221
self.options.color.primary_working_color = primary;
225222
self.options.color.secondary_working_color = secondary;
226223
}
227-
BrushToolMessageOptionsUpdate::NoDisplayLegacyWarning => self.options.legacy_warning_was_shown = true,
228224
}
229225

230226
self.send_layout(responses, LayoutTarget::ToolOptions);
@@ -322,20 +318,6 @@ impl Fsm for BrushToolFsmState {
322318
document, global_tool_data, input, ..
323319
} = tool_action_data;
324320

325-
if !tool_options.legacy_warning_was_shown {
326-
responses.add(DialogMessage::DisplayDialogError {
327-
title: "Unsupported tool".into(),
328-
description: "
329-
The current Brush tool is a legacy feature with\n\
330-
significant quality and performance limitations.\n\
331-
It will be replaced soon by a new implementation.\n\
332-
"
333-
.trim()
334-
.into(),
335-
});
336-
responses.add(BrushToolMessage::UpdateOptions(BrushToolMessageOptionsUpdate::NoDisplayLegacyWarning));
337-
}
338-
339321
let ToolMessage::Brush(event) = event else { return self };
340322
match (self, event) {
341323
(BrushToolFsmState::Ready, BrushToolMessage::DragStart) => {

0 commit comments

Comments
 (0)