Skip to main content

Search and Top Navigation

#8585 closed bug (fixed)

Opened September 19, 2012 01:17PM UTC

Closed October 09, 2012 05:09PM UTC

Sortable broken in absolutely positioned div with bottom 0 when using jQuery 1.8+

Reported by: gabaum10 Owned by:
Priority: minor Milestone: 1.9.0
Component: ui.sortable Version: 1.8.23
Keywords: Cc:
Blocked by: Blocking:
Description

I am seeing some strange behavior with Jquery 1.8 with Jquery UI 1.8.23 where scrolling is not working on a sortable list. Here is a fiddle example:

http://jsfiddle.net/m765z/3/

As you can see, the list is not scrolling on the drag. If you downgrade to 1.7.2, everything works as desired.

Here is a StackOverflow question that another user opened about this very issue:

http://stackoverflow.com/questions/12361986/jquery-1-8-breaks-sortable-list-scrolling-inside-absolute-positioned-element-wi

Attachments (0)
Change History (9)

Changed September 19, 2012 01:21PM UTC by scottgonzalez comment:1

component: ui.coreui.sortable
status: newopen

Changed September 27, 2012 04:37PM UTC by bleroy comment:2

The demo pages, such as http://jqueryui.com/demos/sortable/#delay-start don't scroll either.

Changed September 30, 2012 09:28AM UTC by andri comment:3

I wouldn't say this issue is that minor. If you've got drop areas only accessible after scrolling, it breaks functionality.

Changed October 08, 2012 06:42PM UTC by dmaz99 comment:4

I agree with Andri. I have exactly that situation where the droppable is generally going to be off the screen and therefore this part of my application is completely unusable.

Changed October 08, 2012 07:05PM UTC by scottgonzalez comment:5

The priority doesn't really change anything. Feel free to submit a pull request if you'd like to see this resolved sooner.

Changed October 09, 2012 02:52PM UTC by dmaz99 comment:6

Fair enough. :)

Turns out this is an issue with the argument list change to $.css() in jQuery 1.8. Looks like you removed the instances of $.curCSS() in master, but for 1.8, without knowing the code as well as you do, I thought it would be safer to redefine $.curCSS() as a function that remaps the arguments.

https://github.com/jquery/jquery-ui/pull/739

Changed October 09, 2012 03:42PM UTC by scottgonzalez comment:7

@dmaz99 Can you confirm whether this is working for you with 1.9.0?

Changed October 09, 2012 04:19PM UTC by dmaz99 comment:8

Replying to [comment:7 scott.gonzalez]:

@dmaz99 Can you confirm whether this is working for you with 1.9.0?

Just tested and it works. Sorry I missed that 1.9.0 was released yesterday. We'll make due with this patch for our next release, but we'll take a look at moving to 1.9 soon.

Thanks for your help.

Changed October 09, 2012 05:09PM UTC by scottgonzalez comment:9

resolution: → fixed
status: openclosed
summary: Sortable broken in absolutely positioned div with bottom 0Sortable broken in absolutely positioned div with bottom 0 when using jQuery 1.8+

Thanks for confirming. I'm going to just close this as fixed in 1.9.0 then and not worry about fixing 1-8-stable. It seems like anyone using jQuery 1.8+ should be able to easily upgrade to jQuery UI 1.9.x.