quick HTML cleanup

1 files changed, 78 insertions, 56 deletions

diff --git a/root/admin/account_modify.tt b/root/admin/account_modify.tt
index 6a1ecc8..8acd4b4 100644
--- a/root/admin/account_modify.tt
+++ b/root/admin/account_modify.tt
@@ -1,66 +1,88 @@
 <a href="[% c.uri_for('/admin/password') %]/[% uid %]">Reset password</a>
 <a href="[% c.uri_for('/admin/account_group') %]/[% uid %]">Groups</a>
 
-<form method=post>
-<input type='hidden' name='operation' value='replace'>
-<table border=0>
-<tr><th>Attribute</th><th>Value</th></tr>
-[% FOREACH attr IN values %]
-  <tr>
-    <td>[% attr.name %]</td>
-    <td>[% FOREACH val IN attr.values %][% IF attr.editable %]<input type=hidden name="[% attr.name %]_old" value="[% val %]"><input name="[% attr.name %]_new" value="[% val %]">[% ELSE %][% val %]<br/>[% END %]
-[% IF attr.addable AND attr.editable %]<a href="[% c.uri_for('/admin/account_edit') %]/add/[% attr.name %]">Add</a>[% END %]
-[% IF attr.removable AND attr.editable %]<a href="[% c.uri_for('/admin/account_modifydel') %]/[% uid %]/[% attr.name %]/[% val %]">Delete</a>[% END %]
-    [% END %]</td>
-  </tr>
-[% END %]
-<tr><td colspan=2 align=center><input type='Submit' value='Update'></td></tr>
-</table>
+<form method="post" action="">
+    <input type='hidden' name='operation' value='replace' />
+    <table border=0>
+        <tr>
+            <th>Attribute</th>
+            <th>Value</th>
+        </tr>
+        [% FOREACH attr IN values %]
+        <tr>
+            <td>[% attr.name %]</td>
+            <td>
+                [% FOREACH val IN attr.values %]
+                    [% IF attr.editable %]
+                        <input type="hidden" name="[% attr.name %]_old" value="[% val %]" />
+                        <input name="[% attr.name %]_new" value="[% val %]" />
+                    [% ELSE %]
+                        [% val %]
+                        <br/>
+                    [% END %]
+                    [% IF attr.addable AND attr.editable %]
+                        <a href="[% c.uri_for('/admin/account_edit') %]/add/[% attr.name %]">Add</a>
+                    [% END %]
+                    [% IF attr.removable AND attr.editable %]
+                        <a href="[% c.uri_for('/admin/account_modifydel') %]/[% uid %]/[% attr.name %]/[% val %]">Delete</a>
+                    [% END %]
+                [% END %]
+            </td>
+        </tr>
+        [% END %]
+        <tr>
+            <td colspan=2 align=center>
+                <input type='Submit' value='Update'>
+            </td>
+        </tr>
+    </table>
 </form>
 
-<table border=0>
-<tr>
-<td>
-<form method=post action="[% c.uri_for('/admin/account_modify') %]/[% uid %]">
-<input type='hidden' name='operation' value='add'>
- Add attribute 
- <select name='attribute'>[% FOREACH attr IN may %]
-  <option value="[% attr %]">[% attr %]</option>[% END %]
- </select>
- with value
-<input name='value'>
- <input type=submit value='Add'>
+<hr />
+
+<form method="post" action="[% c.uri_for('/admin/account_modify') %]/[% uid %]">
+    <input type="hidden" name="operation" value="add" />
+
+    <p>
+    Add attribute 
+    <select name='attribute'>
+        [% FOREACH attr IN may %]
+        <option value="[% attr %]">[% attr %]</option>
+        [% END %]
+    </select>
+    with value
+    <input name="value" value="" />
+    <input type="submit" value="Add" />
+    </p>
 </form>
-</td>
-</tr>
+
+<hr />
+
 [% IF groups %]
-<tr>
-<td>
-Promote user to posixAccount with primary group: 
-<form method=post action="[% c.uri_for('/admin/account_promote') %]">
-<input type='hidden' name='dn' value='[% dn %]'>
-<select name='gid'>
-[% FOREACH group IN groups %]
-<option value='[% group.gidNumber %]'>[% group.name %]</option>
-[% END %]
-</select>
-<input type=submit value='Promote'>
+<p>Promote user to posixAccount with primary group: </p>
+
+<form method="post" action="[% c.uri_for('/admin/account_promote') %]">
+    <input type="hidden" name="dn" value="[% dn %]" />
+    <select name="gid">
+        [% FOREACH group IN groups %]
+            <option value="[% group.gidNumber %]">[% group.name %]</option>
+        [% END %]
+    </select>
+    <input type="submit" value="Promote" />
 </form>
-</td>
-</tr>
-[% END %]
-<tr>
-<td>
-<form method=post action="[% c.uri_for('/admin/account_addoc') %]">
-<input type='hidden' name='dn' value='[% dn %]'>
-<input type='hidden' name='uid' value='[% uid %]'>
-<select name='objectclass'>
-[% FOREACH oc IN offer_ocs %]
-<option value='[% oc %]'>[% oc %]</option>
 [% END %]
-</select>
-<input type='submit' value='Add ObjectClass'>
+
+<hr />
+
+<form method="post" action="[% c.uri_for('/admin/account_addoc') %]">
+    <input type='hidden' name='dn' value='[% dn %]'>
+    <input type='hidden' name='uid' value='[% uid %]'>
+    <select name='objectclass'>
+        [% FOREACH oc IN offer_ocs %]
+            <option value='[% oc %]'>[% oc %]</option>
+        [% END %]
+    </select>
+    <input type="submit" value="Add ObjectClass" />
 </form>
-</td>
-</tr>
-</table>
+
+<hr />
\ No newline at end of file