From efde5d7b6fd2147ff516ce031ecd61754f15466a Mon Sep 17 00:00:00 2001
From: Piotr Sarnacki <drogus@gmail.com>
Date: Fri, 13 Mar 2015 13:39:14 +0100
Subject: [PATCH] Don't show toTop or tailing buttons if there's no log

---
 app/templates/jobs/pre.hbs | 25 ++++++++++++++-----------
 app/views/pre.coffee       |  5 +++++
 2 files changed, 19 insertions(+), 11 deletions(-)

diff --git a/app/templates/jobs/pre.hbs b/app/templates/jobs/pre.hbs
index e969488b..e60f99c4 100644
--- a/app/templates/jobs/pre.hbs
+++ b/app/templates/jobs/pre.hbs
@@ -12,19 +12,22 @@
       <a class="button button--grey" {{bind-attr href="view.plainTextLogUrl"}}><span class="icon icon--downloadLog"></span> Download Log</a>
     </menu>
     <div class="log-body">
-      <a href="#" id="tail" class="log-tail" {{action "toggleTailing" target=view}}>
-        <span class="tail-status"></span>
-        <span class="tail-label button button--grey"><span class="icon icon--down"></span>
-          {{#if view.job.isFinished}}
-            Scroll to End of Log
-          {{else}}
-            Follow Log
-          {{/if}}
-        </span>
-      </a>
+      {{#if view.showTailing}}
+        <a href="#" id="tail" class="log-tail" {{action "toggleTailing" target=view}}>
+          <span class="tail-status"></span>
+          <span class="tail-label button button--grey"><span class="icon icon--down"></span>
+            {{#if view.job.isFinished}}
+              Scroll to End of Log
+            {{else}}
+              Follow Log
+            {{/if}}
+          </span>
+        </a>
+      {{/if}}
+
       <pre id="log" class="ansi"></pre>
 
-      {{#if view.log.hasContent}}
+      {{#if view.showToTop}}
         <a href='#' class="to-top" {{action "toTop" target=view}}>Top</a>
       {{/if}}
 
diff --git a/app/views/pre.coffee b/app/views/pre.coffee
index 89f475d5..67ffeea2 100644
--- a/app/views/pre.coffee
+++ b/app/views/pre.coffee
@@ -111,6 +111,11 @@ View = BasicView.extend
       job.get('canRemoveLog') && @get('hasPermission')
   ).property('job.canRemoveLog', 'hasPermission')
 
+  showToTop: (->
+    @get('log.hasContent') && @get('job.canRemoveLog')
+  ).property('log.hasContent', 'job.canRemoveLog')
+  showTailing: Ember.computed.alias('showToTop')
+
   actions:
     toTop: () ->
       Travis.tailing.stop()