Bug 920681: Remove the cellspacing and cellpadding attributes from tables

r/a=justdave

3 files changed, 41 insertions, 36 deletions

diff --git a/template/en/default/admin/versions/confirm-delete.html.tmpl b/template/en/default/admin/versions/confirm-delete.html.tmpl
index ca5b4ef21..928170f50 100644
--- a/template/en/default/admin/versions/confirm-delete.html.tmpl
+++ b/template/en/default/admin/versions/confirm-delete.html.tmpl
@@ -21,33 +21,33 @@
   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">Version:</td>
-  <td valign="top">[% version.name FILTER html %]</td>
-</tr>
-<tr>
-  <td valign="top">Version of Product:</td>
-  <td valign="top">[% product.name FILTER html %]</td>
-</tr>
-<tr>
-  <td valign="top">[% terms.Bugs %]:</td>
-  <td valign="top">
-[% IF version.bug_count %]
-  <a title="List of [% terms.bugs %] targetted at version '
-           [%- version.name FILTER html %]'"
-     href="buglist.cgi?version=[% version.name FILTER uri %]&amp;product=
-          [%- product.name FILTER uri %]">
-          [%- version.bug_count FILTER none %]</a>
-[% ELSE %]
-  None
-[% END %]
-  </td>
-</tr>
+<table id="admin_table">
+  <tr class="column_header">
+    <th>Field</th>
+    <th>Value</th>
+  </tr>
+  <tr>
+    <td>Version</td>
+    <td>[% version.name FILTER html %]</td>
+  </tr>
+  <tr>
+    <td>Version of Product</td>
+    <td>[% product.name FILTER html %]</td>
+  </tr>
+  <tr>
+    <td>[% terms.Bugs %]</td>
+    <td>
+      [% IF version.bug_count %]
+        <a title="List of [% terms.bugs %] targetted at version '
+                  [%- version.name FILTER html %]'"
+           href="buglist.cgi?version=[% version.name FILTER uri %]&amp;product=
+                 [%- product.name FILTER uri %]">
+                 [%- version.bug_count FILTER none %]</a>
+      [% ELSE %]
+        None
+      [% END %]
+    </td>
+  </tr>
 </table>
 
 <h2>Confirmation</h2>
diff --git a/template/en/default/admin/versions/create.html.tmpl b/template/en/default/admin/versions/create.html.tmpl
index 3544a697b..46bd38c17 100644
--- a/template/en/default/admin/versions/create.html.tmpl
+++ b/template/en/default/admin/versions/create.html.tmpl
@@ -17,10 +17,11 @@
 [% PROCESS global/header.html.tmpl
   title = title
   subheader = subheader
+  style_urls = ['skins/standard/admin.css']
 %]
 
 <form method="post" action="editversions.cgi">
-  <table border="0" cellpadding="4" cellspacing="0">
+  <table id="admin_table_edit">
     <tr>
       <th><label for="version">Version:</label></th>
       <td><input id="version" size="64" maxlength="64" name="version"
diff --git a/template/en/default/admin/versions/edit.html.tmpl b/template/en/default/admin/versions/edit.html.tmpl
index 195a94c26..4ea202c04 100644
--- a/template/en/default/admin/versions/edit.html.tmpl
+++ b/template/en/default/admin/versions/edit.html.tmpl
@@ -17,20 +17,24 @@
                    [%- product.name FILTER html %]'[% END %]
 [% PROCESS global/header.html.tmpl
   title = title
+  style_urls = ['skins/standard/admin.css']
 %]
 
 <form method="post" action="editversions.cgi">
-  <table border="0" cellpadding="4" cellspacing="0">
-
+  <table id="admin_table_edit">
     <tr>
-      <th class="field_label"><label for="version">Version:</label></th>
-      <td><input id="version" size="64" maxlength="64" name="version" value="
-      [%- version.name FILTER html %]"></td>
+      <th><label for="version">Version:</label></th>
+      <td>
+        <input id="version" size="64" maxlength="64" name="version"
+               value="[% version.name FILTER html %]">
+      </td>
     </tr>
     <tr>
-      <th class="field_label"><label for="isactive">Enabled For [% terms.Bugs %]:</label></th>
-      <td><input id="isactive" name="isactive" type="checkbox" value="1"
-                 [% 'checked="checked"' IF version.isactive %]></td>
+      <th><label for="isactive">Enabled For [% terms.Bugs %]:</label></th>
+      <td>
+        <input id="isactive" name="isactive" type="checkbox" value="1"
+                 [% 'checked="checked"' IF version.isactive %]>
+      </td>
     </tr>
   </table>