Comment 11 for bug 1246550

Revision history for this message
Johan Engelen (johanengelen) wrote :

This is a difficult fix. Krzysztof can help out with this I hope. I had a look into it, but don't know enough about the PathManipulator code to make a good fix, without his input first.
Krzys: the problem is that "addCanvasIndicators" is not called for any LPE. The intent of addCanvasIndicators is to show any kind of helper indicators of what the LPE is doing (for example, the "switcher" thingie for LPE Knot, back in 0.47).
addCanvasIndicators is called (for the effect itself and for its parameters) by Effect::getCanvasIndicators.
But Effect::getCanvasIndicators is never called.
Somewhere, the pathmanipulator code should call getCanvasIndicators, and draw those indicators as well as the standard path helperpath. Best I can think of is hooking the code up somewhere into the call graph of PathManipulator::_updateOutline().

Thanks!