fixed the arrow's parameters

Signed-off-by: krVatsal <kumarvatsal34@gmail.com>

Author: krVatsal

editor/src/messages/tool/common_functionality/shapes/arrow_shape.rs

@@ -35,6 +35,9 @@ impl Arrow {
 		layer: LayerNodeIdentifier,
 		tool_data: &mut ShapeToolData,
 		_modifier: ShapeToolModifierKey,
+		shaft_width: f64,
+		head_width: f64,
+		head_length: f64,
 		responses: &mut VecDeque<Message>,
 	) {
 		// Track current mouse position in viewport space
@@ -56,11 +59,7 @@ impl Arrow {
 			return;
 		};
 
-		// Calculate proportional dimensions based on arrow length
-		let shaft_width = length_document * 0.1;
-		let head_width = length_document * 0.3;
-		let head_length = length_document * 0.2;
-
+		// Use fixed dimensions from tool options - only length changes during drag
 		// Update Arrow node parameters with document space coordinates (like Line tool)
 		responses.add(NodeGraphMessage::SetInput {
 			input_connector: InputConnector::node(node_id, 1),

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

@@ -44,6 +44,9 @@ pub struct ShapeToolOptions {
 	grid_type: GridType,
 	spiral_type: SpiralType,
 	turns: f64,
+	arrow_shaft_width: f64,
+	arrow_head_width: f64,
+	arrow_head_length: f64,
 }
 
 impl Default for ShapeToolOptions {
@@ -58,6 +61,9 @@ impl Default for ShapeToolOptions {
 			spiral_type: SpiralType::Archimedean,
 			turns: 5.,
 			grid_type: GridType::Rectangular,
+			arrow_shaft_width: 14.0,
+			arrow_head_width: 32.0,
+			arrow_head_length: 28.0,
 		}
 	}
 }
@@ -76,6 +82,9 @@ pub enum ShapeOptionsUpdate {
 	SpiralType(SpiralType),
 	Turns(f64),
 	GridType(GridType),
+	ArrowShaftWidth(f64),
+	ArrowHeadWidth(f64),
+	ArrowHeadLength(f64),
 }
 
 #[impl_message(Message, ToolMessage, Shape)]
@@ -236,6 +245,51 @@ fn create_weight_widget(line_weight: f64) -> WidgetInstance {
 		.widget_instance()
 }
 
+fn create_arrow_shaft_width_widget(shaft_width: f64) -> WidgetInstance {
+	NumberInput::new(Some(shaft_width))
+		.unit(" px")
+		.label("Shaft")
+		.min(0.1)
+		.max(1000.)
+		.on_update(|number_input: &NumberInput| {
+			ShapeToolMessage::UpdateOptions {
+				options: ShapeOptionsUpdate::ArrowShaftWidth(number_input.value.unwrap()),
+			}
+			.into()
+		})
+		.widget_instance()
+}
+
+fn create_arrow_head_width_widget(head_width: f64) -> WidgetInstance {
+	NumberInput::new(Some(head_width))
+		.unit(" px")
+		.label("Head W")
+		.min(0.1)
+		.max(1000.)
+		.on_update(|number_input: &NumberInput| {
+			ShapeToolMessage::UpdateOptions {
+				options: ShapeOptionsUpdate::ArrowHeadWidth(number_input.value.unwrap()),
+			}
+			.into()
+		})
+		.widget_instance()
+}
+
+fn create_arrow_head_length_widget(head_length: f64) -> WidgetInstance {
+	NumberInput::new(Some(head_length))
+		.unit(" px")
+		.label("Head L")
+		.min(0.1)
+		.max(1000.)
+		.on_update(|number_input: &NumberInput| {
+			ShapeToolMessage::UpdateOptions {
+				options: ShapeOptionsUpdate::ArrowHeadLength(number_input.value.unwrap()),
+			}
+			.into()
+		})
+		.widget_instance()
+}
+
 fn create_spiral_type_widget(spiral_type: SpiralType) -> WidgetInstance {
 	let entries = vec![vec![
 		MenuListEntry::new("Archimedean").label("Archimedean").on_commit(move |_| {
@@ -304,6 +358,15 @@ impl LayoutHolder for ShapeTool {
 			widgets.push(Separator::new(SeparatorStyle::Unrelated).widget_instance());
 		}
 
+		if self.options.shape_type == ShapeType::Arrow {
+			widgets.push(create_arrow_shaft_width_widget(self.options.arrow_shaft_width));
+			widgets.push(Separator::new(SeparatorStyle::Related).widget_instance());
+			widgets.push(create_arrow_head_width_widget(self.options.arrow_head_width));
+			widgets.push(Separator::new(SeparatorStyle::Related).widget_instance());
+			widgets.push(create_arrow_head_length_widget(self.options.arrow_head_length));
+			widgets.push(Separator::new(SeparatorStyle::Unrelated).widget_instance());
+		}
+
 		if self.options.shape_type != ShapeType::Line {
 			widgets.append(&mut self.options.fill.create_widgets(
 				"Fill",
@@ -414,6 +477,15 @@ impl<'a> MessageHandler<ToolMessage, &mut ToolActionMessageContext<'a>> for Shap
 			ShapeOptionsUpdate::GridType(grid_type) => {
 				self.options.grid_type = grid_type;
 			}
+			ShapeOptionsUpdate::ArrowShaftWidth(shaft_width) => {
+				self.options.arrow_shaft_width = shaft_width;
+			}
+			ShapeOptionsUpdate::ArrowHeadWidth(head_width) => {
+				self.options.arrow_head_width = head_width;
+			}
+			ShapeOptionsUpdate::ArrowHeadLength(head_length) => {
+				self.options.arrow_head_length = head_length;
+			}
 		}
 
 		update_dynamic_hints(&self.fsm_state, responses, &self.tool_data);
@@ -905,7 +977,18 @@ impl Fsm for ShapeToolFsmState {
 					ShapeType::Star => Star::update_shape(document, input, viewport, layer, tool_data, modifier, responses),
 					ShapeType::Circle => Circle::update_shape(document, input, viewport, layer, tool_data, modifier, responses),
 					ShapeType::Arc => Arc::update_shape(document, input, viewport, layer, tool_data, modifier, responses),
-					ShapeType::Arrow => Arrow::update_shape(document, input, viewport, layer, tool_data, modifier, responses),
+					ShapeType::Arrow => Arrow::update_shape(
+						document,
+						input,
+						viewport,
+						layer,
+						tool_data,
+						modifier,
+						tool_options.arrow_shaft_width,
+						tool_options.arrow_head_width,
+						tool_options.arrow_head_length,
+						responses,
+					),
 					ShapeType::Spiral => Spiral::update_shape(document, input, viewport, layer, tool_data, responses),
 					ShapeType::Grid => Grid::update_shape(document, input, layer, tool_options.grid_type, tool_data, modifier, responses),
 					ShapeType::Rectangle => Rectangle::update_shape(document, input, viewport, layer, tool_data, modifier, responses),