From ab848b9ebfd61056629a7bf93dc993d99763475d Mon Sep 17 00:00:00 2001
From: Dan Cryer <dan@block8.co.uk>
Date: Fri, 25 Apr 2014 11:25:46 +0000
Subject: [PATCH] UI improvements for the Build view screen

---
 PHPCI/View/Build/view.phtml | 98 +++++++++++++++++++++++++++++++------
 1 file changed, 82 insertions(+), 16 deletions(-)

diff --git a/PHPCI/View/Build/view.phtml b/PHPCI/View/Build/view.phtml
index 0c6817c9..52d4db80 100644
--- a/PHPCI/View/Build/view.phtml
+++ b/PHPCI/View/Build/view.phtml
@@ -1,38 +1,60 @@
-<div id="title">
-	<h1 style="display: inline-block"><?php print $build->getProject()->getTitle(); ?> - Build #<?php print $build->getId(); ?></h1>
+<div id="title" class="panel">
+    <img class="pull-left" src="http://www.gravatar.com/avatar/<?php print md5($build->getCommitterEmail()); ?>?d=mm">
+
+    <div class="panel-heading">
+        <h1 class="panel-title">
+            <?php print $build->getProject()->getTitle(); ?>
+            <span>#<?php print $build->getId(); ?></span>
+
+            <label class="pull-right label"></label>
+        </h1>
+    </div>
+
 
     <div id="build-info">
+        <?php if ($build->getCommitMessage()): ?>
+        <div class="commit-message">
+            <?php print $build->getCommitMessage(); ?>
+        </div>
+        <?php endif; ?>
+
         <strong>Branch: </strong> <?php print $build->getBranch(); ?><br>
-        <strong>Committer: </strong> <?php print $build->getCommitterEmail(); ?><br>
-        <strong>Commit ID: </strong> <?php print $build->getCommitId() == 'Manual' ? 'HEAD' : $build->getCommitId(); ?><br>
-        <strong>Commit Message: </strong> <?php print $build->getCommitMessage(); ?>
+        <strong>Committer: </strong> <?php print $build->getCommitterEmail(); ?>
+
+        <?php if ($build->getCommitId() != 'Manual'): ?>
+        <br><strong>Commit ID: </strong> <?php print $build->getCommitId(); ?><br>
+        <?php endif; ?>
     </div>
 </div>
 
 <div class="row">
 	<div class="col-lg-3">
-        <ul class="nav nav-pills nav-stacked">
-			<li><a href="<?php echo PHPCI_URL ?>"><i class="icon-home"></i> Dashboard</a></li>
-			<li><a href="<?php echo PHPCI_URL ?>project/view/<?php print $build->getProject()->getId(); ?>"><i class="icon-folder-open"></i> <?php print htmlspecialchars($build->getProject()->getTitle()); ?></a></li>
-		</ul>
-			<h5>Options</h5>
-        <ul class="nav nav-pills nav-stacked">
-			<li><a href="<?php echo PHPCI_URL ?>build/rebuild/<?php print $build->getId(); ?>"><i class="icon-cog"></i> Rebuild</a></li>
-			<?php if($this->User()->getIsAdmin()): ?>
-			<li><a href="<?php echo PHPCI_URL ?>build/delete/<?php print $build->getId(); ?>" id="delete-build"><i class="icon-trash"></i> Delete Build</a></li>
-			<?php endif; ?>
+        <ul class="list-group">
+			<a class="list-group-item" href="<?php echo PHPCI_URL ?>"><i class="icon-home"></i> Dashboard</a>
+			<a class="list-group-item" href="<?php echo PHPCI_URL ?>project/view/<?php print $build->getProject()->getId(); ?>"><i class="icon-folder-open"></i> <?php print htmlspecialchars($build->getProject()->getTitle()); ?></a>
 		</ul>
+
+
+                <h5>Options</h5>
+            <ul class="list-group">
+                <a class="list-group-item" href="<?php echo PHPCI_URL ?>build/rebuild/<?php print $build->getId(); ?>"><i class="icon-cog"></i> Rebuild</a>
+                <?php if($this->User()->getIsAdmin()): ?>
+                    <a class="list-group-item" href="<?php echo PHPCI_URL ?>build/delete/<?php print $build->getId(); ?>" id="delete-build"><i class="icon-trash"></i> Delete Build</a>
+                <?php endif; ?>
+            </ul>
+
 	</div>
 
 	<div class="col-lg-9">
 		<div id="status"></div>
-        <div id="plugins"></div>
+        <div id="plugins" class="row"></div>
     </div>
 </div>
 
 <script>
     var PHPCI = new PHPCIObject(<?php print $build->getId() ?>);
     PHPCI.buildData = <?php print $data; ?>;
+    PHPCI.fileLinkTemplate = <?php print json_encode($build->getFileLinkTemplate()); ?>;
 </script>
 
 <?php
@@ -52,5 +74,49 @@ foreach ($plugins as $plugin) {
             ).onCloseConfirmed = function () {window.location = '/'};
         });
 
+        $(window).on('build-updated', function(data) {
+            updateBuildStatus(data.queryData.status);
+        });
+
+        updateBuildStatus(<?php print $build->getStatus(); ?>);
     });
+
+    function updateBuildStatus(status) {
+        var statusClass = null;
+        var statusText = null;
+
+        switch (status) {
+            case 0:
+                statusClass = 'info';
+                statusText = 'Pending';
+                break;
+            case 1:
+                statusClass = 'warning';
+                statusText = 'Running';
+                break;
+            case 2:
+                statusClass = 'success';
+                statusText = 'Success';
+                break;
+            case 3:
+                statusClass = 'danger';
+                statusText = 'Failed';
+                break;
+        }
+
+        $('#title')
+            .removeClass('panel-info')
+            .removeClass('panel-warning')
+            .removeClass('panel-success')
+            .removeClass('panel-danger')
+            .addClass('panel-' + statusClass);
+
+        $('#title .label')
+            .removeClass('label-info')
+            .removeClass('label-warning')
+            .removeClass('label-success')
+            .removeClass('label-danger')
+            .addClass('label-' + statusClass)
+            .text(statusText);
+    }
 </script>