GraphiteEditor / Graphite

2D vector & raster editor that melds traditional layers & tools with a modern node-based, non-destructive, procedural workflow.
https://graphite.rs
Apache License 2.0
9.03k stars 437 forks source link

Preserve point IDs without a path node #2068

Open 0HyperCube opened 2 hours ago

0HyperCube commented 2 hours ago

When there is no path node, the compute_modified_vector function (included below) relies on the click_targets. However click targets don't store PointIds, SegementIds, or colinear handles. This leads to a couple of issues:

https://github.com/GraphiteEditor/Graphite/blob/c3a3c4c907419e2ebb9a382546a1d7b66f57cdc0/editor/src/messages/portfolio/document/utility_types/network_interface.rs#L2826-L2831

adamgerhant commented 2 hours ago

I think it would be best to automatically insert a Path node when the editor creates vector data. I don't think relying on the click targets is a good idea.