Bug 920681: Remove the cellspacing and cellpadding attributes from tables

r/a=justdave

1 files changed, 81 insertions, 87 deletions

diff --git a/template/en/default/admin/components/confirm-delete.html.tmpl b/template/en/default/admin/components/confirm-delete.html.tmpl
index 6644acca8..9018dcda2 100644
--- a/template/en/default/admin/components/confirm-delete.html.tmpl
+++ b/template/en/default/admin/components/confirm-delete.html.tmpl
@@ -22,101 +22,96 @@ from '[% product.name FILTER html %]' product
   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">Component:</td>
-  <td valign="top">[% comp.name FILTER html %]</td>
-</tr>
-<tr>
-  <td valign="top">Component Description:</td>
-  <td valign="top">[% comp.description FILTER html_light %]</td>
-</tr>
-<tr>
-  <td valign="top">Default assignee:</td>
-  <td valign="top">[% comp.default_assignee.login FILTER html %]</td>
-  
-[% IF Param('useqacontact') %]
-</tr>
-<tr>
-  <td valign="top">Default QA contact:</td>
-  <td valign="top">[% comp.default_qa_contact.login FILTER html %]</td>
-[% END %]
-  
-</tr>
-<tr>
-  <td valign="top">Component of Product:</td>
-  <td valign="top">[% product.name FILTER html %]</td>
-
-[% IF product.description %]
-</tr>
-<tr>
-  <td valign="top">Product Description:</td>
-  <td valign="top">[% product.description FILTER html_light %]</td>
-[% END %]
+<table id="admin_table">
+  <tr class="column_header">
+    <th>Field</th>
+    <th>Value</th>
+  </tr>
+  <tr>
+    <td>Component</td>
+    <td>[% comp.name FILTER html %]</td>
+  </tr>
+  <tr>
+    <td>Component Description</td>
+    <td>[% comp.description FILTER html_light %]</td>
+  </tr>
+  <tr>
+    <td>Default assignee</td>
+    <td>[% comp.default_assignee.login FILTER html %]</td>
+  </tr>
+
+  [% IF Param('useqacontact') %]
+    <tr>
+      <td>Default QA contact</td>
+      <td>[% comp.default_qa_contact.login FILTER html %]</td>
+    </tr>
+  [% END %]
 
-[% IF Param('usetargetmilestone') %]
-</tr>
-<tr>
-  <td valign="top">Product Milestone URL:</td>
-  <td valign="top">
-    <a href="[% product.milestone_url FILTER html %]">
-      [% product.milestone_url FILTER html %]
-    </a>
-  </td>
-[% END %]
+  <tr>
+    <td>Component of Product</td>
+    <td>[% product.name FILTER html %]</td>
+  </tr>
 
-</tr>
-<tr>
-  <TD VALIGN="top">Open for [% terms.bugs %]:</TD>
-  <TD VALIGN="top">[% IF product.is_active && comp.isactive %]Yes[% ELSE %]No[% END %]</td>
-</tr>
-<tr>
-  <td valign="top">[% terms.Bugs %]:</td>
-  <td valign="top">
-[% IF comp.bug_count %]
-  <a title="List of [% terms.bugs %] for component '[% comp.name FILTER html %]'"
-     href="buglist.cgi?component=[% comp.name FILTER uri %]&amp;product=
-          [%- product.name FILTER uri %]">[% comp.bug_count %]</a>
-[% ELSE %]
-  None
-[% END %]
-  </td>
-</tr>
+  [% IF product.description %]
+    <tr>
+      <td>Product Description</td>
+      <td>[% product.description FILTER html_light %]</td>
+    </tr>
+  [% END %]
+
+  [% IF Param('usetargetmilestone') %]
+    <tr>
+      <td>Product Milestone URL</td>
+      <td>
+        <a href="[% product.milestone_url FILTER html %]">
+          [% product.milestone_url FILTER html %]</a>
+      </td>
+    </tr>
+  [% END %]
+
+  <tr>
+    <td>Open for [% terms.bugs %]</td>
+    <td>[% IF product.is_active && comp.isactive %]Yes[% ELSE %]No[% END %]</td>
+  </tr>
+  <tr>
+    <td>[% terms.Bugs %]</td>
+    <td>
+      [% IF comp.bug_count %]
+        <a title="List of [% terms.bugs %] for component '[% comp.name FILTER html %]'"
+           href="buglist.cgi?component=[% comp.name FILTER uri %]&amp;product=
+                 [%- product.name FILTER uri %]">[% comp.bug_count %]</a>
+      [% ELSE %]
+        None
+      [% END %]
+    </td>
+  </tr>
 </table>
 
 <h2>Confirmation</h2>
-  
+
 [% IF comp.bug_count %]
 
   [% IF !Param("allowbugdeletion") %]
-  <p class="confirmation">
-    Sorry, there
-
-    [% IF comp.bug_count > 1 %] 
-      are [% comp.bug_count %] [%+ terms.bugs %] 
-    [% ELSE %]
-      is [% comp.bug_count %] [%+ terms.bug %] 
-    [% END %]
-
-    outstanding for this component.  You must reassign 
-
-    [% IF comp.bug_count > 1 %]
-       those [% terms.bugs %] 
-    [% ELSE %]
-       that [% terms.bug %]
-    [% END %]
+    <p class="confirmation">
+      Sorry, there
+      [% IF comp.bug_count > 1 %]
+        are [% comp.bug_count %] [%+ terms.bugs %]
+      [% ELSE %]
+        is 1 [% terms.bug %]
+      [% END %]
+      outstanding for this component. You must reassign
 
-    to another component before you can delete this one.
-  </p>
+      [% IF comp.bug_count > 1 %]
+        those [% terms.bugs %]
+      [% ELSE %]
+        that [% terms.bug %]
+      [% END %]
+      to another component before you can delete this one.
+    </p>
   [% ELSE %]
-
-    <table class="confirmation"><tr><td>
-
-      There [% IF comp.bug_count > 1 %] 
+    <p class="confirmation">
+      There
+      [% IF comp.bug_count > 1 %]
         are [% comp.bug_count %] [%+ terms.bugs %] 
       [% ELSE %]
         is 1 [% terms.bug %]
@@ -124,8 +119,7 @@ from '[% product.name FILTER html %]' product
       entered for this component!  When you delete this
       component, <b>ALL</b> stored [% terms.bugs %] and
       their history will be deleted too.
-      </td></tr></table>
-
+    </p>
   [% END %]
 
 [% END %]