Opened 9 years ago

Closed 9 years ago

#2983 closed defect (fixed)

Remove/detach polygon 'end' node breaks polygon

Reported by: Toby Speight Owned by: Chris Browet
Priority: major Milestone:
Component: merkaartor Version:
Keywords: Cc:


Create a polygon. Remove or detach a non-end node; the polygon is still closed. Remove or detach one of the terminal nodes, and the polygon now becomes a polyline terminated at the previously penultimate nodes.

If we're treating polygons cleverly when splitting, then we should be consistent when removing nodes from them.

Change History (4)

comment:1 Changed 9 years ago by Chris Browet

Owner: changed from cbro@… to Chris Browet
Status: newassigned

I guess keeping the polygon closed when removing an end-node is way to go, right?

If the user wants to break the polygon, he should use the proper tool...

comment:2 Changed 9 years ago by Toby Speight

That's what I think, given how we're careful to hide the mechanics of where the start/end is.

I think that the Right Thing is to detect this case and join the loose ends when the start/end node is removed from the way.

I'm willing to look into it myself now (I submitted the ticket when I was busy with other stuff, but I have the time and knowledge now). Stop me if you're already on it!

comment:3 Changed 9 years ago by Chris Browet

I'm on it;-)

comment:4 Changed 9 years ago by Chris Browet

Resolution: fixed
Status: assignedclosed

(In [21420]) FIX : for consistency, when deleting the end-node of a polygon, don't break it (fixes #2983)

Note: See TracTickets for help on using tickets.