Closed (fixed)
Project:
Drupal core
Version:
7.x-dev
Component:
language system
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
16 Aug 2008 at 22:41 UTC
Updated:
14 Jul 2012 at 23:12 UTC
Jump to comment: Most recent file
Comments
Comment #1
gábor hojtsyWow, this is not implemented yet. OMG. Here is a quick patch to make this work and a screenshot to prove it works. Basically, added class to the weight, to the draggable rows and id to the table. Added the tabledrag control to the page. Then also realized the columns should be reordered, so that the checkbox is not first. Same reordering happened for drag and drop enablement on pages such as the menu overview, so I did the same to get to a familiar experience. It all seem to be work visually. Please review.
Comment #2
dries commentedTested and works. I've committed this to CVS HEAD.
I'm marking this patch "code needs work" instead of "fixed" because we don't seem to explain why you might want to change the order. We didn't explain it before this patch either but it seems like it should. Let's follow-up with a patch that makes this more clear for the end-user.
Comment #3
gábor hojtsyAdded this sentence to the help text:
We should also have a copy of the usual "how to drag and drop" text found on all other drag and drop enabled forms copied over here, so I keep this patch needs work.
Comment #4
meba commentedAdding a "how to drag and drop" text. Marking as code needs review.
Comment #5
gábor hojtsyLooks great to me. Drag and drop text is as standard in other modules.
Comment #6
Freso commentedBtw, does drag'n'drops need tests?
Comment #7
dries commentedCommitted to CVS HEAD. Thanks.
Comment #8
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.