#14963 closed feature (fixed)
Restore $.ui.intersect()
Reported by: | mohs8421 | Owned by: | Scott González |
---|---|---|---|
Priority: | blocker | Milestone: | 1.12.2 |
Component: | ui.droppable | Version: | 1.12.0-rc.2 |
Keywords: | Cc: | ||
Blocked by: | Blocking: |
Description
https://bugs.jqueryui.com/ticket/10534
removing this makes it impossible to implement a custom intersect method. For example if the draggable is larger than the droppable the intersect function does only work if it is centrally over the droppable, not "intersecting". I suggest to accept a function in toleranceMode and use that if provided.
Change History (5)
comment:1 Changed 7 years ago by
Milestone: | none → 1.12.0 |
---|---|
Priority: | minor → blocker |
Status: | new → open |
Summary: | Replace $.ui.intersect instead of removing it → Restore $.ui.intersect() |
Type: | bug → feature |
comment:2 Changed 7 years ago by
Owner: | set to Scott González |
---|---|
Resolution: | → fixed |
Status: | open → closed |
In f928f25:
comment:3 Changed 7 years ago by
It appears from my testing that the re-exposed $.ui.intersect is not actually used internally by ui.droppable. So changing that variable does not actually allow you to change the behavior of droppable.
Test case at http://codepen.io/outofambit/pen/zKEwdR
I tried both v1.12.0 and v1.12.1 of jQuery UI
comment:4 Changed 7 years ago by
Milestone: | 1.12.0 → 1.12.2 |
---|
Ultimately, this will be handled by #4737, but that won't be available until the rewrite lands. You can see how this will work by looking at the current progress of the rewrite: https://github.com/jquery/jquery-ui/blob/c15481eeb1806938ce2b3adfa1866e5a8a78fa31/ui/droppable.js#L164
Since that won't help you for 1.12, we'll just restore
$.ui.intersect()
until the droppable rewrite can be released.