From 74a01633d06ba8987e9f4c914550fcffdf38d750 Mon Sep 17 00:00:00 2001
From: Mika Havela <mika.havela@gmail.com>
Date: Tue, 6 Oct 2009 08:53:11 +0200
Subject: 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'.

---
 app/acf-util/roles-viewroles-html.lsp | 9 ++++-----
 1 file 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>
-- 
cgit v1.2.3