Bug 920681: Remove the cellspacing and cellpadding attributes from tables

r/a=justdave

3 files changed, 36 insertions, 37 deletions

diff --git a/template/en/default/admin/fieldvalues/confirm-delete.html.tmpl b/template/en/default/admin/fieldvalues/confirm-delete.html.tmpl
index 39c20e325..579d07cdc 100644
--- a/template/en/default/admin/fieldvalues/confirm-delete.html.tmpl
+++ b/template/en/default/admin/fieldvalues/confirm-delete.html.tmpl
@@ -22,34 +22,34 @@
   style_urls = ['skins/standard/admin.css']
 %]
 
-<table border="1" cellpadding="4" cellspacing="0">
-<tr class="column_header">
-  <th valign="top" class="left">Field</th>
-  <th valign="top" class="left">Value</th>
-</tr>
-<tr>
-  <td valign="top">Field Name:</td>
-  <td valign="top">[% field.description FILTER html %]</td>
-</tr>
-<tr>
-  <td valign="top">Field Value:</td>
-  <td valign="top">[% value.name FILTER html %]</td>
-</tr>
-<tr>
-  <td valign="top">[% terms.Bugs %]:</td>
-  <td valign="top">
-[% IF value.bug_count %]
-  <a title="List of [% terms.bugs %] where '
-            [%- field.description FILTER html %]' is '
-            [%- value.name FILTER html %]'"
-     href="buglist.cgi?[% field.name FILTER uri %]=
-           [%- value.name FILTER uri %]">
-   [%- value.bug_count FILTER html %]</a>
-[% ELSE %]
-  None
-[% END %]
-  </td>
-</tr>
+<table id="admin_table">
+  <tr class="column_header">
+    <th>Field</th>
+    <th>Value</th>
+  </tr>
+  <tr>
+    <td>Field Name</td>
+    <td>[% field.description FILTER html %]</td>
+  </tr>
+  <tr>
+    <td>Field Value</td>
+    <td>[% value.name FILTER html %]</td>
+  </tr>
+  <tr>
+    <td>[% terms.Bugs %]</td>
+    <td>
+      [% IF value.bug_count %]
+        <a title="List of [% terms.bugs %] where '
+                  [%- field.description FILTER html %]' is '
+                  [%- value.name FILTER html %]'"
+           href="buglist.cgi?[% field.name FILTER uri %]=
+                 [%- value.name FILTER uri %]">
+          [%- value.bug_count FILTER html %]</a>
+      [% ELSE %]
+        None
+      [% END %]
+    </td>
+  </tr>
 </table>
 
 <h2>Confirmation</h2>
@@ -149,6 +149,6 @@
 
 [% PROCESS admin/fieldvalues/footer.html.tmpl
   no_edit_link = 1
- +%]
+%]
 
 [% PROCESS global/footer.html.tmpl %] 
diff --git a/template/en/default/admin/fieldvalues/create.html.tmpl b/template/en/default/admin/fieldvalues/create.html.tmpl
index 900f4d901..4d07d5b0e 100644
--- a/template/en/default/admin/fieldvalues/create.html.tmpl
+++ b/template/en/default/admin/fieldvalues/create.html.tmpl
@@ -15,6 +15,7 @@
 [% END %]
 [% PROCESS global/header.html.tmpl
   title = title
+  style_urls = ['skins/standard/admin.css']
 %]
 
 <p>
@@ -23,7 +24,7 @@
 </p>
 
 <form method="post" action="editvalues.cgi">
-  <table border="0" cellpadding="4" cellspacing="0">
+  <table id="admin_table_edit">
     <tr>
       <th><label for="value">Value:</label></th>
       <td>
@@ -74,7 +75,7 @@
               </option>
             [% END %]
           </select>
-          <small>(Leave unset to have this value always appear.)</small>
+          <span class="bz_info">(Leave unset to have this value always appear.)</span>
         </td>
       </tr>
     [% END %]
diff --git a/template/en/default/admin/fieldvalues/edit.html.tmpl b/template/en/default/admin/fieldvalues/edit.html.tmpl
index d076e3071..dbe222590 100644
--- a/template/en/default/admin/fieldvalues/edit.html.tmpl
+++ b/template/en/default/admin/fieldvalues/edit.html.tmpl
@@ -17,15 +17,13 @@
 [% END %]
 [% PROCESS global/header.html.tmpl
   title = title
+  style_urls = ['skins/standard/admin.css']
 %]
 
 <form method="post" action="editvalues.cgi">
-  <table border="0" cellpadding="4" cellspacing="0">
-
+  <table id="admin_table_edit">
     <tr>
-      <th valign="top">
-        <label for="value_new">Field Value:</label>
-      </th>
+      <th><label for="value_new">Field Value:</label></th>
       <td>
         [% IF value.is_static %]
           <input type="hidden" name="value_new" id="value_new"
@@ -71,7 +69,7 @@
               </option>
             [% END %]
           </select>
-          <small>(Leave unset to have this value always appear.)</small>
+          <span class="bz_info">(Leave unset to have this value always appear.)</span>
         </td>
       </tr>
     [% END %]