Opened 7 years ago

Closed 7 years ago

#8585 closed bug (fixed)

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

Change History (9)

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

Component: ui.coreui.sortable
Status: newopen

comment:2 in reply to:  description Changed 7 years ago by bleroy

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

comment:3 Changed 7 years ago by andri

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

comment:4 Changed 7 years ago by dmaz99

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.

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

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

comment:6 Changed 7 years ago by dmaz99

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

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

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

comment:8 in reply to:  7 Changed 7 years ago by dmaz99

Replying to 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.

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

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.

Note: See TracTickets for help on using tickets.