- Usability improvement: replaced many selection boxes by radio buttons.

  Patch by Stefan.

- Usability improvement: introduced a form_group() function and updated
  the node and comment module to take advantage of it.  Patch #149 by
  Eric Farris.

4 files changed, 18 insertions, 12 deletions

diff --git a/modules/comment.module b/modules/comment.module
index 4f81d029f..8e5a2e7e0 100644
--- a/modules/comment.module
+++ b/modules/comment.module
@@ -1627,7 +1627,7 @@ function comment_nodeapi(&$node, $op, $arg = 0) {
         $output = form_radio(t("Disabled"), "comment", 0, ($selected == 0));
         $output .= form_radio(t("Read only"), "comment", 1, ($selected == 1));
         $output .= form_radio(t("Read/write"), "comment", 2, ($selected == 2));
-        return form_item(t("Allow user comments"), $output);
+        return form_group(t("User comments"), $output);
       }
       break;
     case "validate":
diff --git a/modules/comment/comment.module b/modules/comment/comment.module
index 4f81d029f..8e5a2e7e0 100644
--- a/modules/comment/comment.module
+++ b/modules/comment/comment.module
@@ -1627,7 +1627,7 @@ function comment_nodeapi(&$node, $op, $arg = 0) {
         $output = form_radio(t("Disabled"), "comment", 0, ($selected == 0));
         $output .= form_radio(t("Read only"), "comment", 1, ($selected == 1));
         $output .= form_radio(t("Read/write"), "comment", 2, ($selected == 2));
-        return form_item(t("Allow user comments"), $output);
+        return form_group(t("User comments"), $output);
       }
       break;
     case "validate":
diff --git a/modules/node.module b/modules/node.module
index e611697ea..b040a7aac 100644
--- a/modules/node.module
+++ b/modules/node.module
@@ -1127,10 +1127,13 @@ function node_form($edit, $error = NULL) {
 
   if (user_access("administer nodes")) {
     $output .= "<div class=\"admin\">";
+
+    $author = form_textfield(t("Authored by"), "name", $edit->name, 20, 60, $error["name"]);
+    $author .= form_textfield(t("Authored on"), "date", $edit->date, 20, 25, $error["date"]);
+
     $output .= "<div class=\"authored\">";
-    $output .= form_textfield(t("Authored by"), "name", $edit->name, 20, 60, $error["name"]);
-    $output .= form_textfield(t("Authored on"), "date", $edit->date, 20, 25, $error["date"]);
-    $output .= "</div>";
+    $output .= form_group(t("Authoring information"), $author);
+    $output .= "</div>\n";
 
     $options .= form_checkbox(t("Published"), "status", 1, isset($edit->status) ? $edit->status : variable_get("node_status_$edit->type", 1));
     $options .= form_checkbox(t("In moderation queue"), "moderate", 1, isset($edit->moderate) ? $edit->moderate : variable_get("node_moderate_$edit->type", 0));
@@ -1139,8 +1142,8 @@ function node_form($edit, $error = NULL) {
     $options .= form_checkbox(t("Create new revision"), "revision", 1, isset($edit->revision) ? $edit->revision : variable_get("node_revision_$edit->type", 0));
 
     $output .= "<div class=\"options\">";
-    $output .= form_item(t("Options"), $options);
-    $output .= "</div>";
+    $output .= form_group("Options", $options);
+    $output .= "</div>\n";
 
     $extras .= implode("</div><div class=\"extra\">", node_invoke_nodeapi($edit, "form admin"));
     $output .= $extras ? "<div class=\"extra\">$extras</div></div>" : "</div>";
diff --git a/modules/node/node.module b/modules/node/node.module
index e611697ea..b040a7aac 100644
--- a/modules/node/node.module
+++ b/modules/node/node.module
@@ -1127,10 +1127,13 @@ function node_form($edit, $error = NULL) {
 
   if (user_access("administer nodes")) {
     $output .= "<div class=\"admin\">";
+
+    $author = form_textfield(t("Authored by"), "name", $edit->name, 20, 60, $error["name"]);
+    $author .= form_textfield(t("Authored on"), "date", $edit->date, 20, 25, $error["date"]);
+
     $output .= "<div class=\"authored\">";
-    $output .= form_textfield(t("Authored by"), "name", $edit->name, 20, 60, $error["name"]);
-    $output .= form_textfield(t("Authored on"), "date", $edit->date, 20, 25, $error["date"]);
-    $output .= "</div>";
+    $output .= form_group(t("Authoring information"), $author);
+    $output .= "</div>\n";
 
     $options .= form_checkbox(t("Published"), "status", 1, isset($edit->status) ? $edit->status : variable_get("node_status_$edit->type", 1));
     $options .= form_checkbox(t("In moderation queue"), "moderate", 1, isset($edit->moderate) ? $edit->moderate : variable_get("node_moderate_$edit->type", 0));
@@ -1139,8 +1142,8 @@ function node_form($edit, $error = NULL) {
     $options .= form_checkbox(t("Create new revision"), "revision", 1, isset($edit->revision) ? $edit->revision : variable_get("node_revision_$edit->type", 0));
 
     $output .= "<div class=\"options\">";
-    $output .= form_item(t("Options"), $options);
-    $output .= "</div>";
+    $output .= form_group("Options", $options);
+    $output .= "</div>\n";
 
     $extras .= implode("</div><div class=\"extra\">", node_invoke_nodeapi($edit, "form admin"));
     $output .= $extras ? "<div class=\"extra\">$extras</div></div>" : "</div>";