Skip to content

Delete the partially drawn entity when ESC is pressed on pending oper…#1386

Merged
phkahler merged 1 commit intosolvespace:masterfrom
phkahler:esc
Jul 27, 2023
Merged

Delete the partially drawn entity when ESC is pressed on pending oper…#1386
phkahler merged 1 commit intosolvespace:masterfrom
phkahler:esc

Conversation

@phkahler
Copy link
Copy Markdown
Member

…ations by using UndoUndo()

Fixes #1385

@ruevs
Copy link
Copy Markdown
Member

ruevs commented Jul 26, 2023

Hmm... I'll need to pull compile and trace through this if you want an opinion.

@phkahler
Copy link
Copy Markdown
Member Author

Hmm... I'll need to pull compile and trace through this if you want an opinion.

Actually no. It works.

@phkahler phkahler merged commit b34d9a2 into solvespace:master Jul 27, 2023
@ruevs
Copy link
Copy Markdown
Member

ruevs commented Nov 5, 2024

Actually no. It works.

@phkahler actually no, it makes it impossible to draw open bezier splines 😛 😃

@phkahler
Copy link
Copy Markdown
Member Author

phkahler commented Nov 6, 2024

Actually no. It works.

@phkahler actually no, it makes it impossible to draw open bezier splines 😛 😃

@ruevs Right click to end the spline. But I'm definitely open to a version of ESC that stops the spine creation rather than undoing it.

@ruevs
Copy link
Copy Markdown
Member

ruevs commented Nov 6, 2024

I'm so used to ESC that I completely forgot about right-click :-)

ruevs added a commit to ruevs/solvespace that referenced this pull request Nov 21, 2024
With this pressing ESC while drawing a bezier curve finishes it instead of
deleting it. This reverts the behaviour introduced in b34d9a2
(p.r. solvespace#1386). ESC still deletes the last segment of line segments and
cancels arcs and circles.

This makes it easier to draw non closed curves - otherwise the only way
to finish them was with a right-click.
@ruevs
Copy link
Copy Markdown
Member

ruevs commented Nov 21, 2024

Made a pull request #1497

phkahler pushed a commit that referenced this pull request Nov 21, 2024
With this pressing ESC while drawing a bezier curve finishes it instead of
deleting it. This reverts the behaviour introduced in b34d9a2
(p.r. #1386). ESC still deletes the last segment of line segments and
cancels arcs and circles.

This makes it easier to draw non closed curves - otherwise the only way
to finish them was with a right-click.
devin-ai-integration Bot pushed a commit to erkinalp/solvespace that referenced this pull request Apr 3, 2025
With this pressing ESC while drawing a bezier curve finishes it instead of
deleting it. This reverts the behaviour introduced in b34d9a2
(p.r. solvespace#1386). ESC still deletes the last segment of line segments and
cancels arcs and circles.

This makes it easier to draw non closed curves - otherwise the only way
to finish them was with a right-click.
dennisyangji pushed a commit to Orthogonalpub/ode_solvespace that referenced this pull request Nov 25, 2025
With this pressing ESC while drawing a bezier curve finishes it instead of
deleting it. This reverts the behaviour introduced in 10f2c44
(p.r. solvespace#1386). ESC still deletes the last segment of line segments and
cancels arcs and circles.

This makes it easier to draw non closed curves - otherwise the only way
to finish them was with a right-click.
@phkahler phkahler deleted the esc branch December 30, 2025 00:00
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.

Feature Request: Drop last segment in a line when pressing escape

2 participants