On Mon, 2013-09-09 at 18:31 +0200, Sebastian Götte wrote:
I am slightly concerned about the performance of this "live updating" approach, especially in more complex scenarios including libavoid routing. The old code just updated the connectors after all changes have been made, but the live update looks so nice... Fortunately, I am the proud owner of a very slow laptop, so I can test this rather well.
I would leave performance issues to a later date.
I started adding some status bar messages. If I am not entirely mistaken, the proper place for UI documentation should be the "inkscape-docs" repository on launchpad [0].
Probably in the long term. For the moment, I would just create a Wiki page somewhere. When the code gets merged into trunk, then the release notes should be updated:
http://wiki.inkscape.org/wiki/index.php/Release_notes/0.49
Ok, I now coupled object and point selection. Points/items can still be selected individually, but when the selection of an item is changed, the selection of all contained points is changed as well. When the selection of a connector is changed, the selection of the connector's endpoints is changed. The only quirk of this approac is that when two connectors share an endpoint and one is selected and [Delete] is pressed, the other connector is deleted as well since at the point [Delete] is pressed the selection contains both the one connector and both of its endpoints, including the shared one and when the shared endpoint is deleted, the other connector is deleted. I think that's not too much of a problem, though.
When a connector is deleted, I would not delete the points. That should be a separate operation (explicitly select the points and then delete them).
I still have one spurious segfault left that sometimes occurs when deleting an SPPoint. The next time I trigger it I will look into this one. The current code can be found on launchpad [1].
Great improvements!
Tav