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

Path tools should filter out snapping to the procedurally generated contents of its own layer #1689

Open
Keavon opened this issue Mar 15, 2024 · 1 comment
Labels
Bug Something isn't working Feature New feature or request Paper Cut A small UX annoyance we should strive to improve Rust Involves Rust programming for the backend

Comments

@Keavon
Copy link
Member

Keavon commented Mar 15, 2024

For example, a Poisson-Disk Points node which generates circles on the inside of a shape. If you use the Path tool to modify that shape, it wants to snap to is own circles. We only want the Path tool snapping dragged points to other points in the same vector path, not the generated shapes that are derived from it.

@Keavon Keavon added Bug Something isn't working Good First Issue Good for newcomers Rust Involves Rust programming for the backend labels Mar 15, 2024
@0HyperCube
Copy link
Member

@Keavon currently you can snap a point to one of the previous segments in the path assuming it isn't one of the adjacent segments to the one you are moving. This was behaviour copied from Inkscape. Do you wish to keep this behaviour?
snapping to segments in the current path

If so it is more challenging to identify which data is derived procedurally and which data acts as the original.

@0HyperCube 0HyperCube added Feature New feature or request Paper Cut A small UX annoyance we should strive to improve and removed Good First Issue Good for newcomers labels Mar 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working Feature New feature or request Paper Cut A small UX annoyance we should strive to improve Rust Involves Rust programming for the backend
Projects
Status: Short-Term
Development

No branches or pull requests

2 participants