From 4fa178ae3f0f612cf91b26a2dea498602794be3b Mon Sep 17 00:00:00 2001
From: Frédéric Buclin <LpSolit@gmail.com>
Date: Tue, 31 Dec 2013 13:48:41 +0100
Subject: Bug 920681: Remove the cellspacing and cellpadding attributes from
 tables r/a=justdave

---
 template/en/default/admin/groups/create.html.tmpl | 67 ++++++++++++-----------
 1 file changed, 36 insertions(+), 31 deletions(-)

(limited to 'template/en/default/admin/groups/create.html.tmpl')

diff --git a/template/en/default/admin/groups/create.html.tmpl b/template/en/default/admin/groups/create.html.tmpl
index db65629d0..ca95bab56 100644
--- a/template/en/default/admin/groups/create.html.tmpl
+++ b/template/en/default/admin/groups/create.html.tmpl
@@ -13,30 +13,35 @@
 [% PROCESS global/header.html.tmpl
   title = "Add group"
   subheader = "This page allows you to define a new user group."
+  style_urls = ['skins/standard/admin.css']
   doc_section = "groups.html#create-groups"
 %]
 
 <form method="post" action="editgroups.cgi">
-  <table border="1" cellpadding="4" cellspacing="0"><tr>
-    <th>New Name</th>
-    <th>New Description</th>
-    <th>New User RegExp</th>
-    <th>Use For [% terms.Bugs %]</th>
-  </tr><tr>
-    <td><input size="20" name="name"></td>
-    <td><input size="40" name="desc"></td>
-    <td><input size="30" name="regexp"></td>
-    <td><input type="checkbox" name="isactive" value="1" checked></td>
-  </tr>
-  <tr>
-    <th>Icon URL:</th>
-    <td colspan="3"><input type="text" size="70" maxlength="255" id="icon_url" name="icon_url"></td>
-  </tr>
-  [% Hook.process('field') %]
+  <table id="admin_table_edit">
+    <tr>
+      <th>Group Name:</th>
+      <td><input size="60" maxlength="255" name="name"></td>
+    </tr>
+    <tr>
+      <th>Description:</th>
+      <td><input size="60" name="desc"></td>
+    </tr>
+    <tr>
+      <th>User Regexp:</th>
+      <td><input size="60" name="regexp"></td>
+    </tr>
+    <tr>
+      <th>Icon URL:</th>
+      <td><input type="text" size="60" id="icon_url" name="icon_url"></td>
+    </tr>
+    <tr>
+      <th>Use For [% terms.Bugs %]:</th>
+      <td><input type="checkbox" name="isactive" value="1" checked></td>
+    </tr>
+    [% Hook.process('field') %]
   </table>
 
-  <hr>
-
   <input type="checkbox" id="insertnew" name="insertnew" value="1"
     [% IF Param("makeproductgroups") %] checked[% END %]>
   <label for="insertnew">Insert new group into all existing products.</label>
@@ -46,7 +51,9 @@
   <input type="hidden" name="token" value="[% token FILTER html %]">
 </form>
 
-<p><b>Name</b> is what is used with the B<!-- blah -->ugzilla->user->in_group()
+<hr>
+
+<p><b>Group Name</b> is what is used with the B[%%]ugzilla->user->in_group()
 function in any customized cgi files you write that use a given group.
 It can also be used by people submitting [% terms.bugs %] by email to
 limit [% terms.abug %] to a certain set of groups.</p>
@@ -55,15 +62,6 @@ limit [% terms.abug %] to a certain set of groups.</p>
 to members of the group where they can choose whether
 the [% terms.bug %] will be restricted to others in the same group.</p>
 
-<p>The <b>Use For [% terms.Bugs %]</b> flag determines whether or not the
-group is eligible to be used for [% terms.bugs %]. If you clear this, it will
-no longer be possible for users to add [% terms.bugs %] to this group,
-although [% terms.bugs %] already in the group will remain in the group.
-Doing so is a much less drastic way to stop a group from growing
-than deleting the group would be. <b>Note: If you are creating
-a group, you probably want it to be usable for [% terms.bugs %], in which
-case you should leave this checked.</b></p>
-
 <p><b>User RegExp</b> is optional, and if filled in, will 
 automatically grant membership to this group to anyone with an 
 email address that matches this regular expression.</p>
@@ -75,6 +73,15 @@ email address that matches this regular expression.</p>
   in comments in [% terms.bugs %] besides the name of the author of comments.
 </p>
 
+<p>The <b>Use For [% terms.Bugs %]</b> flag determines whether or not the
+group is eligible to be used for [% terms.bugs %]. If you clear this, it will
+no longer be possible for users to add [% terms.bugs %] to this group,
+although [% terms.bugs %] already in the group will remain in the group.
+Doing so is a much less drastic way to stop a group from growing
+than deleting the group would be. <b>Note: If you are creating
+a group, you probably want it to be usable for [% terms.bugs %], in which
+case you should leave this checked.</b></p>
+
 [% IF Param("makeproductgroups") %]
   <p>By default, the new group will be associated with existing products.
   Unchecking the "Insert new group into all existing products" option will
@@ -87,8 +94,6 @@ email address that matches this regular expression.</p>
   a product.</p>
 [% END %]
 
-<p>Back to the <a href="./">main [% terms.bugs %] page</a>
-
-or to the <a href="editgroups.cgi">group list</a>.
+<p>Back to the <a href="editgroups.cgi">group list</a>.</p>
 
 [% PROCESS global/footer.html.tmpl %] 
-- 
cgit v1.2.3-24-g4f1b