Bug Tracker

Modify

Ticket #2685 (closed bug: wontfix)

Opened 5 years ago

Last modified 5 years ago

ui.droppable.js & overflow

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

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();

Change History

comment:1 Changed 5 years 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!

Please follow the  bug reporting guidlines and use  jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.

View

Add a comment

Modify Ticket

Action
as closed
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.