[OpenLayers-Trac] Re: [OpenLayers] #2936: Handler.Drag: Does not move until the cursor, and does not finish properly.

OpenLayers trac-20090302 at openlayers.org
Sun Jan 23 09:36:00 EST 2011


#2936: Handler.Drag: Does not move until the cursor, and does not finish properly.
--------------------------+-------------------------------------------------
 Reporter:  jorix         |       Owner:  erilem      
     Type:  bug           |      Status:  new         
 Priority:  blocker       |   Milestone:  2.11 Release
Component:  Handler.Drag  |     Version:  2.10        
 Keywords:                |       State:  Review      
--------------------------+-------------------------------------------------

Comment(by erilem):

 The issue can be easily reproduced on
 http://www.openlayers.org/dev/examples/fullScreen.html: move the map w/o
 immediately releasing the mouse after moving, you'll see "jumps" on
 mouseup.

 And actually the jump somehow makes sense, as it positions the map under
 the mouse at the same place where the mousedown occurred. We get this jump
 effect when {{{interval}}} is non-zero, i.e. when we purposefully miss
 mousemove events.

 [attachment:patch-2936-A0.diff] fixes the "jump on mouseup" issue. With
 the patch the jump occurs earlier, when the "delayed drag" timeout
 expires. So the patch does improve things in my opinion - better user
 experience.

 Tests continue to pass.

 @jorix, my patch is just a subset of yours. Small patches with restricted
 focuses are much easier to review. Tell me if you think some of your tests
 could be added to my patch. Thanks for reporting the issue and the patch.

-- 
Ticket URL: <http://trac.openlayers.org/ticket/2936#comment:5>
OpenLayers <http://openlayers.org/>
A free AJAX map viewer


More information about the Trac mailing list