Skip to main content

Search and Top Navigation

#2685 closed bug (wontfix)

Opened April 14, 2008 02:48PM UTC

Closed April 17, 2008 01:08PM UTC

Last modified May 24, 2008 03:39AM UTC

ui.droppable.js & overflow

Reported by: Maximus Owned by: paul
Priority: minor Milestone:
Component: ui.core Version: 1.2.3
Keywords: ui droppable overflow dimensions Cc:
Blocked by: Blocking:
Description

I noticed a problem in ui.droppable.js @ line 137. In the event that the drop target has the attribute of overlow: visible, the existing stored values are _NOT_ valid, but are assumed to be so. When this scenario occurs, you cannot do a drag & drop reorder of sub-content past the original dimensions boundaries. For my needs, I needed to deal with Vertical overflow (as most will), and made alterations as follows.

The simple fix is this:

--- t = droppable.offset.top, b = t + droppable.item.proportions.height;

+++ t = droppable.offset.top, b = t + $(droppable.item.element).outerHeight();

Attachments (0)
Change History (2)

Changed April 17, 2008 01:08PM UTC by paul comment:1

resolution: → wontfix
status: newclosed

I won't "fix" this due to performance reasons. It makes everything /way/ slower. Anyway, there already is an option that does exactly what you want: call draggable with the option refreshPositions set to true, and it will also update the size on every mousemove. Thanks anyway!

Changed May 24, 2008 03:39AM UTC by comment:2

milestone: 1.2.4

Milestone 1.2.4 deleted