Handler.Box: Do not clean the box to be deactivate.

[jorix]

This bug is easy to verify using http://openlayers.org/dev/examples/navtoolbar.html:

• Select "ZoomBox"
• Start a drag
• Move the cursor over "Navigation" button.
• Release the mouse and press and "Navigation" button.
• Oops! the rectangle does not disappear.

This bug is related with #3193

[jorix]

Please review.

[ahocevar]

@jorix: can you please also try if configuring the Navigation control with documentDrag: true fixes the issue (without your patch applied)? Thanks.

[jorix]

Replying to ahocevar:

@jorix: can you please also try if configuring the Navigation control with documentDrag: true fixes the issue (without your patch applied)? Thanks.

It does not work, using OpenLayers.Control.Navigation.documentDrag: true also fails.

[ahocevar]

Handler.Box tests fail with this patch applied.

[jorix]

Replying to ahocevar:

Handler.Box tests fail with this patch applied.

Sorry! I thought I had passed tests for this patch, but it was another patch that I tested.

With the new attachment:handler.box-3198.2.patch all tests (controls and handlers) are working properly.

Now tested on Chrome-10 and IE8

[ahocevar]

(In [11729]) removing box when deactivating the handler. p=jorix, r=me (closes #3198)