Ticket #982 (closed bug: fixed)

Opened 6 years ago

Last modified 6 years ago

Drag fires click (FF)

Reported by: crschmidt Owned by:
Priority: major Milestone: 2.5 Release
Component: Handler.Drag Version: 2.5 RC1
Keywords: Cc:
State:

Description

Dragging the map fires click. See click.html for exapmle. Doesn't happen in 2.4

Attachments

drag.patch Download (2.7 KB) - added by tschaub 6 years ago.
simplify drag handler

Change History

  Changed 6 years ago by crschmidt

r3891 causedd this regression

Changed 6 years ago by tschaub

simplify drag handler

follow-up: ↓ 3   Changed 6 years ago by tschaub

  • keywords review added

All tests pass in IE/FF. Will gladly commit with review.

in reply to: ↑ 2   Changed 6 years ago by elemoine

Replying to tschaub:

All tests pass in IE/FF. Will gladly commit with review.

ACK'ed.

  Changed 6 years ago by crschmidt

  • keywords commit added; review removed

Agreed. Please commit (but leave open and mark pullup)

  Changed 6 years ago by tschaub

(In [4354]) Simplify click handling in the drag handler - this makes the sequence down, move, up a complete drag (see #982).

  Changed 6 years ago by tschaub

  • keywords pullup added; commit removed

  Changed 6 years ago by crschmidt

  • keywords pullup removed
  • status changed from new to closed
  • resolution set to fixed

(In [4390]) Pullup changes to trunk for RC2. Includes drag-fires click changes (Closes #982), comment/documentation/requires changes (Closes #983, #993, #988), Fixing post support in proxy.cgi (Closes #991), baseLayer zoom level change (Closes #990), typo in Layer.Image.setURL (Closes #985), and a fix or the Layer.Google bug caused by Google's changing internals (#994). RC2, here we come.

Note: See TracTickets for help on using tickets.