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/reports/keywords.html.tmpl | 40 +++++++++-----------------
 1 file changed, 14 insertions(+), 26 deletions(-)

(limited to 'template/en/default/reports/keywords.html.tmpl')

diff --git a/template/en/default/reports/keywords.html.tmpl b/template/en/default/reports/keywords.html.tmpl
index b9a2fd795..598979d33 100644
--- a/template/en/default/reports/keywords.html.tmpl
+++ b/template/en/default/reports/keywords.html.tmpl
@@ -12,7 +12,6 @@
   #   name: the name of the keyword
   #   description: keyword description. Can contain some limited HTML code.
   #   bug_count: number of bugs with that keyword
-  # caneditkeywords: boolean. True if this user can edit keywords
  %]
 
 [% PROCESS global/header.html.tmpl
@@ -20,27 +19,19 @@
   style_urls = ['skins/standard/admin.css']
 %]
 
-[% FOREACH keyword = keywords %]
-  [% IF loop.index % 50 == 0 %]
-    [% IF loop.index != 0 %]
-       </table>
-    [% END %]
-
-    <table border="1" cellpadding="4" cellspacing="0">
-      <tr class="column_header">
-        <th>Name</th>
-        <th>Description</th>
-        <th>Open [% terms.Bugs %]</th>
-        <th>Total [% terms.Bugs %]</th>
-      </tr>
-  [% END %]
+<table id="admin_table">
+  <tr class="column_header">
+    <th>Name</th>
+    <th>Description</th>
+    <th>Open [% terms.Bugs %]</th>
+    <th>Total [% terms.Bugs %]</th>
+  </tr>
 
+  [% FOREACH keyword = keywords %]
   <tr id="[% keyword.name FILTER html %]">
-    <th>
-        [% keyword.name FILTER html %]
-    </th>
+    <td>[% keyword.name FILTER html %]</td>
     <td>[% keyword.description FILTER html_light %]</td>
-    <td align="center">
+    <td class="center">
       [% IF keyword.bug_count > 0 %]
         <a href="buglist.cgi?keywords=[% keyword.name FILTER uri %]&amp;resolution=---">
           Search</a>
@@ -48,7 +39,7 @@
         none
       [% END %]
     </td>
-    <td align="right">
+    <td class="right">
       [% IF keyword.bug_count > 0 %]
         <a href="buglist.cgi?keywords=[% keyword.name FILTER uri %]">
           [% keyword.bug_count %]</a>
@@ -57,13 +48,10 @@
       [% END %]
     </td>
   </tr>
-[% END %]
-
-[% IF keywords.size > 0 %]
-  </table>
-[% END %]
+  [% END %]
+</table>
 
-[% IF caneditkeywords %]
+[% IF user.in_group("editkeywords") %]
   <p>
     <a href="editkeywords.cgi">Edit keywords</a>.
   </p>
-- 
cgit v1.2.3-24-g4f1b