23 Nov
2014
23 Nov
'14
10:38 p.m.
Hi Jabier, I just discovered that you added a "default unit" to LivePathEffect::Effect class. Unfortunately, we will have to remove that, or at least make a drastic change there.
A LivePathEffect is meant to be a purely mathematical function, that depends as little as possible on the rest of Inkscape. To work with units, one would have to know the full item2document transform times the SVG units used, and use that in the effect calculation. I'm really not sure if we should do this.
What the code is doing now is wrong if an item has a transform applied to it.
regards, Johan