Ticket #1190 (closed defect: moved)

Opened 8 years ago

Last modified 7 years ago

Drawing tools are misaligned on panned/zoomed images

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


Open Blobs, zoom in, try to draw with paintbrush. Or just size the window smaller then move the scroll bars to pan. Or zoom in, then back out again. The drawn line no longer tracks the mouse properly.

Change History

comment:1 Changed 8 years ago by dscho

It seems as if the offset is applied twice. IOW if you scroll down so that the top-most pixels have y-coordinate 20, the y-offset with which things are drawn is 40, not 20.

comment:2 Changed 8 years ago by bdezonia

  • Owner changed from curtis to bdezonia
  • Status changed from new to assigned

comment:3 Changed 8 years ago by bdezonia

Note even the probe coordinates are wrong when zoom level changes. The origin of the image reports numbers much larger than 0,0 and zoomin in and out does not return to a good known state.

comment:4 Changed 8 years ago by bdezonia

  • Owner changed from bdezonia to curtis

Curtis, I think you and/or Lee had recently changed the definition of window origins and panel origins during the display refactoring. Please take a look at this or comment and assign back.

comment:5 Changed 8 years ago by curtis

  • Blocking 1185 added

comment:6 Changed 8 years ago by curtis

As of 1bb013f3, the drawing tools are 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 #716 for another symptom of the same bug.

comment:7 Changed 8 years ago by curtis

  • Version 2.0.0-beta2 deleted
  • Blocking 1326 added; 1185 removed
  • Milestone changed from imagej-2.0.0-beta3 to imagej-2.0.0-beta4

comment:8 Changed 8 years ago by curtis

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

comment:9 Changed 7 years ago by bdezonia

  • Milestone changed from imagej2-b7-ndim-data to imageJ-2.0.0-TO-REFILE

comment:10 Changed 5 years ago by curtis

  • Status changed from assigned to closed
  • Resolution set to moved
Note: See TracTickets for help on using tickets.