From 8d53b7589fc448b84a8fa63fb1ee47552c06769c Mon Sep 17 00:00:00 2001
From: Fabien Basmaison <contact@arkhi.org>
Date: Tue, 27 Apr 2021 17:23:37 +0200
Subject: [PATCH] cover: Update /get-started/books:

- Remove `.content` from templates.
- Remove a stray unclosed label.
---
 .../templates/get_started/book_preview.html   |  8 +--
 bookwyrm/templates/get_started/books.html     | 63 +++++++++++--------
 2 files changed, 40 insertions(+), 31 deletions(-)

diff --git a/bookwyrm/templates/get_started/book_preview.html b/bookwyrm/templates/get_started/book_preview.html
index 04d0c424d..578fef708 100644
--- a/bookwyrm/templates/get_started/book_preview.html
+++ b/bookwyrm/templates/get_started/book_preview.html
@@ -1,8 +1,8 @@
 {% load i18n %}
-<div class="column is-narrow is-clipped has-text-centered">
-    {% include 'snippets/book_cover.html' with book=book %}
-    <label class="label" for="id_shelve_{{ book.id }}">
-    <div class="select is-small">
+<div class="column is-cover">
+    {% include 'snippets/book_cover.html' with book=book cover_class='is-h-l is-h-m-mobile' %}
+
+    <div class="select is-small mt-1 mb-3">
         <select name="{{ book.id }}" aria-label="{% blocktrans with book_title=book.title %}Have you read {{ book_title }}?{% endblocktrans %}">
             <option disabled selected value>Add to your books</option>
             {% for shelf in request.user.shelf_set.all %}
diff --git a/bookwyrm/templates/get_started/books.html b/bookwyrm/templates/get_started/books.html
index ae2651220..9613508b9 100644
--- a/bookwyrm/templates/get_started/books.html
+++ b/bookwyrm/templates/get_started/books.html
@@ -23,34 +23,43 @@
 
 <form class="block" name="add-books" method="post" action="{% url 'get-started-books' %}">
     {% csrf_token %}
+
     <h3 class="title is-5">{% trans "Suggested Books" %}</h3>
-    <fieldset name="books" class="columns scroll-x is-mobile">
-        {% if book_results %}
-        <div class="column is-narrow content">
-            <p class="help mb-0">Search results</p>
-            <div class="columns is-mobile">
-                {% for book in book_results %}
-                {% include 'get_started/book_preview.html' %}
-                {% endfor %}
-            </div>
-        </div>
-        {% endif %}
-        {% if popular_books %}
-        <div class="column is-narrow content">
-            <p class="help mb-0">
-                {% blocktrans %}Popular on {{ site_name }}{% endblocktrans %}
-            </p>
-            <div class="columns is-mobile">
-                {% for book in popular_books %}
-                {% include 'get_started/book_preview.html' %}
-                {% endfor %}
-            </div>
-        </div>
-        {% endif %}
-        {% if not book_results and not popular_books %}
-        <p><em>{% trans "No books found" %}</em></p>
-        {% endif %}
-    </fieldset>
+
+    <div class="block scroll-x">
+        <fieldset name="books" class="columns is-mobile">
+            {% if book_results %}
+                <div class="column is-narrow">
+                    <p class="help mb-0">Search results</p>
+
+                    <div class="columns is-mobile">
+                        {% for book in book_results %}
+                        {% include 'get_started/book_preview.html' %}
+                        {% endfor %}
+                    </div>
+                </div>
+            {% endif %}
+
+            {% if popular_books %}
+                <div class="column is-narrow">
+                    <p class="help mb-0">
+                        {% blocktrans %}Popular on {{ site_name }}{% endblocktrans %}
+                    </p>
+
+                    <div class="columns is-mobile">
+                        {% for book in popular_books %}
+                        {% include 'get_started/book_preview.html' %}
+                        {% endfor %}
+                    </div>
+                </div>
+            {% endif %}
+
+            {% if not book_results and not popular_books %}
+                <p><em>{% trans "No books found" %}</em></p>
+            {% endif %}
+        </fieldset>
+    </div>
+
     <button type="submit" class="button is-primary">{% trans "Save &amp; continue" %}</button>
 </form>
 {% endblock %}