This is a proof of concept made to port the profile module's profile page into a form. The benefits of this are that you'd be able to change the way the Profile module's elements appear to the user without having to manually edit each item's Category or Weight. And of course, it would look much cleaner.
The end result will be similar to the Blocks administration page, where you can drag items region to region. In this case, however, you'll be able to move items from category to category. If you take a look at the attached screenshot, you'll get an idea of where it's at right now. The screenshot is of where the patch is right now.
profile_admin_overview_submit to save the dataprofile_admin_overview| Comment | File | Size | Author |
|---|---|---|---|
| #14 | profile_drag_and_drop.patch | 13.88 KB | quicksketch |
| #12 | profile_drag_and_drop.patch | 13.81 KB | robloach |
| #11 | profile_drag_and_drop.patch | 13.24 KB | robloach |
| #9 | profile_drag_and_drop.patch | 13.24 KB | robloach |
| #6 | profile_drag_and_drop.patch | 13.25 KB | quicksketch |
Comments
Comment #1
catchNo time to review anything now, but subscribing for later. Nice one taking this up!
Comment #2
quicksketchAdding to the list of things to look at. Rock on!
Comment #3
robloachMade some progress with it. It now includes select boxes for the weight and categories. Clicking on Save configuration saves the fields correctly. Needs much more work, help is greatly appreciated.
Comment #4
robloachJust an update of the things left on this:
Comment #5
robloachHere's an updated version with better management of the form fields. Still requires separation of the categories and fixings with the drag and drop. Any help appreciated.
Comment #6
quicksketchSo maybe we have time for another drag and drop patch?
I took a stab at this tonight and as Rob has indicated, this is about the same complexity as the blocks page. Fortunately, since we've already figured out the blocks page, this was easy to duplicate. This patch takes the same approach, adding a special javascript file to fill in the additional functionality needed to display a message in an empty region. In this case, if the region is left empty it will be removed automatically upon form submission.
Thanks Rob for getting this rolling. The biggest caveat I encountered with the initial patch was the HTML ID "profile-admin-overview" was already in use for the form on the page. Since the table was also using this ID, none of the javascript was working because it was finding the form instead of the table on the page. I changed the table ID to "profile-fields" and a lot more things started working.
I think this patch is fully working and well documented. Ready for review.
Comment #7
Stefan Nagtegaal commentedThis is a nice example of how the ordering of node-edit/add fields should look like.. :-)
Comment #8
catchWith no categories/fields enabled I get this:
With one field, I get drag and drop enabled - this should probably be disabled like the vocabulary page follow-up.
With more than one field, and with multiple categories. ooooooh this is nice.
Minor nitpick - in IE7 (and IE6, which still has that shadowing thing going on) the drag and drop icon is a pixel or two too high, and when you mouseover, the yellow bleeds over the blue border of the category. I didn't notice this with the other patches so it's either specific to this form, or just more noticeable due to the extra category row.
Pretty much there, but marking as needs work for the E_ALL, not too fussed about the IE7 pixel but should be trivial either way.
Comment #9
robloachAhhh, that makes sense. I was wondering why it wasn't working. Your patch was giving some hunk failures, so here's one updated to HEAD. Also was missing the first declaration of
$categories, which resulted in some PHP warnings, and had a call in the javascript toconsole, which I doubt we need anymore. These issues are resolved in the attached patch. Would be good to get a couple more reviewers on it.Thanks a lot for helping out with this, it rocks!
Comment #10
robloachSet to CNW for catch's note on disabling the drag and drop when there's only one field. The other issues you brought up are resolved though (excluding the IE bugs).
Comment #11
robloachOops, malformed profile.js. This fixes it.
Comment #12
robloachThis patch adds the drag and drop only when there is more than one field. I used the vocabulary as a guideline. Ready for review.
Comment #13
catchThat fixes both the notices and the drag and drop on one item bugs. In my view this is ready. Could do with testing on a platform other than XP just in case.
Comment #14
quicksketchI found 2 small problems corrected in this patch:
- Weight field on the overview page had delta = 10 while on the field form it only had 5. Updating fields could cause an unintentionally changed weight. I set both to 10.
- The profile.js file was still being added when tabledrag.js was not, causing a javascript error on load when only one field existed.
I tested in Opera 9, Safari 3, and Firefox 2 will complete success.
Comment #15
blackdog commentedTested on FF 2 and Safari 3 on Mac, works like a charm!
Comment #16
quicksketchMaybe a bump to get it on radar.
Comment #17
BioALIEN commentedI can confirm its working too. Nice work guys :)
Comment #18
gábor hojtsyJust had some time this morning to test this patch. The changes look sensible and the functionality looks great. Having this weirdness with categories (that we need to create categories when adding fields, and that we remove categories when removing the last field) is the sign of how denormalized this category feature is implemented. Unfortunately there was not much interest in normalizing this in D6, so we go with this architecture for now.
Thanks for keeping up this work. Although again, I don't think this is critical - I monitor the RTBC queue, so no need to bump to critical to get my attention.
Comment #19
keith.smith commentedNote this [related] bug at http://drupal.org/node/198231
Comment #20
(not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.