From 9e7af7eaad35b62fbf6104c7a3ff6b97a4eacc22 Mon Sep 17 00:00:00 2001
From: Lisa Passing <mail@lislis.de>
Date: Mon, 20 Apr 2015 12:33:24 +0200
Subject: [PATCH] explicitly check for pullRequest caches and some style
 changes

---
 app/styles/app/_mixins/mixins.sass | 14 ++++++++++++++
 app/styles/app/layouts/caches.sass | 13 ++++++++++++-
 app/templates/caches.hbs           | 20 +++++++++++---------
 3 files changed, 37 insertions(+), 10 deletions(-)

diff --git a/app/styles/app/_mixins/mixins.sass b/app/styles/app/_mixins/mixins.sass
index 9b31edf3..e4609b31 100644
--- a/app/styles/app/_mixins/mixins.sass
+++ b/app/styles/app/_mixins/mixins.sass
@@ -67,3 +67,17 @@
 
 %inline-block
   display: inline-block
+
+%helptext
+  font-size: $font-size-m
+  color: $grey1
+  a
+    color: $grey3
+    text-decoration: none
+    &:hover
+      text-decoration: underline
+
+%section-title
+  font-size: $font-size-sm
+  color: $grey1
+  font-weight: 400
diff --git a/app/styles/app/layouts/caches.sass b/app/styles/app/layouts/caches.sass
index 6671eb4b..8112594b 100644
--- a/app/styles/app/layouts/caches.sass
+++ b/app/styles/app/layouts/caches.sass
@@ -52,4 +52,15 @@
     a:hover
       text-decoration: underline
   .button--delete
-    float: right
\ No newline at end of file
+    float: right
+
+.caches-helptext
+  @extend %helptext
+
+.caches-title
+  @extend %helptext
+  @extend %section-title
+  @include clearfix
+  float: left
+  margin: 1em 0 .5em
+
diff --git a/app/templates/caches.hbs b/app/templates/caches.hbs
index 0c1d9de3..c79ca46a 100644
--- a/app/templates/caches.hbs
+++ b/app/templates/caches.hbs
@@ -1,25 +1,27 @@
 {{#if model.pushes.length}}
 
   <div class="caches-header">
-    <h1 class="build-title">All caches <small>(<a href="http://docs.travis-ci.com/user/caching/" title="">documentation</a>)</small></h1>
+    <h1 class="caches-title">All caches <small>(<a href="http://docs.travis-ci.com/user/caching/" title="">documentation</a>)</small></h1>
     <a href="#" {{action "deleteRepoCache"}} {{bind-attr class="isDeleting:deleting :delete-repo-caches :button--delete"}}>
       Delete all repository caches
     </a>
   </div>
-  <h2 class="build-title">Pushes</h2>
+  <h2 class="caches-title">Pushes</h2>
   <ul id="caches" class="caches">
     {{#each cache in model.pushes }}
       {{caches-item cache=cache repo=repo caches=model.pushes}}
     {{/each}}
   </ul>
 
-  <h2 class="build-title">Pull Requests</h2>
-  <ul class="caches">
-    {{#each cache in model.pullRequests }}
-      {{caches-item cache=cache repo=repo caches=model.pullRequests}}
-    {{/each}}
-  </ul>
+  {{#if model.pullRequests.length}}
+    <h2 class="caches-title">Pull Requests</h2>
+    <ul class="caches">
+      {{#each cache in model.pullRequests }}
+        {{caches-item cache=cache repo=repo caches=model.pullRequests}}
+      {{/each}}
+    </ul>
+  {{/if}}
 
 {{else}}
-  <p>No caches have been created yet,<br>read more on <a href="http://docs.travis-ci.com/user/caching/" title="">how to setup caching with your build</a>.</p>
+  <p class="caches-helptext">No caches have been created yet,<br>read more on <a href="http://docs.travis-ci.com/user/caching/" title="">how to setup caching with your build</a>.</p>
 {{/if}}