From 7f1908a9382389504f7b8474e9a36ff90c7489e0 Mon Sep 17 00:00:00 2001
From: Rob Vermaas <rob.vermaas@gmail.com>
Date: Thu, 25 Feb 2010 09:17:17 +0000
Subject: [PATCH]  * hydra: changes in list in stead of table, also showing
 changes of changed 'build output' inputs

---
 src/root/build.tt  |  3 ++-
 src/root/common.tt | 37 +++++++++++++++++++------------------
 2 files changed, 21 insertions(+), 19 deletions(-)

diff --git a/src/root/build.tt b/src/root/build.tt
index 8730fb47..bca38ef9 100644
--- a/src/root/build.tt
+++ b/src/root/build.tt
@@ -81,7 +81,6 @@
 <p class="error">[% flashMsg %]</p>
 [% END %]
 
-
 <div id="generic-tabs">
 	<ul>
 		<li><a href="#tabs-summary">Summary</a></li>
@@ -191,6 +190,8 @@
                     following changes were made since:
                 </p>
                 [% INCLUDE renderInputDiff build2=build , build1=prevSuccessfulBuild %]
+                <br />
+                <p/>
             [% END %]
 
 			[% IF build.resultInfo.errormsg && build.resultInfo.buildstatus != 5 %]
diff --git a/src/root/common.tt b/src/root/common.tt
index 0cd750ef..0058483e 100644
--- a/src/root/common.tt
+++ b/src/root/common.tt
@@ -278,37 +278,38 @@
 [% END %]
 
 [% BLOCK renderInputDiff; %]
-
-<table class="tablesorter">
-    <thead>
-        <tr><th>Name</th><th>Change</th></tr>
-    </thead>
-    <tbody>
+<ul class="inputdiff">
     [% FOREACH bi1 IN build1.inputs %]
         [% deletedInput = 1 %]
         [% FOREACH bi2 IN build2.inputs %]
             [% IF bi1.name == bi2.name %]
-                <tr>
                 [% IF bi1.type == bi2.type %]
-                    [% IF bi1.value != bi2.value || bi1.uri != bi2.uri || bi1.revision != bi2.revision || bi1.path != bi2.path || bi1.dependency != bi2.dependency %]
-                        <td>[% bi1.name %]</td>
-                        <td>
-                            [% INCLUDE renderInputValue input=bi1 %] to [% INCLUDE renderInputValue input=bi2 %]
-                        </td>
+                    [% IF bi1.value != bi2.value || bi1.uri != bi2.uri %]
+                        <li><b>[% bi1.name %]</b>: [% INCLUDE renderInputValue input=bi1 %] to [% INCLUDE renderInputValue input=bi2 %]</li>
+                    [% ELSIF bi1.uri == bi2.uri && bi1.revision != bi2.revision %]
+                        <li>
+                            <b>[% bi1.name %]</b>: [% bi1.uri %] revision changed from [% bi1.revision %] to [% bi2.revision %].
+                        </li>
+                    [% ELSIF bi1.dependency.id != bi2.dependency.id || bi1.path != bi2.path %]
+                        <li>
+                            <b>[% bi1.name %]</b>: [% INCLUDE renderInputValue input=bi1 %] to [% INCLUDE renderInputValue input=bi2 %]
+                            <br/>
+                            <br/>
+                            [% INCLUDE renderInputDiff build1=bi1.dependency, build2=bi2.dependency %]
+                        </li>
                     [% END %]
                 [% ELSE %]                  
-                    <td>[% bi1.name %]</td>
-                    <td>Changed input type from '[% type = bi1.type; inputTypes.$type %]' to '[% type = bi2.type; inputTypes.$type %]'</td>
+                    <li><b>[% bi1.name %]</b>: Changed input type from '[% type = bi1.type; inputTypes.$type %]' to '[% type = bi2.type; inputTypes.$type %]'</li>
                 [% END %]
-                </tr>
+                
                 [% deletedInput = 0 %]
             [% END %]
         [% END %]
         [% IF deletedInput == 1 %]
-            <tr><td>[% bi1.name %]</td><td>Input not present in this build.</td></tr>
+            <li><b>[% bi1.name %]</b>: Input not present in this build.</li>
         [% END %]
     [% END %]
-    </tbody>
-</table>
+</ul>
+
 [% END %]