From a conversation with Krzysztof there is also a goal to replace Knot (if by Knot you mean SPKnot) entirely with UI::ControlPoint, UI::SelectableControlPoint, and UI::Node since SPKnot had certain aspects that aren’t well designed. I can forward the conversation to you but it’s started as a thread in the mailing list with topic: “SPKnot, KnotHolderEntity, and KnotHolder”.
Regards, _______________________ Papoj "Hua" Thamjaroenporn papojt@...3117...
On Dec 6, 2015, at 6:46 PM, Martin Owens <doctormo@...400...> wrote:
Developers,
I've just been scrutinising the code for guidelines, specifically the code that draws the little circle indicating the main co-ordinate of the guideline.
What it looks like is back in 2009 the code src/display/sp-ctrlpoint code was a clone of the much older original code in src/display/sp-ctrlquadr which we've had since at least 2005.
ctrlquadr draws a square using cairo, used by ui/tools/text-tool only ctrlpoint draws a circle using cairo, used by display/guideline only
If we could replace these two instances with Knot objects, we should be able to remove this code completely? (11.8KB, 1150 lines) Are there any land minds you are aware of with this code? (somewhere it's being used without being able to find it?)
For comparison, SPCtrlLine and SPCtrlCurve are used by pen-tool, mesh-tool, gradient-tool, text-tool and the measure-tool. So they're very well used in comparison.
Best Regards, Martin Owens
Go from Idea to Many App Stores Faster with Intel(R) XDK Give your users amazing mobile app experiences with Intel(R) XDK. Use one codebase in this all-in-one HTML5 development environment. Design, debug & build mobile apps & 2D/3D high-impact games for multiple OSs. http://pubads.g.doubleclick.net/gampad/clk?id=254741911&iu=/4140 _______________________________________________ Inkscape-devel mailing list Inkscape-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/inkscape-devel