From cb35873e29f6f2cc1a92d82d3189d23c63aafeb1 Mon Sep 17 00:00:00 2001 From: Bryan Housel Date: Thu, 12 Jan 2017 20:08:42 +0530 Subject: [PATCH] Don't allow continue from the connecting vertex of a closed way Now that addNode tries to preserve circularity, this could cause the continued segment to loop back. And anyway it's confusing why one vertex allows it and none others do. --- modules/operations/continue.js | 1 + 1 file changed, 1 insertion(+) diff --git a/modules/operations/continue.js b/modules/operations/continue.js index 4feaab5ef..5d24f463a 100644 --- a/modules/operations/continue.js +++ b/modules/operations/continue.js @@ -15,6 +15,7 @@ export function operationContinue(selectedIDs, context) { function candidateWays() { return graph.parentWays(vertex).filter(function(parent) { return parent.geometry(graph) === 'line' && + !parent.isClosed() && parent.affix(vertex.id) && (geometries.line.length === 0 || geometries.line[0] === parent); });