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

r/a=justdave

1 files changed, 24 insertions, 24 deletions

diff --git a/template/en/default/account/prefs/permissions.html.tmpl b/template/en/default/account/prefs/permissions.html.tmpl
index 450765afc..53583f35b 100644
--- a/template/en/default/account/prefs/permissions.html.tmpl
+++ b/template/en/default/account/prefs/permissions.html.tmpl
@@ -17,12 +17,11 @@
   #           other people.
   #%]
 
-<table align="center">
-  <tr>
-    <td>
+    <div id="permissions">
       [% IF has_bits.size %]
-        You have the following permission bits set on your account:
-        <table align="center">
+        <p>You have the following permission bits set on your account:</p>
+
+        <table>
           [% FOREACH bit_description = has_bits %]
             <tr>
               <td>[% bit_description.name FILTER html %]</td>
@@ -34,46 +33,47 @@
         [% FOREACH privs = ["editcomponents", "canconfirm", "editbugs"] %]
           [% SET products = ${"local_$privs"} %]
           [% IF products && products.size %]
-            <br>
             <p>
               You also have local '[% privs FILTER html %]' privileges
               for the following products:
             </p>
-            <p>
+            <ul>
               [% FOREACH product = products %]
-                [% product.name FILTER html %]<br>
+                <li>[% product.name FILTER html %]</li>
               [% END %]
-            </p>
+            </ul>
           [% END %]
         [% END %]
 
       [% ELSE %]
-        There are no permission bits set on your account.
+        <p>There are no permission bits set on your account.</p>
       [% END %]
 
       [% IF user.in_group('editusers') %]
-        <br>
-        You have editusers privileges.  You can turn on and off
-        all permissions for all users.
+        <p>
+          You have editusers privileges. You can turn on and off all
+          permissions for all users.
+        </p>
       [% ELSIF set_bits.size %]
-        <br>
-        And you can turn on or off the following bits for
-        <a href="editusers.cgi">other users</a>:
-          <table align="center">
+        <p>
+          And you can turn on or off the following bits for
+          <a href="editusers.cgi">other users</a>:
+        </p>
+
+        <table>
           [% FOREACH bit_description = set_bits %]
             <tr>
               <td>[% bit_description.name FILTER html %]</td>
               <td>[% bit_description.desc FILTER html_light %]</td>
             </tr>
           [% END %]
-          </table>
+        </table>
       [% END %]
 
       [% IF user.in_group('bz_sudoers') %]
-        <br>
-        You are a member of the <b>bz_sudoers</b> group, so you can 
-        <a href="relogin.cgi?action=prepare-sudo">impersonate someone else</a>.
+        <p>
+          You are a member of the <b>bz_sudoers</b> group, so you can
+          <a href="relogin.cgi?action=prepare-sudo">impersonate someone else</a>.
+        </p>
       [% END %]
-    </td>
-  </tr>
-</table>
+    </div>