[OpenLayers-Trac] Re: [OpenLayers] #3402: Handler.Drag: Remove setEvent on click when we have touch.

OpenLayers trac-20090302 at openlayers.org
Mon Jul 4 18:02:31 EDT 2011


#3402: Handler.Drag: Remove setEvent on click when we have touch.
--------------------------+-------------------------------------------------
 Reporter:  jorix         |       Owner:              
     Type:  bug           |      Status:  new         
 Priority:  trivial       |   Milestone:  2.12 Release
Component:  Handler.Drag  |     Version:  2.11 RC1    
 Keywords:                |       State:              
--------------------------+-------------------------------------------------
Changes (by jorix):

  * state:  Review =>


Comment:

 Replying to [comment:2 erilem]:

 I saw this problem by working in #3399 (I guess you remember similar
 problem in #3052) and thinking ahocevar's patch #3363. I thought only drag
 ;)

 > Thanks Jorix.
 >
 > Do you think we need this change in 2.11? Does your patch fix a problem
 that effectively affects mobile users? If not I wouldn't apply this patch
 for 2.11.

 The choice is yours. But I think it does no harm at 2.11, and otherwise a
 control may see a mouse event in Handler.evt using touch devices, ups!

 > Also, shouldn't we do the same in {{{Handler.Click}}} and
 {{{Handler.Point}}}? This makes me think that we may want to include this
 logic in the base class.

 Yeah, right. New patch [attachment:3402.1.patch 3402.1.patch] attached
 with a first approach applied in the classes and more tests.

 What to do with {{{"mouseout"}}}?

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


More information about the Trac mailing list