From c0f9c9b4aa97b37b92f5175f4b005d8b4adbe114 Mon Sep 17 00:00:00 2001
From: Eelco Dolstra <eelco.dolstra@logicblox.com>
Date: Fri, 22 Feb 2013 18:11:29 +0100
Subject: [PATCH] Make some more tables clickable

---
 src/root/common.tt |  6 +++---
 src/root/search.tt | 12 ++++++------
 src/root/users.tt  | 47 +++++++++++++++++++++++-----------------------
 3 files changed, 32 insertions(+), 33 deletions(-)

diff --git a/src/root/common.tt b/src/root/common.tt
index d691bc8d..6a8b576c 100644
--- a/src/root/common.tt
+++ b/src/root/common.tt
@@ -34,17 +34,17 @@ BLOCK renderJobsetName %]
 
 
 BLOCK renderJobName %]
-<a href="[% c.uri_for('/job' project jobset job) %]"><tt>[% job %]</tt></a>
+<a [% IF inRow %]class="row-link"[% END %] href="[% c.uri_for('/job' project jobset job) %]"><tt>[% job %]</tt></a>
 [% END;
 
 
 BLOCK renderFullJobsetName %]
-<tt>[% INCLUDE renderProjectName %]:[% INCLUDE renderJobsetName %]</tt>
+<tt>[% INCLUDE renderProjectName inRow=0 %]:[% INCLUDE renderJobsetName %]</tt>
 [% END;
 
 
 BLOCK renderFullJobName %]
-<tt>[% IF !hideProjectName; INCLUDE renderProjectName %]:[% END; IF !hideJobsetName; INCLUDE renderJobsetName %]:[% END; INCLUDE renderJobName %]</tt>
+<tt>[% IF !hideProjectName; INCLUDE renderProjectName inRow=0 %]:[% END; IF !hideJobsetName; INCLUDE renderJobsetName inRow=0 %]:[% END; INCLUDE renderJobName %]</tt>
 [% END;
 
 
diff --git a/src/root/search.tt b/src/root/search.tt
index 7b67b367..f64be628 100644
--- a/src/root/search.tt
+++ b/src/root/search.tt
@@ -5,14 +5,14 @@
 
   <p>The following projects match your query:</p>
 
-  <table class="table table-striped table-condensed">
+  <table class="table table-striped table-condensed clickable-rows">
     <thead>
       <tr><th>Project</th><th>Description</th></tr>
     </thead>
     <tbody>
       [% FOREACH p IN projects %]
         <tr>
-          <td><span class="[% IF !p.enabled %]disabled-project[% END %]">[% INCLUDE renderProjectName project=p.name %]</span></td>
+          <td><span class="[% IF !p.enabled %]disabled-project[% END %]">[% INCLUDE renderProjectName project=p.name inRow=1 %]</span></td>
           <td>[% HTML.escape(p.description) %]</td>
         </tr>
       [% END %]
@@ -25,14 +25,14 @@
 
   <p>The following jobsets match your query:</p>
 
-  <table class="table table-striped table-condensed">
+  <table class="table table-striped table-condensed clickable-rows">
     <thead>
       <tr><th>Jobset</th><th>Description</th></tr>
     </thead>
     <tbody>
       [% FOREACH j IN jobsets %]
         <tr>
-          <td><span class="[% IF !j.enabled %]disabled-jobset[% END %]">[% INCLUDE renderFullJobsetName project=j.get_column('project') jobset=j.name %]</span></td>
+          <td><span class="[% IF !j.enabled %]disabled-jobset[% END %]">[% INCLUDE renderFullJobsetName project=j.get_column('project') jobset=j.name inRow=1 %]</span></td>
           <td>[% HTML.escape(j.description) %]</td>
         </tr>
       [% END %]
@@ -45,14 +45,14 @@
 
   <p>The following jobs match your query:[% IF jobs.size > limit %] <span class="text-warning">(first [% limit %] results only)</span>[% END %]</p>
 
-  <table class="table table-striped table-condensed">
+  <table class="table table-striped table-condensed clickable-rows">
     <thead>
       <tr><th>Job</th></tr>
     </thead>
     <tbody>
       [% FOREACH j IN jobs %]
         <tr>
-          <td><span class="[% IF !j.get_column('enabled') %]disabled-job[% END %]">[% INCLUDE renderFullJobName project=j.get_column('project') jobset=j.get_column('jobset') job=j.name %]</span></td>
+          <td><span class="[% IF !j.get_column('enabled') %]disabled-job[% END %]">[% INCLUDE renderFullJobName project=j.get_column('project') jobset=j.get_column('jobset') job=j.name inRow=1 %]</span></td>
         </tr>
       [% END %]
     </tbody>
diff --git a/src/root/users.tt b/src/root/users.tt
index 56d54088..13ec5e62 100644
--- a/src/root/users.tt
+++ b/src/root/users.tt
@@ -1,30 +1,29 @@
 [% WRAPPER layout.tt title="Users" %]
 [% PROCESS common.tt %]
 
-<table class="table table-striped table-condensed">
-    <thead>
-        <tr>
-            <th>Username</th>
-            <th>Name</th>
-            <th>Email</th>
-            <th>Roles</th>
-            <th>Eval. notifications</th>
-            <th>Options</th>
-        </tr>
-    </thead>
-    <tbody>
-        [% FOREACH u IN users %]
-        <tr>
-            <td><a href="[% c.uri_for(c.controller('Admin').action_for('user_edit'), [u.username]) %]">[% u.username %]</a></td>
-            <td>[% u.fullname %]</td>
-            <td>[% u.emailaddress %]</td>
-            <td>[% FOREACH r IN u.userroles %]<i>[% r.role %]</i> [% END %]</td>
-            <td>[% IF u.emailonerror %]Yes[% ELSE %]No[% END %]</td>
-            <td>[% INCLUDE maybeLink uri = c.uri_for(c.controller('Admin').action_for('reset_password'), [u.username]) content = "Reset password" confirmmsg = "Are you sure you want to reset the password for this user?" class = "btn btn-mini" %]</td>
-        </tr>
-        [% END %]
-    </tbody>
-
+<table class="table table-striped table-condensed clickable-rows">
+  <thead>
+    <tr>
+      <th>Username</th>
+      <th>Name</th>
+      <th>Email</th>
+      <th>Roles</th>
+      <th>Eval. notifications</th>
+      <th>Options</th>
+    </tr>
+  </thead>
+  <tbody>
+    [% FOREACH u IN users %]
+      <tr>
+        <td><a class="row-link" href="[% c.uri_for(c.controller('Admin').action_for('user_edit'), [u.username]) %]">[% u.username %]</a></td>
+        <td>[% u.fullname %]</td>
+        <td>[% u.emailaddress %]</td>
+        <td>[% FOREACH r IN u.userroles %]<i>[% r.role %]</i> [% END %]</td>
+        <td>[% IF u.emailonerror %]Yes[% ELSE %]No[% END %]</td>
+        <td>[% INCLUDE maybeLink uri = c.uri_for(c.controller('Admin').action_for('reset_password'), [u.username]) content = "Reset password" confirmmsg = "Are you sure you want to reset the password for this user?" class = "btn btn-mini" %]</td>
+      </tr>
+    [% END %]
+  </tbody>
 </table>
 
 <p><a class="btn" href="[% c.uri_for(c.controller('Admin').action_for('create_user')) %]">