[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