Skip to content

Fix editor crash due to mismanaged selected points on layers#2640

Merged
Keavon merged 9 commits into
GraphiteEditor:masterfrom
seam0s-dev:2636-editor-crash-fix
May 18, 2025
Merged

Fix editor crash due to mismanaged selected points on layers#2640
Keavon merged 9 commits into
GraphiteEditor:masterfrom
seam0s-dev:2636-editor-crash-fix

Conversation

@seam0s-dev

Copy link
Copy Markdown
Contributor

Fixes #2636

@seam0s-dev

seam0s-dev commented May 12, 2025

Copy link
Copy Markdown
Contributor Author

Actions to be affected when anchors/handles overlays is toggled:

  • Nudging selected points with shift+[arrow key].
  • Reposition a selected point (primary handle, end handle or anchor) with mouse pointer.
  • Move multiple selected points (primary handle, end handle or anchor) with mouse pointer.
  • Selecting/deselecting points on the currently selected layer with the path tool.

Actions not to be affected when anchors/handles overlays is toggled:

  • Changing selected filled layer(s) while in the path tool.
  • Move around selected filled layer(s) while in the path tool. (currently, broken)

@seam0s-dev seam0s-dev marked this pull request as ready for review May 17, 2025 09:07
@Keavon Keavon enabled auto-merge (squash) May 18, 2025 21:09
@Keavon Keavon merged commit ea59f10 into GraphiteEditor:master May 18, 2025
4 checks passed
@seam0s-dev seam0s-dev deleted the 2636-editor-crash-fix branch May 28, 2025 15:48
@Keavon

Keavon commented Jun 13, 2026

Copy link
Copy Markdown
Member

Hi @seam0s-dev, thanks again for this code contribution to the project! We're still hoping you will respond to the request to relicense this code. Please see #4208 ASAP, thank you!

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.

Double-click selecting anchors in a filled region, with anchors disabled, crashes the editor

2 participants