Bug Tracker

Ticket #2685 (closed bug: wontfix)

Opened 8 months ago

Last modified 8 months ago

ui.droppable.js & overflow

Reported by: Maximus Assigned to: paul
Type: bug Priority: minor
Milestone: 1.2.4 Component: ui
Version: 1.2.3 Keywords: ui droppable overflow dimensions
Cc: Needs: Patch

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

Change History

Changed 8 months ago by paul

  • status changed from new to closed
  • resolution set to wontfix

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!

Note: See TracTickets for help on using tickets.