Page MenuHomePhabricator

PlanarFigure: for lines, crosses, etc. control points can be placed on top of each other
Closed, ResolvedPublic

Description

PlanarFigure interaction mechanism prevents control points of a PlanarFigure to be placed on top of each other. This works fine in the current version for ROIs and other structures with undefined number of control points. However, for lines, crosses and other figures with a fixed number of control points, the last point can be placed on top of another point; the corresponding check does not work properly.

The cause for this is that not all previous control points have are considered when checking for point overlap.

Event Timeline

[43739f]: Merge branch 'bug-9232-planarfigure-control-point-overlap'

Merged commits:

2011-09-01 09:30:52 Mathias Seitel [045cea]
Make control point overlap avoidance work for both ROIs and lines


2011-08-19 15:43:15 Markus Engel [457041]
Merge branch 'bug-9121-scrollinteractor-invoke-events' into releases/snapshot-201109

  • bug-9121-scrollinteractor-invoke-events: invoking special events when interaction is started or finished

Problem solved and solution commited to master and to branch releases/snapshot-201109.

[51f012]: Merge branch 'bug-9232-correct-planarfigure-interaction-rebased'

  • bu

Merged commits:

2011-09-21 17:56:42 Markus Engel [d3d1e4]
corrected logic for PlanarFigure interaction for double clicking

Solved another issue: with the above "fix" it was now possible to set e.g. a single line with just one double-click (leading to two control points exactly on the same position).

The fix from 2011-09-21 now makes sure that for BOTH ROIs (quasi-unlimited number of control points) AND for lines etc. (fixed number of control points), the check makes sure that control points cannot be placed on top of each other, and that the planar figure can be finalized as expected.