From f028e2eb6efd0f21c0b7377ce904541a8b370dc5 Mon Sep 17 00:00:00 2001
From: Anthony Johnson <aj@ohess.org>
Date: Fri, 8 Dec 2017 15:19:50 -0700
Subject: [PATCH] Don't display the build suggestions div if there are no
 suggestions

This added extraneous spacing to the build page.
---
 .../templates/builds/build_detail.html        | 39 +++++++++----------
 1 file changed, 19 insertions(+), 20 deletions(-)

diff --git a/readthedocs/templates/builds/build_detail.html b/readthedocs/templates/builds/build_detail.html
index 7a66007c0c6..b07a252a479 100644
--- a/readthedocs/templates/builds/build_detail.html
+++ b/readthedocs/templates/builds/build_detail.html
@@ -92,28 +92,27 @@
       </span>
     </div>
 
-    <div class="build-ideas">
     {% if not build.success and build.commands.count < 4 %}
-      <p>
-
-        {% url 'wipe_version' build.version.project.slug build.version.slug as wipe_url %}
-        {% blocktrans %}
-        Having trouble with your build environment?
-        Try <a href="{{ wipe_url }}">reseting it</a>.
-        {% endblocktrans %}
-      </p>
-    {% endif %}
-
-    {% if not build.success and "setup.py install" in build.commands.last.output %}
-      <p>
-        {% url 'projects_advanced' build.version.project.slug as advanced_url %}
-        {% blocktrans %}
-        Don't want <em>setup.py install</em> called?
-        Change the <strong>Install Project</strong> setting in your <a href="{{ advanced_url }}">advanced settings</a>.
-        {% endblocktrans %}
-      </p>
+      <div class="build-ideas">
+        <p>
+          {% url 'wipe_version' build.version.project.slug build.version.slug as wipe_url %}
+          {% blocktrans %}
+            Having trouble with your build environment?
+            Try <a href="{{ wipe_url }}">reseting it</a>.
+          {% endblocktrans %}
+        </p>
+      </div>
+    {% elif not build.success and "setup.py install" in build.commands.last.output %}
+      <div class="build-ideas">
+        <p>
+          {% url 'projects_advanced' build.version.project.slug as advanced_url %}
+          {% blocktrans %}
+            Don't want <em>setup.py install</em> called?
+            Change the <strong>Install Project</strong> setting in your <a href="{{ advanced_url }}">advanced settings</a>.
+          {% endblocktrans %}
+        </p>
+      </div>
     {% endif %}
-    </div>
 
     {% if build.output %}
       {# If we have build output, this is an old build #}