From db9561db89ac4c7b447910c30f188617b1efcf73 Mon Sep 17 00:00:00 2001
From: Virgil Grigoras <2580723+vigri@users.noreply.github.com>
Date: Sun, 2 Sep 2018 22:21:37 +0200
Subject: [PATCH] Show add to shelf button only when there is a result

---
 cps/templates/search.html | 51 +++++++++++++++++----------------------
 1 file changed, 22 insertions(+), 29 deletions(-)

diff --git a/cps/templates/search.html b/cps/templates/search.html
index 6853c819..4a592d16 100644
--- a/cps/templates/search.html
+++ b/cps/templates/search.html
@@ -1,43 +1,36 @@
 {% extends "layout.html" %}
 {% block body %}
 <div class="discover">
-
     {% if entries|length < 1 %}
       <h2>{{_('No Results for:')}} {{searchterm}}</h2>
       <p>{{_('Please try a different search')}}</p>
     {% else %}
-      <h2>{{entries|length}} {{_('Results for:')}} {{searchterm}}</h2>
-    {%endif%}
+      <h2>{{entries|length}} {{_('Results for:')}} {{searchterm}}</h2>    
       {% if g.user.is_authenticated %}
-      {% if g.user.shelf.all() or g.public_shelfes %}
-      <div id="shelf-actions" class="btn-toolbar" role="toolbar">
-        <div class="btn-group" role="group" aria-label="Add to shelves">
-          <button id="add-to-shelf" type="button" class="btn btn-primary btn-sm dropdown-toggle" data-toggle="dropdown" aria-haspopup="true" aria-expanded="false">
-            <span class="glyphicon glyphicon-list"></span> {{_('Add to shelf')}}
-            <span class="caret"></span>
-          </button>
-          <ul id="add-to-shelves" class="dropdown-menu" aria-labelledby="add-to-shelf">
-            {% for shelf in g.user.shelf %}
-              {% if shelf.is_public != 1 %}
-                <li>
-                  <a href="{{ url_for('search_to_shelf', shelf_id=shelf.id) }}"> {{shelf.name}}</a>
-                  
-                </li>
-              {% endif %}
-            {% endfor %}
-            {% for shelf in g.public_shelfes %}
-                <li>
-                  <a href="{{ url_for('search_to_shelf', shelf_id=shelf.id) }}">{{shelf.name}}</a>
-                </li>
-            {% endfor %}
-          </ul>
-        </div>
-      </div>
+        {% if g.user.shelf.all() or g.public_shelfes %}
+          <div id="shelf-actions" class="btn-toolbar" role="toolbar">
+            <div class="btn-group" role="group" aria-label="Add to shelves">
+              <button id="add-to-shelf" type="button" class="btn btn-primary btn-sm dropdown-toggle" data-toggle="dropdown" aria-haspopup="true" aria-expanded="false">
+                <span class="glyphicon glyphicon-list"></span> {{_('Add to shelf')}}
+                <span class="caret"></span>
+              </button>
+              <ul id="add-to-shelves" class="dropdown-menu" aria-labelledby="add-to-shelf">
+                {% for shelf in g.user.shelf %}
+                  {% if shelf.is_public != 1 %}
+                    <li><a href="{{ url_for('search_to_shelf', shelf_id=shelf.id) }}"> {{shelf.name}}</a></li>
+                  {% endif %}
+                {% endfor %}
+                {% for shelf in g.public_shelfes %}
+                  <li><a href="{{ url_for('search_to_shelf', shelf_id=shelf.id) }}">{{shelf.name}}</a></li>
+                {% endfor %}
+              </ul>
+            </div>
+          </div>
+        {% endif %}
       {% endif %}
-    {% endif %}
+  {% endif %}
 
   <div class="row">
-
     {% for entry in entries %}
     <div class="col-sm-3 col-lg-2 col-xs-6 book">
       <div class="cover">