Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Disable the Path tool's "Colinear Handles" checkbox when no interior anchors are selected #2339

Merged

Conversation

indierusty
Copy link
Contributor

@indierusty indierusty commented Mar 2, 2025

@Keavon
Copy link
Member

Keavon commented Mar 6, 2025

!build

Copy link

github-actions bot commented Mar 6, 2025

📦 Build Complete for 2d02cca
https://fdeab65c.graphite.pages.dev

@Keavon
Copy link
Member

Keavon commented Mar 6, 2025

Can you please make it so, when multi-selecting both inner points and end point(s), it filters out the end points and displays the checkmark status of just the inner points? And then if clicking the checkbox to toggle those inner points, it ignores the endpoints but still correctly toggles it for the rest, updating the checkbox status as well.

@Keavon Keavon force-pushed the disable-collinear-handles-option branch from 2d02cca to 6b699b9 Compare March 6, 2025 13:28
@indierusty indierusty marked this pull request as draft March 6, 2025 15:28
@Keavon Keavon changed the title Disable Colinear Handles checkbox when only endpoint or its handle is selected. Disable the Path tool's "Colinear Handles" checkbox when no interior anchors are selected Mar 7, 2025
@indierusty indierusty marked this pull request as ready for review March 7, 2025 13:48
@Keavon
Copy link
Member

Keavon commented Mar 7, 2025

!build

Copy link

github-actions bot commented Mar 7, 2025

📦 Build Complete for 997a77c
https://ac86a9b9.graphite.pages.dev

@Keavon Keavon merged commit 0a91dd2 into GraphiteEditor:master Mar 8, 2025
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