Merge branch 'main' into production

bookwyrm/templates/user/shelf.html

@@ -1,9 +1,8 @@
-{% extends 'layout.html' %}
+{% extends 'user/user_layout.html' %}
 {% load bookwyrm_tags %}
-{% block content %}
 
-<div class="columns">
-    <div class="column">
+{% block header %}
+<header class="columns">
     <h1 class="title">
         {% if is_self %}Your
         {% else %}
@@ -11,11 +10,10 @@
         {% endif %}
         shelves
     </h1>
-    </div>
-</div>
-
-{% include 'snippets/user_header.html' with user=user %}
+</header>
+{% endblock %}
 
+{% block panel %}
 <div class="block columns">
     <div class="column">
         <div class="tabs" role="tablist">

bookwyrm/tests/views/test_shelf.py

@@ -41,7 +41,7 @@ class ShelfViews(TestCase):
             is_api.return_value = False
             result = view(request, self.local_user.username, shelf.identifier)
         self.assertIsInstance(result, TemplateResponse)
-        self.assertEqual(result.template_name, 'shelf.html')
+        self.assertEqual(result.template_name, 'user/shelf.html')
         self.assertEqual(result.status_code, 200)
 
         with patch('bookwyrm.views.shelf.is_api_request') as is_api:

bookwyrm/views/shelf.py

@@ -62,7 +62,7 @@ class Shelf(View):
             'books': [b.book for b in books],
         }
 
-        return TemplateResponse(request, 'shelf.html', data)
+        return TemplateResponse(request, 'user/shelf.html', data)
 
     @method_decorator(login_required, name='dispatch')
     # pylint: disable=unused-argument