Opened 5 years ago

Last modified 4 years ago

#10534 reopened feature

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:

Description

Currently, droppable defines a function $.ui.intersect 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.

Change History (7)

comment:1 Changed 5 years ago by mikesherov

Status: new → open

comment:2 Changed 5 years ago by mikesherov

Owner: set to mikesherov
Status: open → assigned

comment:3 Changed 5 years ago by mikesherov

Milestone: none → 1.12.0
Priority: minor → blocker

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

Summary: Droppable: move $.ui.intersect behind the closure → Droppable: Remove $.ui.intersect()

comment:5 Changed 5 years ago by Mike Sherov

Resolution: → fixed
Status: assigned → closed

Droppable: Remove $.ui.intersect()

Fixes #10534

Changeset: 20a636844961a1fb2de7a892ab28890091822e6a

comment:6 Changed 4 years ago by Scott González

Milestone: 1.12.0 → none
Resolution: fixed
Status: closed → reopened

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

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

In f928f25:

Droppable: Re-expose $.ui.intersect()

Fixes #14963
Ref #10534

Note: See TracTickets for help on using tickets.