#202 closed enhancement (fixed)
[patch] Allow drag and drop of table rows
| Reported by: | Owned by: | alex | |
|---|---|---|---|
| Priority: | high | Milestone: | |
| Component: | Core | Version: | 0.2 |
| Keywords: | Cc: | sylvain@… | |
| Blocked By: | Blocking: |
Description
Here's a patch to allow table rows to be drag'n dropped. This requires adding a surrounding <table> and <tbody> to the <tr>, and setting fixed widths to the <td>s to avoid them being packed.
Attachments (2)
Change History (12)
Changed 14 years ago by
| Attachment: | table_dnd.diff added |
|---|
comment:1 Changed 14 years ago by
Also tries to fix a memory leak by ensuring dragClone is nulled, but the leak is still there.
comment:2 Changed 13 years ago by
| Owner: | changed from anonymous to alex |
|---|---|
| Status: | new → assigned |
| Version: | → 0.1 |
Hi Sylvain,
I'd love to merge this patch, but I don't have a CLA from you. The Dojo CLA is identical to the Apache CLA, and can be found at http://dojotoolkit.org/icla.txt
Regards.
comment:3 Changed 13 years ago by
After failing to fax the CLA (don't know what end was broken), I sent it by snail mail around dec 20th. Did you get it?
comment:4 Changed 13 years ago by
| Cc: | sylvain@… added |
|---|---|
| Resolution: | → fixed |
| Status: | assigned → closed |
Sorry Sylvain, we're in the middle of moving our office location, and our fax number changed. It has been received.
comment:5 Changed 13 years ago by
| Milestone: | → 0.3release |
|---|---|
| Resolution: | fixed |
| Status: | closed → reopened |
| Version: | 0.1 → 0.2 |
Marked as closed, but the patch didn't made it in SVN...
comment:6 Changed 13 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | reopened → closed |
comment:7 Changed 13 years ago by
| Resolution: | fixed |
|---|---|
| Status: | closed → reopened |
Hi,
I'm trying to use the table drag and drop and can't get it to work. Having seen this patch, I decided to compare it to the HtmlDragAndDrop?.js in both the 0.3 release and the 5/17/2006 nightly build and there is no code that resembles the patch. Do I need to download from subversion?
comment:8 Changed 13 years ago by
| Milestone: | 0.3release → 0.3.1 |
|---|---|
| Status: | reopened → new |
what happened here... did this patch rot on the vine? was this fixed?
comment:9 Changed 13 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
merged in [4245]. Marking fixed.
Patch against r2164