#3173 closed bug (fixed)

Opened August 10, 2008 07:41PM UTC

Closed May 02, 2017 06:03PM UTC

Last modified May 02, 2017 07:10PM UTC

Sortable: connected sortables do not scroll on transfer

Reported by: anonymous Owned by: paul
Priority: minor Milestone: 1.12.2
Component: ui.sortable Version: 1.5.2
Keywords: sortable, connected, scrolling Cc:
Blocked by: Blocking:
Description

the setup:

  • two connected sortable lists
  • lists have fixed height and width and overflow auto
  • external helper element to work around scrolling within the lists on dragging

the bug:

if you move an item within the list it has been bevor the lists scroll correctly. but if you move an item to the other list it wont scroll the list while dragging it around

the code:

<?xml version="1.0" encoding="UTF-8" ?>
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
	<head>
		<meta http-equiv="Content-Type" content="text/html; charset=UTF-8" />
		<title>Insert title here</title>		
		<style type="text/css">
			body { margin:0; padding:0; }
			ul { list-style-type:none; }
			ul.boxed { border:1px solid #000; margin:20px; padding:0; width:200px; height:200px; float:left; overflow:auto; }
			li { width:100%; max-width:200px; padding:5px 0; background-color:#ccc; margin:2px 0; }
			ul#list-3 { width:0;height:0;padding:0;margin:0; }
		</style>
		<script type="text/javascript" src="jquery.js"></script>
		<script type="text/javascript" src="jquery.ui.js" ></script>

	</head>
	<body>
		
		<ul id="list-3"></ul>
		
		<ul id="list-1" class="boxed">
			<li>A</li>
			<li>B</li>
			<li>C</li>

			<li>D</li>
			<li>E</li>
			<li>F</li>
			<li>G</li>
		</ul>
		
		<ul id="list-2" class="boxed">
			<li>H</li>

			<li>I</li>
			<li>J</li>
		</ul>
		
		<script type="text/javascript">			
			function test(event, element) {
				myHelper = $(element).clone();
				myHelper.show();
				$("#list-3").append(myHelper);
				return myHelper;
			}
		
			$("#list-1").sortable({
			    connectWith: ["#list-2"],
			    helper: test
			});
			
			$("#list-2").sortable({ 
			    connectWith: ["#list-1"],
			    helper: test
			});
		</script>
	</body>
</html>
Attachments (0)
Change History (10)

Changed August 16, 2008 08:16PM UTC by Cloudream comment:1

milestone: → TDB

Changed January 02, 2009 02:19PM UTC by scottgonzalez comment:2

milestone: TBD → 1.6

Changed March 08, 2009 02:29PM UTC by rdworth comment:3

milestone: 1.7 → 1.8

Changed September 27, 2010 02:25PM UTC by swiftb3 comment:4

I can verify that this bug also exists when dragging from a draggable connected to a sortable.

Any update as to when this might be fixed, since 1.8 has been here for awhile?

Changed April 25, 2011 08:34PM UTC by jdmarshall comment:5

Also experiencing this issue.

Changed May 18, 2011 01:31AM UTC by skhilko comment:6

Are there any workarounds while it's being fixed?

Changed October 11, 2012 02:54PM UTC by scottgonzalez comment:7

milestone: 1.9.0 → 2.0.0

Changed October 25, 2012 12:35AM UTC by petersendidit comment:8

status: new → open
summary: connected sortables do not scroll on transfer → Sortable: connected sortables do not scroll on transfer

Testcase: http://jsfiddle.net/petersendidit/aCd5X/1/

Changed May 02, 2017 06:03PM UTC by scottgonzalez comment:9

resolution: → fixed
status: open → closed

In [changeset:"c866e455373028a62a0956455a229fef63e91fac" c866e45]:

#!CommitTicketReference repository="" revision="c866e455373028a62a0956455a229fef63e91fac"
Sortable: Fix various scrolling issues

* Created _scroll extension point and migrated scroll code from _mouseDrag
* Cleaned up logic for scrolled
* Fixed appendTo functionality to match documentation
* Remove unnecessary function calls
* Move set-up position functions to appropriate place
* Base scrollParent on placeholder and not helper
* Update scrollParent when switching containers

Fixes #3173
Fixes #15165
Fixes #15166
Fixes #15167
Fixes #15168
Fixes #15169
Fixes #15170
Closes gh-1793

Changed May 02, 2017 07:10PM UTC by scottgonzalez comment:10

milestone: 2.0.0 → 1.12.2