Skip to content

Improve the Shape tool's arrow drawing controls#3650

Merged
Keavon merged 6 commits intoGraphiteEditor:masterfrom
krVatsal:arrow-tool-features-improvement
Feb 15, 2026
Merged

Improve the Shape tool's arrow drawing controls#3650
Keavon merged 6 commits intoGraphiteEditor:masterfrom
krVatsal:arrow-tool-features-improvement

Conversation

@krVatsal
Copy link
Contributor

Based on feedback on #3343

This PR adds configurable width controls for the Arrow shape tool and ensures arrow dimensions remain constant during drawing, with only the length changing.

Fixed Arrow Dimensions During Drawing

Previously, all arrow dimensions scaled proportionally with the arrow length, causing the entire arrow to "zoom" as it was drawn. Now:

  • Width dimensions remain constant (shaft, head width, head length stay fixed)
  • Only the arrow length changes as you drag
  • This creates a more natural drawing experience where the arrow stretches rather than scales

@krVatsal
Copy link
Contributor Author

Hey @Keavon, can you guide me what could be the possible reason for this slower runtime after i updated the branch?

@Keavon Keavon force-pushed the arrow-tool-features-improvement branch from 745f93f to 56faa3b Compare January 24, 2026 21:44
@Keavon
Copy link
Member

Keavon commented Jan 26, 2026

@krVatsal I will need more details to be able to help with that. Discord may be a better venue to ask for help about that.

Signed-off-by: krVatsal <kumarvatsal34@gmail.com>
Signed-off-by: krVatsal <kumarvatsal34@gmail.com>
@Keavon Keavon force-pushed the arrow-tool-features-improvement branch from aae50c4 to 2f56bf2 Compare February 14, 2026 07:21
@Keavon Keavon changed the title Arrow tool features improvement Improve the Shape tool's arrow drawing controls Feb 15, 2026
@Keavon Keavon enabled auto-merge (squash) February 15, 2026 21:04
@Keavon Keavon merged commit 82f7dc7 into GraphiteEditor:master Feb 15, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants