Ticket #1470 (closed defect: fixed)
Opened 2012-09-11T15:51:39-05:00
Last modified 2012-12-17T12:28:22-06:00
Eliminate bogus extra polygon tool
| Reported by: | curtis | Owned by: | curtis |
|---|---|---|---|
| Priority: | major | Milestone: |
|
| Component: | Tools | Version: | 2.0.0-beta4 |
| Severity: | serious | Keywords: | |
| Cc: | Blocked By: | ||
| Blocking: | #1574 |
Description
There is a grayed out polygon tool next to the working polygon tool on the Swing toolbar. It was put there as a workaround to another issue, but of course we need to eliminate it.
Change History
comment:1 Changed 2012-09-11T15:53:07-05:00 by curtis
comment:2 Changed 2012-09-11T16:27:49-05:00 by dscho
- Status changed from new to assigned
We need to merge the two. In other words, we cannot use the IJBezierTool as-is, because it would not allow us to add handling for subtracting a new polygon/Bézier from the current composite ROI using the Alt key.
comment:4 Changed 2012-12-17T12:25:48-06:00 by bdezonia
- Blocking 878 removed
- Milestone changed from imagej2-b7-ndim-data to imagej2-b6-legacy-undo
comment:7 Changed 2012-12-17T12:27:03-06:00 by bdezonia
- Status changed from assigned to closed
- Resolution set to fixed
One way to fix it would be to add a "visible" flag to the @Plugin annotation (I want to do this anyway) but I question whether we really need to have this tool at all. Is the problem due to the conflation of ImageJ Tools with JHotDraw adapters? If so, another solution could be to split the logic of those two things into two separate classes, with JHotDrawAdapter no longer extending Tool.