Search and Top Navigation
#3556 closed bug (fixed)
Opened November 08, 2008 11:54AM UTC
Closed January 08, 2009 02:04PM UTC
Draggable connectToSortable revert bug
| Reported by: | Flat Stanley | Owned by: | paul |
|---|---|---|---|
| Priority: | minor | Milestone: | 1.7 |
| Component: | ui.draggable | Version: | 1.5.2 |
| Keywords: | connectToSortable revert | Cc: | |
| Blocked by: | Blocking: |
Description
When we set a revert: "invalid" to a draggable connected to a sortable, we would expect that dropping the draggable onto the sortable would be considered a "valid" drop. However, I find that the item reverts to its original position and then adds itself to the sortable list. So, it seems to be considered an "invalid" drop.
Attachments (1)
Change History (6)
Changed December 23, 2008 04:28PM UTC by comment:1
Changed January 02, 2009 04:37AM UTC by comment:2
| milestone: | TBD → 1.6 |
|---|---|
| resolution: | → fixed |
| status: | new → closed |
Closing as fixed based on Marc's comment. Feel free to re-open this ticket if you continue to have problems with 1.6rc3+.
Changed January 03, 2009 05:09AM UTC by comment:3
tested with 1.6rc4
The bug is still valid unfortunately. Maybe I can make a testcase tomorrow.
- Trac tells me I don't have permission to reopen. *sigh*
Changed January 03, 2009 01:35PM UTC by comment:4
| resolution: | fixed |
|---|---|
| status: | closed → reopened |
Changed January 07, 2009 10:02PM UTC by comment:5
I'm attaching a patch that fixes this problem. I used the trunk version.
Changed January 08, 2009 02:04PM UTC by comment:6
| resolution: | → fixed |
|---|---|
| status: | reopened → closed |
Was a bit more difficult to fix than the provided patch (the provided patch simply kills revert if connectToSortable is used). Fixed in r1557.
I found this to be fixed in trunk. Should be released in 1.6rc3. (I'm not on the team.)