diff options
| author | Mika Havela <mika.havela@gmail.com> | 2009-10-06 08:53:11 +0200 |
|---|---|---|
| committer | Mika Havela <mika.havela@gmail.com> | 2009-10-06 08:53:11 +0200 |
| commit | 74a01633d06ba8987e9f4c914550fcffdf38d750 (patch) | |
| tree | c965fa91fa2acf77306533521711153afa0ac62c | |
| parent | 60495849ac3e8297ec093c7a8d9dd2579ae730ff (diff) | |
| download | acf-core-74a01633d06ba8987e9f4c914550fcffdf38d750.tar.bz2 acf-core-74a01633d06ba8987e9f4c914550fcffdf38d750.tar.xz | |
Displaying existing roles by using <DL><DT><DD> instead of using a table.
This small fix causes this view to behave (follow same ideas) as other view-files.
You notice the result when using other skins than 'alps'.
| -rw-r--r-- | app/acf-util/roles-viewroles-html.lsp | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/app/acf-util/roles-viewroles-html.lsp b/app/acf-util/roles-viewroles-html.lsp index 9575ac6..8e01a00 100644 --- a/app/acf-util/roles-viewroles-html.lsp +++ b/app/acf-util/roles-viewroles-html.lsp @@ -15,19 +15,17 @@ </form> <H2>Existing roles</H2> +<DL> <% if view.value.default_roles then %> - <table> <% for x,role in pairs(view.value.default_roles.value) do %> - <tr><td><dt><img src='/skins/static/tango/16x16/categories/applications-system.png' height='16' width='16'> <%= html.html_escape(role) %></dt> + <dt><img src='/skins/static/tango/16x16/categories/applications-system.png' height='16' width='16'> <%= html.html_escape(role) %></dt> <dd> [<a href='viewroleperms?role=<%= html.html_escape(role) %>'>View this role</a>] [<a href='editrole?role=<%= html.html_escape(role) %>&redir=<%= html.html_escape(page_info.orig_action) %>'>Edit this role</a>] - </dd></td></tr> + </dd> <% end %> - </table> <% end %> <% if view.value.defined_roles then %> - <dl> <% table.sort(view.value.defined_roles.value) %> <% for x,role in pairs(view.value.defined_roles.value) do %> <dt><img src='/skins/static/tango/16x16/apps/system-users.png' height='16' width='16'> <%= html.html_escape(role) %></dt> @@ -39,3 +37,4 @@ <% end %> </dl> <% end %> +</DL> |