Opened 9 years ago

Closed 7 years ago

#7351 closed bug (fixed)

Sortable: Dragging sortable item onto linked list and scrolling to that list causes flickering

Reported by: nav Owned by:
Priority: minor Milestone: 1.8.24
Component: ui.sortable Version: 1.8.12
Keywords: Cc:
Blocked by: Blocking:

Description

If you have a scrollable element that contains connected sortable lists (or div's table's etc), when you try to drag from one list to another connected list and at the same time scroll this new list into view it causes a flickering between items on that 'active' sort list. This appears to be an issue in the latest FF, latest Chrome and IE8 - possibly more.

I found that this isn't an issue in version 1.8.9 but is in 1.8.12 as illustrated here:

http://jsbin.com/iruqa4/17/edit

Swap JQuery UI source files in code view to see the issue and fixed solution.

Change History (4)

comment:1 Changed 9 years ago by nav

I've noticed now that in the older version when you slightly edge an item over to a connected list it doesn't fire the 'over' event (and move the list over) even though the item is now on that list. This isn't an issue with the latest 1.8.12 version however.

comment:2 Changed 7 years ago by Scott González

Milestone: 1.9.02.0.0

comment:3 Changed 7 years ago by petersendidit

Status: newopen
Summary: Dragging sortable item onto linked list and scrolling to that list causes flickeringSortable: Dragging sortable item onto linked list and scrolling to that list causes flickering

Still is a problem in 1.9.1 http://jsfiddle.net/petersendidit/YSeut/

comment:4 Changed 7 years ago by tj.vantoll

Milestone: 2.0.01.8.24
Resolution: fixed
Status: openclosed

This is working fine in master http://jsfiddle.net/tj_vantoll/cyQHK/4/.

The problem was apparently fixed in 1.8.24 because I see the issue in 1.8.23 (http://jsfiddle.net/tj_vantoll/yZQ8T/) but when I bump the version up to 1.8.24 it goes away. Not sure which of these changes would've affected this http://jqueryui.com/changelog/1.8.24/.

The test case was temporarily re-broke because of #8792 (which is why petersendidit commented as such above).

Note: See TracTickets for help on using tickets.