Posted by senpai on December 1, 2007 at 8:17pm
Let's edit the help text for deleting a group. Right now, it says:
Group posts:
o Do nothing.
o Delete all group posts which don't also belong to another group.
o Move all group posts to the group listed below.
In addition to deleting this group, you may choose how to disposition the posts within it.
Target group:
[a dropdown list shows up here]
If you chose <strong>Move all group posts</strong> above, specify a destination group.I'd like it to look more like:
Group posts:
o Do nothing.
o Delete all group posts which don't also belong to another group.
o Move all group posts to another group.
If you chose to <strong>Move all group posts</strong>, select another group to assign them to:
Target group:
[a dropdown list shows up here]I'm willing to submit a patch for this today, but I'd need to know which file to patch. What version of O.G. are we currently running here?
Comments
js
in addition, i would like to see some js which enables the dropdown only if the 'move' radio button is selected. you can look at the current og.js for inspiration there.
we are running DRUPAL-5 tip which is currently the same as HEAD for the og project.
I posted a first-draft patch
Here's my first draft of a patch to address this concern:
http://drupal.org/node/197957
I also started on a little JS trickery to disable that dropdown menu until the user has selected the correct radio button. Here's a thought. Should we actually hide the entire form element with jQuery upon page load, and then show it if the proper radio button is clicked, or is disabling the dropdown upon page load the 'better' way to improve the UI?
Senpai (my d.o account)
Joel Farris | my 'certified to rock' score
Transparatech
http://transparatech.com
619.717.2805
I chose the show/hide route.
I chose the show/hide route. The patch is attached in the O.G. issue queue's comment #11.
Senpai (my d.o account)
Joel Farris | my 'certified to rock' score
Transparatech
http://transparatech.com
619.717.2805