Opened 4 years ago

Closed 4 years ago

#14594 closed bug (wontfix)

delay requires additional mouse event to work properly

Reported by: davideanderson Owned by:
Priority: minor Milestone: none
Component: ui.sortable Version: 1.11.3
Keywords: Cc:
Blocked by: Blocking:

Description

the _mouseDown function checks to see if the mouseDelayMet is true, and if not, it sets up a timeout which then sets mouseDelayMet to be true.

However, all that timeout does is set the value to true. The event handling has already completed when it fires, and nothing happens until another mouse event happens, such as a mouse move. Which means, _mouseStart does not get executed if you simply click and hold.

Here is a jsFiddle. Simply click and hold on an item, and nothing happens. After 1 second, the outline should turn red. However, if you delay for a second, then move the mouse, then the outline changes to red, thus the additional mouse move was required to have the delay work.

Change History (1)

comment:1 Changed 4 years ago by Scott González

Resolution: wontfix
Status: newclosed

This is not something we're going to change. The delay option is being removed.

Note: See TracTickets for help on using tickets.