Skip to main content

Search and Top Navigation

#10534 reopened feature ()

Opened August 11, 2014 12:19PM UTC

Last modified May 05, 2016 03:15AM UTC

Droppable: Remove $.ui.intersect()

Reported by: mikesherov Owned by: mikesherov
Priority: blocker Milestone: none
Component: ui.droppable Version: 1.11.0
Keywords: Cc:
Blocked by: Blocking:

Currently, droppable defines a function

which is only used in droppable itself and is not documented anywhere. I propose we move this method inside the closure. I also just removed a unit test that was testing it directly to rather test the functionality that relies upon it, so this should be fairly straightforward.

Attachments (0)
Change History (7)

Changed August 11, 2014 12:19PM UTC by mikesherov comment:1

status: newopen

Changed August 11, 2014 12:19PM UTC by mikesherov comment:2

owner: → mikesherov
status: openassigned

Changed August 11, 2014 01:47PM UTC by mikesherov comment:3

milestone: none1.12.0
priority: minorblocker

Changed August 20, 2014 12:55PM UTC by scottgonzalez comment:4

summary: Droppable: move $.ui.intersect behind the closureDroppable: Remove $.ui.intersect()

Changed November 05, 2014 11:21PM UTC by Mike Sherov comment:5

resolution: → fixed
status: assignedclosed

Droppable: Remove $.ui.intersect()

Fixes #10534

Changeset: 20a636844961a1fb2de7a892ab28890091822e6a

Changed May 04, 2016 10:59PM UTC by scottgonzalez comment:6

milestone: 1.12.0none
resolution: fixed
status: closedreopened

This was re-exposed in f928f258b7978980a5f101b54f06778d14d219df to fix #14963. We'll hold off on the removal until we can release the droppable rewrite.

Changed May 05, 2016 03:15AM UTC by scottgonzalez comment:7

In [changeset:"f928f258b7978980a5f101b54f06778d14d219df" f928f25]:

#!CommitTicketReference repository="" revision="f928f258b7978980a5f101b54f06778d14d219df"
Droppable: Re-expose 

Fixes #14963
Ref #10534