Ticket #2531908 (closed defect)

Reporter


pqr
Opened: 03/18/12
Last modified: 12/12/12
Status: closed
Type: defect
Resolution: fixed

Owner


Dav Glass
Target Release: 3.7.0
Priority: P4 (low)
Summary: Sortable utility - can't drop on empty container space
Description:

Sortable utility: http://yuilibrary.com/yui/docs/sortable/

Problem:
I found that using sortable utility I cant drop item on empty container space.
Example: http://jsfiddle.net/TR5Xv/

This behavor of Sortable causes bad user experience.

Related forum topic: http://yuilibrary.com/forum/viewtopic.php?f=92&t=9628

Type: defect Observed in Version: 3.4.1
Component: Sortable Severity: S3 (normal)
Assigned To: Dav Glass Target Release: 3.7.0
Location: Library Code Priority: P4 (low)
Tags: Sortable Relates To:
Browsers: Firefox - Latest
URL:
Test Information:

Change History

Thomas S. Sha

YUI Contributor

Posted: 03/18/12
  • component changed from None to Sortable
  • owner changed from Thomas S. Sha to Dav Glass
  • priority changed to P3 (normal)
  • status changed from new to assigned

Dav Glass

Posted: 03/19/12
  • location changed to Library Code
  • milestone changed to 3.6.0
  • status changed from assigned to accepted

Dav Glass

Posted: 05/11/12
  • estimated changed from 0 to 0.25
  • priority changed from P3 (normal) to P4 (low)
  • remaining changed from 0 to 0.25
  • sprint changed to sprint 2

Dav Glass

Posted: 05/15/12
  • completed changed from 0 to 0.25
  • remaining changed from 0.25 to 0

Dav Glass

Posted: 05/15/12
  • resolution changed to fixed
  • status changed from accepted to checkedin

Dav Glass

Posted: 05/15/12

Fixes #2531908 - Removed limitation on dropping with only one item
View Commit: 26a21b3d3b6e68589d5de1044bdfd970d64e0770

Jenny Donnelly

YUI Developer

Posted: 08/1/12
  • status changed from checkedin to closed

Shipped in 3.6.0. Marking closed/fixed.

Arnaud Didry

YUI Contributor

Posted: 08/23/12
  • milestone changed from 3.6.0
  • resolution changed from fixed
  • status changed from closed to reopened

Hi,

Last commit broke Y.Sortable as you can see in the examples : http://yuilibrary.com/yui/docs/sortable/simple-sortable.html (By the way this example is probably wrong, the sortable container should be the ul node instead of the parent div#demo node).

Here is a possible fix https://github.com/yui/yui3/pull/228

I updated @pqr example with the patch http://jsfiddle.net/TR5Xv/3/

Dav Glass

Posted: 08/27/12
  • milestone changed to FUTURE
  • status changed from reopened to accepted

Arnaud Didry

YUI Contributor

Posted: 09/8/12
  • resolution changed to fixed
  • status changed from accepted to checkedin

Fixes #2531908 - Improve new nodes handling

Commit 26a21b3d3b6e68589d5de1044bdfd970d64e0770 caused dragged items
to be appended to the container while dragging over sortable nodes margins.

Nodes only need to be appended on the "drop:enter" event and if they aren't
already present in the container.
View Commit: 86e473b5f0e1a9ff2688d397fd61113f6a660dae

Arnaud Didry

YUI Contributor

Posted: 09/10/12

Fixes #2531908 - Improve new nodes handling

Commit 26a21b3d3b6e68589d5de1044bdfd970d64e0770 caused dragged items
to be appended to the container while dragging over sortable nodes margins.

Nodes only need to be appended on the "drop:enter" event and if they aren't
already present in the container.
View Commit: 86e473b5f0e1a9ff2688d397fd61113f6a660dae

Dav Glass

Posted: 09/10/12

Jenny Donnelly

YUI Developer

Posted: 12/12/12
  • milestone changed from FUTURE to 3.7.0
  • status changed from checkedin to closed