Is this going to be implemented?
The wait_for_input
property is not defined for any of the new operators, i get a bunch of warnings:
This should be an actual enum property instead of a bool, and then use expand=True
in the ui code to draw as toggle buttons rather than drawing two buttons for the same property.
Curves are copied in their entirety first, then point attributes get rewritten: bke::CurvesGeometry dst_curves(src_curves)
. It's fine, couldn't find anything wrong in testing either, custom…
I think we should change the attributes to store a loc/rot/scale transform instead of the barycentric vectors + offset. I don't want to expose the legacy 3-point method unless necessary. And if the texture transform is already stored as general barycentric unit vectors then we might as well store the linear transform directly.
I suggest to just store the loc/rot/scale instead of three vectors. That's a bit more flexible and doesn't rely so much on this arbitrary GPv2 method of picking three points to generate the transform.
This function is used only inside grease_pencil.cc, doesn't need to be exposed in the header.