Bug 345713 - Unable to join two curves with Path editing tool
Summary: Unable to join two curves with Path editing tool
Status: RESOLVED FIXED
Alias: None
Product: krita
Classification: Applications
Component: Layers/Vector (show other bugs)
Version: 2.9.1
Platform: macOS (DMG) All
: NOR wishlist
Target Milestone: ---
Assignee: Krita Bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-03-31 03:39 UTC by Alvyn
Modified: 2020-04-07 16:09 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Alvyn 2015-03-31 03:39:31 UTC
When two curves are selected with "Shape handling" tool, one can choose "Path editing" tool to edit two curves concurrently. It will make more sense if one can select two terminal points of two curves and connect two points with "Join with segments" or "Merge points" tool so that two curves become one curve.

Now this function only work on two terminal points of the same curve.

Reproducible: Always

Steps to Reproduce:
1. Select two curves in a vector layer with shape handling
2. Click on "Path editing" tool
3. Select two terminal points of each of the curves
4. Try to use "Join with segments" or "Merge points" tool  to combine two curves.

Actual Results:  
No merge occur

Expected Results:  
Two curves become one curves with the targeting points merged.
Comment 1 Halla Rempt 2015-04-08 09:22:29 UTC
Yes, this would be good to have. Thanks for you report!
Comment 2 Halla Rempt 2016-01-24 10:32:06 UTC
WISHGROUP: Vector kickstarter main project
Comment 3 Dmitry Kazakov 2017-03-20 15:07:12 UTC
Git commit ca57e0c2acc8ba256f2e75ec5b64d9c6bfa53e1a by Dmitry Kazakov.
Committed on 20/03/2017 at 14:10.
Pushed by dkazakov into branch 'kazakov/svg-loading'.

Implement merging of path points of separate shapes

M  +1    -0    libs/flake/CMakeLists.txt
M  +8    -6    libs/flake/KoPathShape.cpp
M  +2    -2    libs/flake/KoPathShape.h
M  +6    -1    libs/flake/KoSelection.cpp
A  +141  -0    libs/flake/commands/KoMultiPathPointMergeCommand.cpp     [License: GPL (v2+)]
A  +59   -0    libs/flake/commands/KoMultiPathPointMergeCommand.h     [License: GPL (v2+)]
M  +50   -27   libs/flake/commands/KoPathCombineCommand.cpp
M  +4    -0    libs/flake/commands/KoPathCombineCommand.h
M  +26   -6    libs/flake/commands/KoPathPointMergeCommand.cpp
M  +2    -0    libs/flake/commands/KoPathPointMergeCommand.h
M  +253  -0    libs/flake/tests/TestPointMergeCommand.cpp
M  +8    -0    libs/flake/tests/TestPointMergeCommand.h
M  +58   -36   libs/flake/tools/KoPathTool.cpp
M  +1    -0    libs/flake/tools/KoPathTool.h

https://commits.kde.org/krita/ca57e0c2acc8ba256f2e75ec5b64d9c6bfa53e1a
Comment 4 Tiar 2020-04-07 16:09:53 UTC
It seems to be implemented already.