diff options
| author | Jabier Arraiza Cenoz <jabier.arraiza@marker.es> | 2015-08-14 08:20:45 +0000 |
|---|---|---|
| committer | Jabiertxof <jtx@jtx.marker.es> | 2015-08-14 08:20:45 +0000 |
| commit | 439df6b61c37528b2ca6210f77a4bb50aa692906 (patch) | |
| tree | 48d4bd6fbc4aa09adf48d1d41db132cde4846a3a /src/ui/tool/path-manipulator.cpp | |
| parent | Refactor of BSPline code attemping to fix the duplicate end node bug, not sur... (diff) | |
| download | inkscape-439df6b61c37528b2ca6210f77a4bb50aa692906.tar.gz inkscape-439df6b61c37528b2ca6210f77a4bb50aa692906.zip | |
A bit more refactor of BSPline tool. Still the bug duplicating end node :(
(bzr r14301)
Diffstat (limited to 'src/ui/tool/path-manipulator.cpp')
| -rw-r--r-- | src/ui/tool/path-manipulator.cpp | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/src/ui/tool/path-manipulator.cpp b/src/ui/tool/path-manipulator.cpp index 7c9fb5841..7b7bc98ee 100644 --- a/src/ui/tool/path-manipulator.cpp +++ b/src/ui/tool/path-manipulator.cpp @@ -56,7 +56,7 @@ enum PathChange { }; } // anonymous namespace -const double HANDLE_CUBIC_GAP = 0.01; +const double HANDLE_CUBIC_GAP = 0.001; const double NO_POWER = 0.0; const double DEFAULT_START_POWER = 0.3334; @@ -695,10 +695,12 @@ unsigned PathManipulator::_deleteStretch(NodeList::iterator start, NodeList::ite // if we are removing, we readjust the handlers if(_isBSpline()){ if(start.prev()){ - start.prev()->front()->setPosition(_bsplineHandleReposition(start.prev()->front(),start.prev()->back())); + double bspline_weight = _bsplineHandlePosition(start.prev()->back(), false); + start.prev()->front()->setPosition(_bsplineHandleReposition(start.prev()->front(), bspline_weight)); } if(end){ - end->back()->setPosition(_bsplineHandleReposition(end->back(),end->front())); + double bspline_weight = _bsplineHandlePosition(end->front(), false); + end->back()->setPosition(_bsplineHandleReposition(end->back(),bspline_weight)); } } |