Auto scrolling (i.e. scroll:true) with Sortable/Draggable hasn't worked since jquery-ui 1.8.24 powered by jQuery 1.8.2

Hi dcarrith,

Thanks for taking the time to contribute to the jQuery UI project. Could you please provide a reduced test case that shows the issue that you're experiencing. You can use this as a starting point - http://jsfiddle.net/tj_vantoll/wjxe4/.

If you're having these issues in IE this might be related to #9057.

Thanks!

Ran into the same problem.

Used your starting point and created a simple version that breaks in > IE9: http://jsfiddle.net/vtYGg/2/

Try dragging any elements down and you will see that it never scrolls in IE. Works in all other browsers I tested.

I did some more testing this morning after I received the email notification when @onereason commented on the issue. I noticed a difference in how "scrolled" was being set. So, I modified the section I had originally commented out (see above in original ticket write-up). This actually fixes the scrolling issue, but it causes another weird issue where the sorted item gets overlayed on top of the item below where it's dropped (i.e. the items don't get shifted up or down appropriately). So, I'm wondering why in the if block, the scrolled variable is getting set to an integer. But, in the else block it gets set to an object. So, to "fix" it, I changed it so the scrolled variable would get set to an object or false, to be in line with the else block (which I know works fine).

		//Do scrolling
		if(this.options.scroll) {

			if(this.scrollParent[0] !== document && this.scrollParent[0].tagName !== "HTML") {

				if((this.overflowOffset.top + this.scrollParent[0].offsetHeight) - event.pageY < o.scrollSensitivity) {
					
					// The original line results in scrolled being set to an integer 
					//this.scrollParent[0].scrollTop = scrolled = this.scrollParent[0].scrollTop + o.scrollSpeed;
					
					// This modified line results in scrolled being set to an object
					scrolled = this.scrollParent[0].scrollTop( this.scrollParent[0].scrollTop + o.scrollSpeed );
				
				} else if(event.pageY - this.overflowOffset.top < o.scrollSensitivity) {
					
					// The original line results in scrolled being set to an integer 
					//this.scrollParent[0].scrollTop = scrolled = this.scrollParent[0].scrollTop - o.scrollSpeed;

					// This modified line results in scrolled being set to an object
					scrolled = this.scrollParent[0].scrollTop( this.scrollParent[0].scrollTop - o.scrollSpeed );
				}

				if((this.overflowOffset.left + this.scrollParent[0].offsetWidth) - event.pageX < o.scrollSensitivity) {
					this.scrollParent[0].scrollLeft = scrolled = this.scrollParent[0].scrollLeft + o.scrollSpeed;
				} else if(event.pageX - this.overflowOffset.left < o.scrollSensitivity) {
					this.scrollParent[0].scrollLeft = scrolled = this.scrollParent[0].scrollLeft - o.scrollSpeed;
				}

			} else {

				if(event.pageY - $(document).scrollTop() < o.scrollSensitivity) {
					scrolled = $(document).scrollTop($(document).scrollTop() - o.scrollSpeed);
				} else if($(window).height() - (event.pageY - $(document).scrollTop()) < o.scrollSensitivity) {
					scrolled = $(document).scrollTop($(document).scrollTop() + o.scrollSpeed);
				}

				if(event.pageX - $(document).scrollLeft() < o.scrollSensitivity) {
					scrolled = $(document).scrollLeft($(document).scrollLeft() - o.scrollSpeed);
				} else if($(window).width() - (event.pageX - $(document).scrollLeft()) < o.scrollSensitivity) {
					scrolled = $(document).scrollLeft($(document).scrollLeft() + o.scrollSpeed);
				}

			}

			if(scrolled !== false && $.ui.ddmanager && !o.dropBehaviour) {
				$.ui.ddmanager.prepareOffsets(this, event);
			}
		}

@dcarrith: Would you be interested in sending a pull request with your proposed fix?

Replying to [comment:4 scott.gonzalez]:

@dcarrith: Would you be interested in sending a pull request with your proposed fix?

Sure, but with this fix in place, there is still an issue where the items aren't being shifted up or down. I can try to fix that, but won't be able to look at that until tonight. I would also want to make sure it still works in a simple jsfiddle.

Update: scratch that. My "fix" wasn't much of a fix. I noticed the js error caused by calling scrollTop on an HTML DIV element, which is not a function. I'll look at it some more tonight. It's kind of strange how that caused it to work though.

Okay, this seems to fix it. Notice the additional check to make sure this.scrollParent[0].offsetHeight is less than $(window).height() in the 2nd line of code:

		//Do scrolling
		if(this.options.scroll) {

			if( ( this.scrollParent[0] !== document && this.scrollParent[0].tagName !== "HTML" ) && ( this.scrollParent[0].offsetHeight < $(window).height() ) ) {

				if((this.overflowOffset.top + this.scrollParent[0].offsetHeight) - event.pageY < o.scrollSensitivity) {
					this.scrollParent[0].scrollTop = scrolled = this.scrollParent[0].scrollTop + o.scrollSpeed;
				} else if(event.pageY - this.overflowOffset.top < o.scrollSensitivity) {
					this.scrollParent[0].scrollTop = scrolled = this.scrollParent[0].scrollTop - o.scrollSpeed;
				}

				if((this.overflowOffset.left + this.scrollParent[0].offsetWidth) - event.pageX < o.scrollSensitivity) {
					this.scrollParent[0].scrollLeft = scrolled = this.scrollParent[0].scrollLeft + o.scrollSpeed;
				} else if(event.pageX - this.overflowOffset.left < o.scrollSensitivity) {
					this.scrollParent[0].scrollLeft = scrolled = this.scrollParent[0].scrollLeft - o.scrollSpeed;
				}

			} else {

				if(event.pageY - $(document).scrollTop() < o.scrollSensitivity) {
					scrolled = $(document).scrollTop($(document).scrollTop() - o.scrollSpeed);
				} else if($(window).height() - (event.pageY - $(document).scrollTop()) < o.scrollSensitivity) {
					scrolled = $(document).scrollTop($(document).scrollTop() + o.scrollSpeed);
				}

				if(event.pageX - $(document).scrollLeft() < o.scrollSensitivity) {
					scrolled = $(document).scrollLeft($(document).scrollLeft() - o.scrollSpeed);
				} else if($(window).width() - (event.pageX - $(document).scrollLeft()) < o.scrollSensitivity) {
					scrolled = $(document).scrollLeft($(document).scrollLeft() + o.scrollSpeed);
				}

			}

			if(scrolled !== false && $.ui.ddmanager && !o.dropBehaviour) {
				$.ui.ddmanager.prepareOffsets(this, event);
			}
		}

Duplicate of #7546.This is a duplicate of #7546 so I'm closing this issue in favor of the existing one. @dcarrith, feel free to continue with your pull request but I believe the core problem is in

scrollParent