Description Randy Hudson 2006-05-19 10:27:35 EDT

For some reason our editor allows a zoom range of 5% to 400%. This might be a little broad, but changing from 5% to 400% takes 32 seconds to animate. The application stops responding in many cases until the end of the 32 seconds.

The problematic code seems to be:
private AnimationModel calculateAnimationModel(double zoom) {
  double dmod = zoom / getZoom();
  int steps = (int)Math.round(dmod > 0 ? dmod : 1/dmod);
  int duration = Math.max(DURATION_INCREMENT, steps * DURATION_INCREMENT);

Animations should never take longer than 250 milliseconds.

In the above code, 0 should probably be 1. Also, I would consider the following change:

double dmod = Math.sqrt(zoom/getZoom());