Bug 840407: Remove the "align" and "valign" attributes

r/a=justdave

2 files changed, 14 insertions, 15 deletions

diff --git a/template/en/default/admin/workflow/comment.html.tmpl b/template/en/default/admin/workflow/comment.html.tmpl
index 94218c8fa..19762d154 100644
--- a/template/en/default/admin/workflow/comment.html.tmpl
+++ b/template/en/default/admin/workflow/comment.html.tmpl
@@ -15,9 +15,9 @@
 <!--
   function toggle_cell(cell) {
     if (cell.checked)
-      cell.parentNode.className = "checkbox-cell checked";
+      cell.parentNode.className = "checked";
     else
-      cell.parentNode.className = "checkbox-cell";
+      cell.parentNode.className = "";
   }
 //-->
 </script>
@@ -28,7 +28,7 @@
 </p>
 
 <form id="workflow_form" method="POST" action="editworkflow.cgi">
-<table>
+<table id="workflow">
   <tr>
     <th colspan="2">&nbsp;</th>
     <th colspan="[% statuses.size FILTER html %]" class="title">To</th>
@@ -38,7 +38,7 @@
     <th rowspan="[% statuses.size + 2 FILTER html %]" class="title">From</th>
     <th>&nbsp;</th>
     [% FOREACH status = statuses %]
-      <th class="col-header[% status.is_open ? " open-status" : " closed-status" %]">
+      <th class="[% status.is_open ? "open-status" : "closed-status" %] center">
         [% display_value("bug_status", status.name) FILTER html %]
       </th>
     [% END %]
@@ -54,22 +54,21 @@
 
       [% FOREACH new_status = statuses %]
         [% IF workflow.${status.id}.${new_status.id}.defined %]
-          <td align="center" class="checkbox-cell
-              [% " checked" IF workflow.${status.id}.${new_status.id} %]"
+          <td class="[% "checked" IF workflow.${status.id}.${new_status.id} %]"
               title="From [% status.name FILTER html %] to [% new_status.name FILTER html %]">
             <input type="checkbox" name="c_[% status.id %]_[% new_status.id %]"
                    id="c_[% status.id %]_[% new_status.id %]" onclick="toggle_cell(this)"
               [% " checked='checked'" IF workflow.${status.id}.${new_status.id} %]>
           </td>
         [% ELSE %]
-          <td class="checkbox-cell forbidden">&nbsp;</td>
+          <td class="forbidden">&nbsp;</td>
         [% END %]
       [% END %]
     </tr>
   [% END %]
 </table>
 
-<p align="center">
+<p class="center">
   <input type="hidden" name="action" value="update_comment">
   <input type="hidden" name="token" value="[% token FILTER html %]">
   <input type="submit" id="update_comment" value="Commit Changes"> -
diff --git a/template/en/default/admin/workflow/edit.html.tmpl b/template/en/default/admin/workflow/edit.html.tmpl
index 4ae613ef5..1904b487e 100644
--- a/template/en/default/admin/workflow/edit.html.tmpl
+++ b/template/en/default/admin/workflow/edit.html.tmpl
@@ -15,9 +15,9 @@
 <!--
   function toggle_cell(cell) {
     if (cell.checked)
-      cell.parentNode.className = "checkbox-cell checked";
+      cell.parentNode.className = "checked";
     else
-      cell.parentNode.className = "checkbox-cell";
+      cell.parentNode.className = "";
   }
 //-->
 </script>
@@ -33,7 +33,7 @@
 </p>
 
 <form id="workflow_form" method="POST" action="editworkflow.cgi">
-<table>
+<table id="workflow">
   <tr>
     <th colspan="2">&nbsp;</th>
     <th colspan="[% statuses.size FILTER html %]" class="title">To</th>
@@ -43,7 +43,7 @@
     <th rowspan="[% statuses.size + 2 FILTER html %]" class="title">From</th>
     <th>&nbsp;</th>
     [% FOREACH status = statuses %]
-      <th class="col-header[% status.is_open ? " open-status" : " closed-status" %]">
+      <th class="[% status.is_open ? "open-status" : "closed-status" %] center">
         [% display_value("bug_status", status.name) FILTER html %]
       </th>
     [% END %]
@@ -61,7 +61,7 @@
         [% IF status.id != new_status.id %]
           [% checked = workflow.${status.id}.${new_status.id}.defined ? 1 : 0 %]
           [% mandatory = (status.id && new_status.name == Param("duplicate_or_move_bug_status")) ? 1 : 0 %]
-          <td align="center" class="checkbox-cell[% " checked" IF checked || mandatory %]"
+          <td class="[% "checked" IF checked || mandatory %]"
               title="From [% status.name FILTER html %] to [% new_status.name FILTER html %]">
             <input type="checkbox" name="w_[% status.id %]_[% new_status.id %]"
                    id="w_[% status.id %]_[% new_status.id %]" onclick="toggle_cell(this)"
@@ -69,7 +69,7 @@
                    [%+ "disabled='disabled'" IF mandatory %]>
           </td>
         [% ELSE %]
-          <td class="checkbox-cell forbidden">&nbsp;</td>
+          <td class="forbidden">&nbsp;</td>
         [% END %]
       [% END %]
     </tr>
@@ -87,7 +87,7 @@
   page and editing the <i>duplicate_or_move_bug_status</i> parameter.
 </p>
 
-<p align="center">
+<p class="center">
   <input type="hidden" name="action" value="update">
   <input type="hidden" name="token" value="[% token FILTER html %]">
   <input type="submit" id="update_workflow" value="Commit Changes"> -