Ticket #3921: perm-display-grouping-genshi.patch

webadmin/htdocs/css/admin.css

@@ -13 +13 @@
 .tabs :link, .tabs :visited { border: none; display: block }
 .tabs :link:hover, .tabs :visited:hover { background: transparent }
 
+/*-- permissions screen.  uncomment to draw border around hovered element */
+.perm-actionname { cursor: pointer; } /*padding-left:2px;*/
+/*.perm-actionname:hover {  border: 1px dotted #aaa; padding-left:1px;} */
+
 .tabcontents {
  padding: 0.4em 2em;
  margin-left: 12em;

webadmin/htdocs/js/admin.js

@@ -18 +18 @@
     classNames.splice(classIndex, class1 ? 1 : 0, class2);
     element.className = classNames.join(' ');
 }
+/**
+ * toggles a checkbox on/off.  Expects an element id.
+ */
+function toggleCheck( elementId ) {
+    chk = document.getElementById(elementId);
+    if ( chk ) {
+        chk.checked = !chk.checked;  
+    }   
+}
 
 var fragmentId = document.location.hash;
 if (fragmentId) {

webadmin/templates/admin_perm.html

@@ -1 +1 @@
 <div xmlns="http://www.w3.org/1999/xhtml"
   xmlns:py="http://genshi.edgewall.org/"
   id="page">
+  <script type="text/javascript" src="/trac/prism/chrome/admin/js/admin.js"></script>
   <h2>Manage Permissions</h2>
 
   <form id="addperm" class="addnew" method="post">
@@ -42 +43 @@
   <form method="post">
     <table class="listing" id="permlist">
       <thead>
-        <tr><th class="sel">&nbsp;</th><th>Subject</th><th>Action</th></tr>
+        <tr><th class="sel">&nbsp;</th><th>Subject</th></tr>
       </thead>
       <tbody>
-        <tr py:for="perm in perms">
-          <td><input type="checkbox" name="sel" value="$perm.key" /></td>
-          <td>$perm.subject</td>
-          <td>$perm.action</td>
-        </tr>
+        <py:for each="subject, permGroup in groupby(perms, lambda tmp: tmp.subject)">
+          <tr>
+            <td>${subject}</td>
+            <td>
+              <py:for each="p in permGroup">
+                <div style="width:160px;float:left;">
+                  <input type="checkbox" name="sel" value="$p.key" id="$p.key"/>
+                  <span class="perm-actionname" onclick="toggleCheck('$p.key');">$p.action</span>
+                </div>
+              </py:for>
+            </td>
+          </tr>
+        </py:for>
       </tbody>
     </table>
     <div class="buttons">