Ticket #716 (accepted defect)

Opened 7 years ago

Last modified 6 years ago

ProbeTool cursor is off

Reported by: bdezonia Owned by: curtis
Priority: major Milestone: imagej2-unscheduled
Component: Tools Version:
Severity: serious Keywords:
Cc: Blocked By:
Blocking: #1629

Description

Now that the ProbeTool is always on its using a different cursor. The point under the tip of the cursor is not the same as the point being displayed in the status area. We may need to edit cursor image so that the tip is at the center. But maybe not because the other tools work fine (overlays drag at the correct points, etc.)

Curtis, assign to whomever you think is appropriate.

Change History

comment:1 Changed 7 years ago by curtis

The behavior seems the same as before to me—wrong, but the same. That is, the threshold falls at the center of each pixel rather than the boundaries. It is very likely a bug with JHotDrawImageCanvas.panelToImageCoords.

comment:2 Changed 7 years ago by curtis

Dscho fixed the panelToImageCoords bug. The probe value is no longer offset from the pixel boundary. However, everything is off by one.

comment:3 Changed 7 years ago by curtis

  • Status changed from new to accepted

comment:4 Changed 7 years ago by curtis

  • Milestone changed from imagej-2.0-beta1 to imagej-2.0-beta2

comment:5 Changed 6 years ago by curtis

  • Blocking 1242 added

comment:6 Changed 6 years ago by curtis

As of 1bb013f3, the probe is no longer wildly off, but rather always off by one data index (plus one screen pixel) in both X and Y. The bug is due to a coordinate system skew between the JHotDrawImageCanvas and DefaultImageCanvas. See also #1190 for another symptom of the same bug.

comment:7 Changed 6 years ago by curtis

  • Milestone changed from imagej-2.0.0-beta3 to imagej-2.0.0-beta4

comment:8 Changed 6 years ago by curtis

  • Blocking 1325 added; 1242 removed

comment:9 Changed 6 years ago by curtis

  • Blocking 1358 added; 1325 removed
  • Milestone changed from imagej-2.0.0-beta4 to imagej-2.0.0-beta5

comment:10 Changed 6 years ago by bdezonia

  • Blocking 1629 added; 1358 removed

comment:11 Changed 6 years ago by bdezonia

  • Milestone changed from imagej2-b7-ndim-data to imagej2-unscheduled
Note: See TracTickets for help on using tickets.