Bug 345713

Summary:	Unable to join two curves with Path editing tool
Product:	[Applications] krita	Reporter:	Alvyn <alvyn.liang>
Component:	Layers/Vector	Assignee:	Krita Bugs <krita-bugs-null>
Status:	RESOLVED FIXED
Severity:	wishlist	CC:	alvyn.liang, halla, tamtamy.tymona
Priority:	NOR
Version:	2.9.1
Target Milestone:	---
Platform:	macOS (DMG)
OS:	All
Latest Commit:		Version Fixed In:

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.