OpenLayers OpenLayers

Ticket #468 (closed bug: fixed)

Opened 2 years ago

Last modified 2 years ago

Overview MapLosing Rectangle

Reported by: euzuro Assigned to: tschaub
Priority: minor Milestone: 2.3 Release
Component: Control.OverviewMap Version: 2.3 RC1
Keywords: Cc:
State:

Description

From Bart:

when you have a small rectangle (small compared to the size of the overview map), and you drag the rectangle in a fast way (relatively fast mouse movement), you can loose the rectangle (it gets dropped).

Attachments

quick_drag.patch (2.0 kB) - added by tschaub on 01/11/07 14:13:55.
hang on to that little rectangle
quick_drag.2.patch (2.1 kB) - added by tschaub on 01/19/07 19:17:20.
hold on to that rect (in FF)

Change History

01/11/07 12:10:02 changed by openlayers

I was using 2.3RC1. The problem occurs when the mouse gets out of the rect area, because of the fast movement.

01/11/07 12:14:39 changed by euzuro

  • version changed from 2.2 to 2.3 RC1.

01/11/07 14:13:55 changed by tschaub

  • attachment quick_drag.patch added.

hang on to that little rectangle

01/11/07 14:14:44 changed by tschaub

  • keywords set to review.
  • milestone set to 2.3 Release.

have review, will commit

01/11/07 14:42:47 changed by tschaub

  • keywords deleted.

frrmph (IE needs more attention)

01/11/07 15:07:39 changed by euzuro

tested patch on mozilla still needs some more work

it helps to make a big overview map to test this (400x400)

if i mouse out of the ext rectangle fast then let go of mouse button

then try to move it back in...

01/19/07 19:17:20 changed by tschaub

  • attachment quick_drag.2.patch added.

hold on to that rect (in FF)

01/19/07 19:18:27 changed by tschaub

I'm bored with this - it works in FF now.

01/19/07 22:40:31 changed by tschaub

  • keywords set to review.

If someone approves of the quick_drag.2.patch, I'll commit. I think it is good enough for 2.3. In IE, when you drag the extent rectangle very fast, you still lose it (and start selecting the elements below). I will rewrite this for 2.4 - it probably should be done with a markers layer (that will allow changing of the marker when a box gets too small), and should probably all be done with events on the map (instead of events on the extent rectangle).

Of course, it all works as it should in Firefox with this patch.

01/20/07 11:30:06 changed by crschmidt

  • keywords changed from review to commit.

Nice work. Go ahead to commit: I'll edit the docs to mention the problem for IE.

01/20/07 11:50:47 changed by tschaub

  • keywords changed from commit to pullup.

applied with r2166

01/22/07 22:15:33 changed by crschmidt

  • keywords deleted.
  • status changed from new to closed.
  • resolution set to fixed.