NOTICE! This is a static HTML version of a legacy ImageJ Trac ticket.

The ImageJ project now uses GitHub Issues for issue tracking.

Please file all new issues there.

Ticket #627 (accepted defect)

Opened 2011-06-21T11:01:32-05:00

Last modified 2013-01-22T15:01:00-06:00

Rotating image can cause window size to be wrong

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

Description (last modified by bdezonia)

Using 7feeec6da90f475fea86cb0e43bfca37dcdfb828: If you load a new Image and rotate it by 90 the window size can be wrong. Usually its wrong just the first time you rotate (the window does not resize). But I have had it that after that failed window transformation further rotate 90s are also wrong over and over as the untransformed original window then also rotates by 90 and stays off by 90.

Change History

comment:1 Changed 2011-06-21T13:11:04-05:00 by bdezonia

Seems to happen regularly with Boats

comment:2 Changed 2011-06-21T13:14:30-05:00 by bdezonia

  • Summary changed from Rotateing image can cause window size to be wrong to Rotating image can cause window size to be wrong

comment:3 Changed 2011-06-21T13:14:46-05:00 by bdezonia

  • Description modified

comment:4 Changed 2011-07-29T13:32:18-05:00 by bdezonia

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

comment:5 Changed 2011-07-29T13:33:51-05:00 by bdezonia

Related to #605?

comment:6 Changed 2011-08-18T13:13:38-05:00 by bdezonia

  • Status changed from new to closed
  • Resolution set to fixed

Fixed in 64c877fbd4981345c0994d2c3e6016acf8fbfb7a. Had to rework some code that may have been added to avoid some other bug. Commented and dated decision in source in case we rediscover bug.

comment:7 Changed 2011-09-26T14:20:36-05:00 by bdezonia

  • Status changed from closed to reopened
  • Resolution fixed deleted

Do not consider this bug fixed until we have a solution for all three of #605, #627, and #797.

comment:8 Changed 2011-09-26T16:09:48-05:00 by bdezonia

Can still duplicate this bug in alpha 5 and also latest version. There must be a race condition that was never realy fixed. Or switching from ThreadSafeEventService to SwingEventService has somehow caused problems. Investigate.

comment:9 Changed 2011-09-28T15:21:48-05:00 by bdezonia

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

At times I have been able to "fix" this bug with changes to when updates(), redoLayouts(), and invokeLaters() happen. But no longer.

When these "fixes" have worked bug #605 would also go away. I believe they are the same bug. I will be closing #605 while noting that any fix to this bug should test that #605 is also fixed.

In the meantime I have made some code changes to make calls to update() and redoLayout() consistent throughout IJ2. Those commits were: 77667ced3ad5d107f906a72036cd6b3a4600a5ca, 2ea227f8de12310eda084de55f20b16772de94c8, & a9a7f0adcc653f2e7cadc1726671a49ecdc8e6c0.

This ticket will go to a back burner until Curtis' work on eliminating redoLayout() shakes out.

comment:10 Changed 2011-10-24T12:04:24-05:00 by curtis

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

Waiting on ticket #828.

comment:11 Changed 2011-11-01T10:55:52-05:00 by curtis

  • Status changed from assigned to accepted

comment:12 Changed 2012-02-27T10:59:37-06:00 by curtis

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

comment:13 Changed 2012-06-25T12:36:45-05:00 by curtis

  • Blocking 1242 added

comment:14 Changed 2012-07-12T10:26:18-05:00 by curtis

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

comment:15 Changed 2012-07-12T11:00:32-05:00 by curtis

  • Blocking 1325 added; 1242 removed

comment:16 Changed 2012-09-06T14:06:14-05:00 by curtis

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

comment:17 Changed 2012-12-12T09:31:01-06:00 by bdezonia

Still a problem on 12-12-12

comment:18 Changed 2013-01-22T14:55:27-06:00 by bdezonia

  • Blocking 1629 added; 1358 removed

comment:19 Changed 2013-01-22T15:01:00-06:00 by bdezonia

  • Milestone changed from imagej2-b7-ndim-data to imagej2-unscheduled