From 57ed0fe4cf3f9434c8f91f1d49cbc7b22dc77df0 Mon Sep 17 00:00:00 2001
From: Ted Trask <ttrask01@yahoo.com>
Date: Mon, 3 Feb 2014 21:21:36 +0000
Subject: Cleanup HTML including removing DL/DT/DD, use tablesorter where
 possible, and use lowercase tags

---
 awall-listpolicies-html.lsp | 41 ++++++++++++++++++++++++-----------------
 1 file changed, 24 insertions(+), 17 deletions(-)

(limited to 'awall-listpolicies-html.lsp')

diff --git a/awall-listpolicies-html.lsp b/awall-listpolicies-html.lsp
index c816bfa..568fd9f 100644
--- a/awall-listpolicies-html.lsp
+++ b/awall-listpolicies-html.lsp
@@ -9,8 +9,15 @@ html = require("acf.html")
 	}
 </script>
 
+<script type="text/javascript">
+	if (typeof $.tablesorter == 'undefined') {
+		document.write('<script type="text/javascript" src="<%= html.html_escape(page_info.wwwprefix) %>/js/jquery.tablesorter.js"><\/script>');
+	}
+</script>
+
 <script type="text/javascript">
         $(document).ready(function() {
+		$("#list").tablesorter({widgets: ['zebra']});
 		$(".deletepolicy").click(function(){ return confirm("Are you sure you want to delete this policy?")});
 	});
 </script>
@@ -19,18 +26,18 @@ html = require("acf.html")
 <% htmlviewfunctions.displaycommandresults({"createpolicy"}, session, true) %>
 
 <h1>Policies</h1>
-<DL>
-<% if form.errtxt then %><P CLASS='error'><%= string.gsub(html.html_escape(form.errtxt), "\n", "<BR>") %></P><% end %>
-<TABLE>
-	<TR style="background:#eee;font-weight:bold;">
-		<TD style="padding-right:20px;white-space:nowrap;text-align:left;" class="header">Action</TD>
-		<TD style="padding-right:20px;white-space:nowrap;text-align:left;" class="header">Status</TD>
-		<TD style="padding-right:20px;white-space:nowrap;text-align:left;" class="header">Name</TD>
-		<TD style="white-space:nowrap;text-align:left;" class="header">Description</TD>
-	</TR>		
+<% if form.errtxt then %><p class='error'><%= string.gsub(html.html_escape(form.errtxt), "\n", "<br/>") %></p><% end %>
+<table id="list" class="tablesorter"><thead>
+	<tr>
+		<th>Action</th>
+		<th>Status</th>
+		<th>Name</th>
+		<th>Description</th>
+	</tr>
+</thead><tbody>
 <% for i,pol in ipairs(form.value) do %>
-	<TR>
-		<TD style="padding-right:20px;white-space:nowrap;">
+	<tr>
+		<td>
 			<%
 			if viewlibrary.check_permission("deletepolicy") and pol.editable and pol.status ~= "enabled" and pol.status ~= "required" then io.write(html.link{value = "deletepolicy?submit=true&filename=" .. pol.filename, label="Delete ", class="deletepolicy" }) end
 			if viewlibrary.check_permission("editpolicy") and pol.editable then io.write(html.link{value = "editpolicy?filename=" .. pol.filename.."&redir="..page_info.orig_action, label="Edit " }) end
@@ -38,13 +45,13 @@ html = require("acf.html")
 			if viewlibrary.check_permission("enablepolicy") and (pol.status == "disabled" or pol.status == "required") then io.write(html.link{value = "enablepolicy?submit=true&name=" .. pol.name, label="Enable " }) end
 			if viewlibrary.check_permission("disablepolicy") and pol.status == "enabled" then io.write(html.link{value = "disablepolicy?submit=true&name=" .. pol.name, label="Disable " }) end
 			%>
-		</TD>
-		<TD style="white-space:nowrap;" width="90%"><%= html.html_escape(string.gsub(pol.status or "", "^%l", string.upper)) %></TD>
-		<TD style="white-space:nowrap;" width="90%"><%= html.html_escape(pol.name) %></TD>
-		<TD style="white-space:nowrap;" width="90%"><%= html.html_escape(pol.description) %></TD>
-	</TR>
+		</td>
+		<td><%= html.html_escape(string.gsub(pol.status or "", "^%l", string.upper)) %></td>
+		<td><%= html.html_escape(pol.name) %></td>
+		<td><%= html.html_escape(pol.description) %></td>
+	</tr>
 <% end %>
-</TABLE></DL>
+</tbody></table>
 
 <% if viewlibrary and viewlibrary.dispatch_component and viewlibrary.check_permission("createpolicy") then
 	viewlibrary.dispatch_component("createpolicy")
-- 
cgit v1.2.3