#6009 closed bug (fixed)

Opened September 01, 2010 12:53PM UTC

Closed November 21, 2011 09:17PM UTC

Last modified November 21, 2011 09:19PM UTC

Upper droppable should receive draggable

Reported by: maxbarbul Owned by:
Priority: minor Milestone: 1.8.17
Component: ui.droppable Version: 1.8.4
Keywords: Cc:
Blocked by: Blocking:
Description

HTML file attached. Try to run it and read following comments.

http://www.seogoogle.ru/sites/dnd_ticket.html

Try to drag draggable over two droppables.

There are positions when they both are hovered though only one should be.

I think that only one droppable should be hovered because only one could receive this draggable so only one should show that it's waiting.

When you drop draggable into intersection of droppable1 and droppable2 (when they are hovered) only droppable1 will receive draggable though droppable2 is on top.

And droppable2 will be keeping hoverClass untill next dragging.

Attachments (2)
  • dnd_ticket.html (2.7 KB) - added by maxbarbul September 01, 2010 12:55PM UTC.

    incorrect drop when second created droppable has greater z-index then first and it hovers first one

  • jquery.ui.draggable.dropintoupper.js (1.0 KB) - added by maxbarbul September 01, 2010 12:59PM UTC.

    This is how I solved problem. I use plaugin for draggable. But it doesn't help with hoverClass

Change History (5)

Changed October 13, 2011 08:55PM UTC by bikeshedder comment:1

I just stumbled across the same issue and prepared a test case for this. This is probably related to #6085.

http://jsfiddle.net/PHQHb/5/

Changed October 22, 2011 11:02AM UTC by bikeshedder comment:2

Annoying issue, simple fix: https://github.com/jquery/jquery-ui/pull/499

Changed November 21, 2011 09:17PM UTC by Michael P. Jung comment:3

resolution: → fixed
status: new → closed

Droppable: Make sure._drop is called for all relevant droppables. Fixes #6009 - Upper droppable should receive draggable. Fixes #6085 - Parent droppable takes precedence over dynamically created child droppable.

Changeset: fdc332ed5fb7f7e688151d5cae9b450569d73da5

Changed November 21, 2011 09:18PM UTC by Michael P. Jung comment:4

Droppable: Make sure._drop is called for all relevant droppables. Fixes #6009 - Upper droppable should receive draggable. Fixes #6085 - Parent droppable takes precedence over dynamically created child droppable.

(cherry picked from commit fdc332ed5fb7f7e688151d5cae9b450569d73da5)

Changeset: f7b28df28809bca05623b7394919d15919334afb

Changed November 21, 2011 09:19PM UTC by scottgonzalez comment:5

keywords: droppable, draggable, accept
milestone: TBD → 1.8.17