[OpenLayers-Trac] Re: [OpenLayers] #3428: Modifying point features does not stop at mouseup

[OpenLayers]

#3428: Modifying point features does not stop at mouseup
---------------------------------+------------------------------------------
 Reporter:  ahocevar             |       Owner:  ahocevar    
     Type:  feature              |      Status:  assigned    
 Priority:  minor                |   Milestone:  2.11 Release
Component:  Control.DragFeature  |     Version:  2.11 RC1    
 Keywords:                       |       State:  Review      
---------------------------------+------------------------------------------
Changes (by jorix):

 * cc: xavier.mamano@… (added)


Comment:

 Replying to [comment:3 ahocevar]:
 > The attached patch makes the workflow in the ticket description work as
 expected again. All tests still pass (but I don't know how to write a unit
 test assertion that shows the problem that the patch fixes). Thanks for
 any review.

 Hi ahocevar,

 I investigated about this issue.

 There is another problem, try:
  * Starting and stopping drag of a point (or vertex of a line or polygon).
  * mousedown and mouseup (without moving the mouse)
  * mousemove, oops! point moves

 [attachment:openlayers-3428.patch openlayers-3428.patch] does not solve
 this second problem. But [attachment:openlayers-3428.patch
 openlayers-3428.patch] adds a behavior that may be interesting to drag a
 point without prior selection.

 And I think there is another way to fix the bug without using
 {{{setTimeout}}}. r11845 added the methods {{{clickFeature}}} and
 {{{clickoutFeature}}} but only for the management of the touch events. So
 it may condition its use only for this case and this fixes both
 problems.[[BR]]
 See [attachment:3428.B1.patch 3428.B1.patch]

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