From 454dd25681af9de9e08ddc021eb29c25ec1ab2f8 Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Fri, 30 Apr 2021 07:49:34 -0700
Subject: [PATCH 01/20] Only make notification count red for mentions

---
 bookwyrm/models/user.py        | 13 +++++++++++++
 bookwyrm/static/js/bookwyrm.js |  2 ++
 bookwyrm/templates/layout.html |  7 +++++--
 bookwyrm/views/updates.py      |  3 ++-
 4 files changed, 22 insertions(+), 3 deletions(-)

diff --git a/bookwyrm/models/user.py b/bookwyrm/models/user.py
index 3efbd6ac8..257264681 100644
--- a/bookwyrm/models/user.py
+++ b/bookwyrm/models/user.py
@@ -150,6 +150,19 @@ class User(OrderedCollectionPageMixin, AbstractUser):
         """for consistent naming"""
         return not self.is_active
 
+    @property
+    def unread_notification_count(self):
+        """ count of notifications, for the templates """
+        return self.notification_set.filter(read=False).count()
+
+    @property
+    def has_unread_mentions(self):
+        """ whether any of the unread notifications are conversations """
+        return self.notification_set.filter(
+            read=False,
+            notification_type__in=["REPLY", "MENTION", "TAG"],
+        ).exists()
+
     activity_serializer = activitypub.Person
 
     @classmethod
diff --git a/bookwyrm/static/js/bookwyrm.js b/bookwyrm/static/js/bookwyrm.js
index 485daf15b..3659a20e4 100644
--- a/bookwyrm/static/js/bookwyrm.js
+++ b/bookwyrm/static/js/bookwyrm.js
@@ -97,10 +97,12 @@ let BookWyrm = new class {
     updateCountElement(counter, data) {
         const currentCount = counter.innerText;
         const count = data.count;
+        const hasMentions = data.has_mentions;
 
         if (count != currentCount) {
             this.addRemoveClass(counter.closest('[data-poll-wrapper]'), 'is-hidden', count < 1);
             counter.innerText = count;
+            this.addRemoveClass(counter.closest('[data-poll-wrapper]'), 'is-danger', hasMentions);
         }
     }
 
diff --git a/bookwyrm/templates/layout.html b/bookwyrm/templates/layout.html
index 84482cdfc..770666e80 100644
--- a/bookwyrm/templates/layout.html
+++ b/bookwyrm/templates/layout.html
@@ -135,8 +135,11 @@
                                 <span class="is-sr-only">{% trans "Notifications" %}</span>
                             </span>
                         </span>
-                        <span class="{% if not request.user|notification_count %}is-hidden {% endif %}tag is-danger is-medium transition-x" data-poll-wrapper>
-                            <span data-poll="notifications">{{ request.user | notification_count }}</span>
+                        <span
+                            class="{% if not request.user.unread_notification_count %}is-hidden {% elif request.user.has_unread_mentions %}is-danger {% endif %}tag is-medium transition-x"
+                            data-poll-wrapper
+                        >
+                            <span data-poll="notifications">{{ request.user.unread_notification_count }}</span>
                         </span>
                     </a>
                 </div>
diff --git a/bookwyrm/views/updates.py b/bookwyrm/views/updates.py
index 349022724..726145626 100644
--- a/bookwyrm/views/updates.py
+++ b/bookwyrm/views/updates.py
@@ -10,7 +10,8 @@ def get_notification_count(request):
     """any notifications waiting?"""
     return JsonResponse(
         {
-            "count": request.user.notification_set.filter(read=False).count(),
+            "count": request.user.unread_notification_count,
+            "has_mentions": request.user.has_unread_mentions,
         }
     )
 

From 2867d703ccc32dada6f9e85c815e3c04354a9788 Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Fri, 30 Apr 2021 07:57:38 -0700
Subject: [PATCH 02/20] Fixes python formatting

---
 bookwyrm/models/user.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/bookwyrm/models/user.py b/bookwyrm/models/user.py
index 257264681..2c5bef4ae 100644
--- a/bookwyrm/models/user.py
+++ b/bookwyrm/models/user.py
@@ -152,12 +152,12 @@ class User(OrderedCollectionPageMixin, AbstractUser):
 
     @property
     def unread_notification_count(self):
-        """ count of notifications, for the templates """
+        """count of notifications, for the templates"""
         return self.notification_set.filter(read=False).count()
 
     @property
     def has_unread_mentions(self):
-        """ whether any of the unread notifications are conversations """
+        """whether any of the unread notifications are conversations"""
         return self.notification_set.filter(
             read=False,
             notification_type__in=["REPLY", "MENTION", "TAG"],

From d2355fef96e8ea517815a2163239fb70236164d4 Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Fri, 30 Apr 2021 08:23:02 -0700
Subject: [PATCH 03/20] Consistent display on followers/following pages

---
 bookwyrm/templates/user/followers.html | 2 +-
 bookwyrm/templates/user/following.html | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/bookwyrm/templates/user/followers.html b/bookwyrm/templates/user/followers.html
index b294db909..2a7ccd1e9 100644
--- a/bookwyrm/templates/user/followers.html
+++ b/bookwyrm/templates/user/followers.html
@@ -25,7 +25,7 @@
         </div>
     </div>
     {% endfor %}
-    {% if not followers.count %}
+    {% if not followers %}
     <div>{% blocktrans with username=user.display_name %}{{ username }} has no followers{% endblocktrans %}</div>
     {% endif %}
 </div>
diff --git a/bookwyrm/templates/user/following.html b/bookwyrm/templates/user/following.html
index 38c01ad27..b41d51264 100644
--- a/bookwyrm/templates/user/following.html
+++ b/bookwyrm/templates/user/following.html
@@ -20,12 +20,12 @@
             </a>
             ({{ follower.username }})
         </div>
-        <div class="column">
+        <div class="column is-narrow">
             {% include 'snippets/follow_button.html' with user=follower %}
         </div>
     </div>
     {% endfor %}
-    {% if not following.count %}
+    {% if not following %}
     <div>{% blocktrans with username=user|username %}{{ username }} isn't following any users{% endblocktrans %}</div>
     {% endif %}
 </div>

From 862ef835366bb716a47f8716c9713ed301e51b36 Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Fri, 30 Apr 2021 08:40:47 -0700
Subject: [PATCH 04/20] Move user templates into more subdirectories

---
 bookwyrm/templates/goal.html                           |  2 +-
 .../templates/user/{user_layout.html => layout.html}   |  0
 .../templates/user/{ => relationships}/followers.html  |  2 +-
 .../templates/user/{ => relationships}/following.html  |  2 +-
 bookwyrm/templates/user/{ => shelf}/books_header.html  |  0
 .../templates/user/{ => shelf}/create_shelf_form.html  |  0
 .../templates/user/{ => shelf}/edit_shelf_form.html    |  0
 bookwyrm/templates/user/{ => shelf}/shelf.html         | 10 +++++-----
 bookwyrm/templates/user/user.html                      |  8 ++++----
 bookwyrm/views/shelf.py                                |  2 +-
 bookwyrm/views/user.py                                 |  4 ++--
 11 files changed, 15 insertions(+), 15 deletions(-)
 rename bookwyrm/templates/user/{user_layout.html => layout.html} (100%)
 rename bookwyrm/templates/user/{ => relationships}/followers.html (96%)
 rename bookwyrm/templates/user/{ => relationships}/following.html (96%)
 rename bookwyrm/templates/user/{ => shelf}/books_header.html (100%)
 rename bookwyrm/templates/user/{ => shelf}/create_shelf_form.html (100%)
 rename bookwyrm/templates/user/{ => shelf}/edit_shelf_form.html (100%)
 rename bookwyrm/templates/user/{ => shelf}/shelf.html (94%)

diff --git a/bookwyrm/templates/goal.html b/bookwyrm/templates/goal.html
index 32b5062dc..22aba08e4 100644
--- a/bookwyrm/templates/goal.html
+++ b/bookwyrm/templates/goal.html
@@ -1,4 +1,4 @@
-{% extends 'user/user_layout.html' %}
+{% extends 'user/layout.html' %}
 {% load i18n %}
 
 {% block header %}
diff --git a/bookwyrm/templates/user/user_layout.html b/bookwyrm/templates/user/layout.html
similarity index 100%
rename from bookwyrm/templates/user/user_layout.html
rename to bookwyrm/templates/user/layout.html
diff --git a/bookwyrm/templates/user/followers.html b/bookwyrm/templates/user/relationships/followers.html
similarity index 96%
rename from bookwyrm/templates/user/followers.html
rename to bookwyrm/templates/user/relationships/followers.html
index 2a7ccd1e9..c69416dce 100644
--- a/bookwyrm/templates/user/followers.html
+++ b/bookwyrm/templates/user/relationships/followers.html
@@ -1,4 +1,4 @@
-{% extends 'user/user_layout.html' %}
+{% extends 'user/layout.html' %}
 {% load i18n %}
 {% load bookwyrm_tags %}
 
diff --git a/bookwyrm/templates/user/following.html b/bookwyrm/templates/user/relationships/following.html
similarity index 96%
rename from bookwyrm/templates/user/following.html
rename to bookwyrm/templates/user/relationships/following.html
index b41d51264..a1feeb76a 100644
--- a/bookwyrm/templates/user/following.html
+++ b/bookwyrm/templates/user/relationships/following.html
@@ -1,4 +1,4 @@
-{% extends 'user/user_layout.html' %}
+{% extends 'user/layout.html' %}
 {% load i18n %}
 {% load bookwyrm_tags %}
 
diff --git a/bookwyrm/templates/user/books_header.html b/bookwyrm/templates/user/shelf/books_header.html
similarity index 100%
rename from bookwyrm/templates/user/books_header.html
rename to bookwyrm/templates/user/shelf/books_header.html
diff --git a/bookwyrm/templates/user/create_shelf_form.html b/bookwyrm/templates/user/shelf/create_shelf_form.html
similarity index 100%
rename from bookwyrm/templates/user/create_shelf_form.html
rename to bookwyrm/templates/user/shelf/create_shelf_form.html
diff --git a/bookwyrm/templates/user/edit_shelf_form.html b/bookwyrm/templates/user/shelf/edit_shelf_form.html
similarity index 100%
rename from bookwyrm/templates/user/edit_shelf_form.html
rename to bookwyrm/templates/user/shelf/edit_shelf_form.html
diff --git a/bookwyrm/templates/user/shelf.html b/bookwyrm/templates/user/shelf/shelf.html
similarity index 94%
rename from bookwyrm/templates/user/shelf.html
rename to bookwyrm/templates/user/shelf/shelf.html
index 336162588..93f0c15f2 100644
--- a/bookwyrm/templates/user/shelf.html
+++ b/bookwyrm/templates/user/shelf/shelf.html
@@ -1,16 +1,16 @@
-{% extends 'user/user_layout.html' %}
+{% extends 'user/layout.html' %}
 {% load bookwyrm_tags %}
 {% load humanize %}
 {% load i18n %}
 
 {% block title %}
-{% include 'user/books_header.html' %}
+{% include 'user/shelf/books_header.html' %}
 {% endblock %}
 
 {% block header %}
 <header class="columns">
     <h1 class="title">
-        {% include 'user/books_header.html' %}
+        {% include 'user/shelf/books_header.html' %}
     </h1>
 </header>
 {% endblock %}
@@ -41,7 +41,7 @@
 </div>
 
 <div class="block">
-    {% include 'user/create_shelf_form.html' with controls_text='create-shelf-form' %}
+    {% include 'user/shelf/create_shelf_form.html' with controls_text='create-shelf-form' %}
 </div>
 
 <div class="block columns is-mobile">
@@ -62,7 +62,7 @@
 </div>
 
 <div class="block">
-    {% include 'user/edit_shelf_form.html' with controls_text="edit-shelf-form" %}
+    {% include 'user/shelf/edit_shelf_form.html' with controls_text="edit-shelf-form" %}
 </div>
 
 <div class="block">
diff --git a/bookwyrm/templates/user/user.html b/bookwyrm/templates/user/user.html
index a97d26ab8..fead8fcf6 100644
--- a/bookwyrm/templates/user/user.html
+++ b/bookwyrm/templates/user/user.html
@@ -1,4 +1,4 @@
-{% extends 'user/user_layout.html' %}
+{% extends 'user/layout.html' %}
 {% load i18n %}
 {% load bookwyrm_tags %}
 
@@ -11,7 +11,7 @@
     </div>
     {% if is_self %}
     <div class="column is-narrow">
-        <a href="/preferences/profile">
+        <a href="{% url 'prefs-profile' %}">
             <span class="icon icon-pencil" title="Edit profile">
                 <span class="is-sr-only">{% trans "Edit profile" %}</span>
             </span>
@@ -25,7 +25,7 @@
 {% if user.bookwyrm_user %}
 <div class="block">
     <h2 class="title">
-        {% include 'user/books_header.html' %}
+        {% include 'user/shelf/books_header.html' %}
     </h2>
     <div class="columns">
         {% for shelf in shelves %}
@@ -56,7 +56,7 @@
 {% elif user == request.user %}
 <div class="block">
     {% now 'Y' as year %}
-    <h2 class="title is-4"><a href="{{ user.local_path }}/goal/{{ year }}">{% blocktrans %}Set a reading goal for {{ year }}{% endblocktrans %}</a></h2>
+    <h2 class="title is-4"><a href="{% url 'user-goal' user.localname year %}">{% blocktrans %}Set a reading goal for {{ year }}{% endblocktrans %}</a></h2>
 </div>
 {% endif %}
 
diff --git a/bookwyrm/views/shelf.py b/bookwyrm/views/shelf.py
index 9bcf0a4ac..ca05ec1ee 100644
--- a/bookwyrm/views/shelf.py
+++ b/bookwyrm/views/shelf.py
@@ -68,7 +68,7 @@ class Shelf(View):
             "books": paginated.get_page(request.GET.get("page")),
         }
 
-        return TemplateResponse(request, "user/shelf.html", data)
+        return TemplateResponse(request, "user/shelf/shelf.html", data)
 
     @method_decorator(login_required, name="dispatch")
     # pylint: disable=unused-argument
diff --git a/bookwyrm/views/user.py b/bookwyrm/views/user.py
index d394c1d73..af31fd47f 100644
--- a/bookwyrm/views/user.py
+++ b/bookwyrm/views/user.py
@@ -112,7 +112,7 @@ class Followers(View):
             "is_self": request.user.id == user.id,
             "followers": paginated.page(request.GET.get("page", 1)),
         }
-        return TemplateResponse(request, "user/followers.html", data)
+        return TemplateResponse(request, "user/relationships/followers.html", data)
 
 
 class Following(View):
@@ -138,7 +138,7 @@ class Following(View):
             "is_self": request.user.id == user.id,
             "following": paginated.page(request.GET.get("page", 1)),
         }
-        return TemplateResponse(request, "user/following.html", data)
+        return TemplateResponse(request, "user/relationships/following.html", data)
 
 
 @method_decorator(login_required, name="dispatch")

From e4cecf2874103890be40f376cb399e2e21e7036c Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Fri, 30 Apr 2021 09:02:14 -0700
Subject: [PATCH 05/20] Coherent logic on which tabs show in the user page

---
 bookwyrm/templates/user/layout.html      | 4 ++--
 bookwyrm/templates/user/lists.html       | 2 +-
 bookwyrm/templates/user/shelf/shelf.html | 4 +++-
 3 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/bookwyrm/templates/user/layout.html b/bookwyrm/templates/user/layout.html
index 0c3e94e3c..0f04f6288 100644
--- a/bookwyrm/templates/user/layout.html
+++ b/bookwyrm/templates/user/layout.html
@@ -41,8 +41,8 @@
     </div>
     {% endif %}
 </div>
+{% block tabs %}
 {% with user|username as username %}
-{% if 'user/'|add:username|add:'/books' not in request.path and 'user/'|add:username|add:'/shelf' not in request.path %}
 <nav class="tabs">
     <ul>
         {% url 'user-feed' user|username as url %}
@@ -70,8 +70,8 @@
         {% endif %}
     </ul>
 </nav>
-{% endif %}
 {% endwith %}
+{% endblock %}
 
 {% block panel %}{% endblock %}
 
diff --git a/bookwyrm/templates/user/lists.html b/bookwyrm/templates/user/lists.html
index b2fd9eefd..aa8ad7fb6 100644
--- a/bookwyrm/templates/user/lists.html
+++ b/bookwyrm/templates/user/lists.html
@@ -1,4 +1,4 @@
-{% extends 'user/user_layout.html' %}
+{% extends 'user/layout.html' %}
 {% load i18n %}
 
 {% block header %}
diff --git a/bookwyrm/templates/user/shelf/shelf.html b/bookwyrm/templates/user/shelf/shelf.html
index 93f0c15f2..639ab502f 100644
--- a/bookwyrm/templates/user/shelf/shelf.html
+++ b/bookwyrm/templates/user/shelf/shelf.html
@@ -15,7 +15,7 @@
 </header>
 {% endblock %}
 
-{% block panel %}
+{% block tabs %}
 <div class="block columns">
     <div class="column">
         <div class="tabs">
@@ -39,7 +39,9 @@
     </div>
     {% endif %}
 </div>
+{% endblock %}
 
+{% block panel %}
 <div class="block">
     {% include 'user/shelf/create_shelf_form.html' with controls_text='create-shelf-form' %}
 </div>

From 418e656aea8d5faf1b84afe27337891324f22f0c Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Fri, 30 Apr 2021 09:17:46 -0700
Subject: [PATCH 06/20] Uses layout for followers/following page

---
 bookwyrm/templates/user/layout.html           |  6 +++-
 .../user/relationships/followers.html         | 30 ++++---------------
 .../user/relationships/following.html         | 30 ++++---------------
 .../templates/user/relationships/layout.html  | 29 ++++++++++++++++++
 bookwyrm/views/user.py                        |  6 ++--
 5 files changed, 47 insertions(+), 54 deletions(-)
 create mode 100644 bookwyrm/templates/user/relationships/layout.html

diff --git a/bookwyrm/templates/user/layout.html b/bookwyrm/templates/user/layout.html
index 0f04f6288..dcdf7ccfc 100644
--- a/bookwyrm/templates/user/layout.html
+++ b/bookwyrm/templates/user/layout.html
@@ -7,7 +7,11 @@
 
 {% block content %}
 <header class="block">
-    {% block header %}{% endblock %}
+    {% block header %}
+    <h1 class="title">
+        {% trans "User Profile" %}
+    </h1>
+    {% endblock %}
 </header>
 
 {# user bio #}
diff --git a/bookwyrm/templates/user/relationships/followers.html b/bookwyrm/templates/user/relationships/followers.html
index c69416dce..223f38c7b 100644
--- a/bookwyrm/templates/user/relationships/followers.html
+++ b/bookwyrm/templates/user/relationships/followers.html
@@ -1,34 +1,14 @@
-{% extends 'user/layout.html' %}
+{% extends 'user/relationships/layout.html' %}
 {% load i18n %}
-{% load bookwyrm_tags %}
 
 {% block header %}
 <h1 class="title">
-    {% trans "User Profile" %}
+    {% trans "Followers" %}
 </h1>
 {% endblock %}
 
-{% block panel %}
-<div class="block">
-    <h2 class="title">{% trans "Followers" %}</h2>
-    {% for follower in followers %}
-    <div class="block columns">
-        <div class="column">
-            <a href="{{ follower.local_path }}">
-                {% include 'snippets/avatar.html' with user=follower %}
-                {{ follower.display_name }}
-            </a>
-            ({{ follower.username }})
-        </div>
-        <div class="column is-narrow">
-            {% include 'snippets/follow_button.html' with user=follower %}
-        </div>
-    </div>
-    {% endfor %}
-    {% if not followers %}
-    <div>{% blocktrans with username=user.display_name %}{{ username }} has no followers{% endblocktrans %}</div>
-    {% endif %}
+{% block nullstate %}
+<div>
+    {% blocktrans with username=user.display_name %}{{ username }} has no followers{% endblocktrans %}
 </div>
-
-{% include 'snippets/pagination.html' with page=followers path=request.path %}
 {% endblock %}
diff --git a/bookwyrm/templates/user/relationships/following.html b/bookwyrm/templates/user/relationships/following.html
index a1feeb76a..5689bc613 100644
--- a/bookwyrm/templates/user/relationships/following.html
+++ b/bookwyrm/templates/user/relationships/following.html
@@ -1,34 +1,14 @@
-{% extends 'user/layout.html' %}
+{% extends 'user/relationships/layout.html' %}
 {% load i18n %}
-{% load bookwyrm_tags %}
 
 {% block header %}
 <h1 class="title">
-    {% trans "User Profile" %}
+    {% trans "Following" %}
 </h1>
 {% endblock %}
 
-{% block panel %}
-<div class="block">
-    <h2 class="title">{% trans "Following" %}</h2>
-    {% for follower in user.following.all %}
-    <div class="block columns">
-        <div class="column">
-            <a href="{{ follower.local_path }}">
-                {% include 'snippets/avatar.html' with user=follower %}
-                {{ follower.display_name }}
-            </a>
-            ({{ follower.username }})
-        </div>
-        <div class="column is-narrow">
-            {% include 'snippets/follow_button.html' with user=follower %}
-        </div>
-    </div>
-    {% endfor %}
-    {% if not following %}
-    <div>{% blocktrans with username=user|username %}{{ username }} isn't following any users{% endblocktrans %}</div>
-    {% endif %}
+{% block nullstate %}
+<div>
+    {% blocktrans with username=user.display_name %}{{ username }} isn't following any users{% endblocktrans %}
 </div>
-
-{% include 'snippets/pagination.html' with page=following path=request.path %}
 {% endblock %}
diff --git a/bookwyrm/templates/user/relationships/layout.html b/bookwyrm/templates/user/relationships/layout.html
new file mode 100644
index 000000000..ab38e6620
--- /dev/null
+++ b/bookwyrm/templates/user/relationships/layout.html
@@ -0,0 +1,29 @@
+{% extends 'user/layout.html' %}
+{% load i18n %}
+{% load bookwyrm_tags %}
+
+{% block panel %}
+<div class="block">
+    {% for follow in follow_list %}
+    <div class="block columns">
+        <div class="column">
+            <a href="{{ follower.local_path }}">
+                {% include 'snippets/avatar.html' with user=follow %}
+                {{ follow.display_name }}
+            </a>
+            ({{ follow.username }})
+        </div>
+        <div class="column is-narrow">
+            {% include 'snippets/follow_button.html' with user=follow %}
+        </div>
+    </div>
+    {% endfor %}
+
+    {% if not follow_list %}
+        {% block nullstate %}
+        {% endblock %}
+    {% endif %}
+</div>
+
+{% include 'snippets/pagination.html' with page=follow_list path=request.path %}
+{% endblock %}
diff --git a/bookwyrm/views/user.py b/bookwyrm/views/user.py
index af31fd47f..5584ce991 100644
--- a/bookwyrm/views/user.py
+++ b/bookwyrm/views/user.py
@@ -110,7 +110,7 @@ class Followers(View):
         data = {
             "user": user,
             "is_self": request.user.id == user.id,
-            "followers": paginated.page(request.GET.get("page", 1)),
+            "follow_list": paginated.page(request.GET.get("page", 1)),
         }
         return TemplateResponse(request, "user/relationships/followers.html", data)
 
@@ -132,11 +132,11 @@ class Following(View):
         if is_api_request(request):
             return ActivitypubResponse(user.to_following_activity(**request.GET))
 
-        paginated = Paginator(user.followers.all(), PAGE_LENGTH)
+        paginated = Paginator(user.following.all(), PAGE_LENGTH)
         data = {
             "user": user,
             "is_self": request.user.id == user.id,
-            "following": paginated.page(request.GET.get("page", 1)),
+            "follow_list": paginated.page(request.GET.get("page", 1)),
         }
         return TemplateResponse(request, "user/relationships/following.html", data)
 

From b65d0d05c9f49c375f32f8252d33a091aac3f0aa Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Fri, 30 Apr 2021 09:23:59 -0700
Subject: [PATCH 07/20] Tabs for relationship views

---
 bookwyrm/templates/user/layout.html             |  1 +
 .../templates/user/relationships/layout.html    | 17 +++++++++++++++++
 2 files changed, 18 insertions(+)

diff --git a/bookwyrm/templates/user/layout.html b/bookwyrm/templates/user/layout.html
index dcdf7ccfc..661d80781 100644
--- a/bookwyrm/templates/user/layout.html
+++ b/bookwyrm/templates/user/layout.html
@@ -45,6 +45,7 @@
     </div>
     {% endif %}
 </div>
+
 {% block tabs %}
 {% with user|username as username %}
 <nav class="tabs">
diff --git a/bookwyrm/templates/user/relationships/layout.html b/bookwyrm/templates/user/relationships/layout.html
index ab38e6620..e83e53471 100644
--- a/bookwyrm/templates/user/relationships/layout.html
+++ b/bookwyrm/templates/user/relationships/layout.html
@@ -2,6 +2,23 @@
 {% load i18n %}
 {% load bookwyrm_tags %}
 
+{% block tabs %}
+{% with user|username as username %}
+<nav class="tabs">
+    <ul>
+        {% url 'user-following' user|username as url %}
+        <li{% if url == request.path or url == request.path|add:'/' %} class="is-active"{% endif %}>
+            <a href="{{ url }}">{% trans "Following" %}</a>
+        </li>
+        {% url 'user-followers' user|username as url %}
+        <li{% if url == request.path or url == request.path|add:'/' %} class="is-active"{% endif %}>
+            <a href="{{ url }}">{% trans "Followers" %}</a>
+        </li>
+    </ul>
+</nav>
+{% endwith %}
+{% endblock %}
+
 {% block panel %}
 <div class="block">
     {% for follow in follow_list %}

From d810d1bb8c7a149488a60ab53f1621f24caab003 Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Fri, 30 Apr 2021 09:26:02 -0700
Subject: [PATCH 08/20] Remove duplicate check for user blocks

---
 bookwyrm/views/user.py | 14 +-------------
 1 file changed, 1 insertion(+), 13 deletions(-)

diff --git a/bookwyrm/views/user.py b/bookwyrm/views/user.py
index 5584ce991..1cf0b7037 100644
--- a/bookwyrm/views/user.py
+++ b/bookwyrm/views/user.py
@@ -17,7 +17,7 @@ from bookwyrm import forms, models
 from bookwyrm.activitypub import ActivitypubResponse
 from bookwyrm.settings import PAGE_LENGTH
 from .helpers import get_user_from_username, is_api_request
-from .helpers import is_blocked, privacy_filter
+from .helpers import privacy_filter
 
 
 # pylint: disable= no-self-use
@@ -31,10 +31,6 @@ class User(View):
         except models.User.DoesNotExist:
             return HttpResponseNotFound()
 
-        # make sure we're not blocked
-        if is_blocked(request.user, user):
-            return HttpResponseNotFound()
-
         if is_api_request(request):
             # we have a json request
             return ActivitypubResponse(user.to_activity())
@@ -99,10 +95,6 @@ class Followers(View):
         except models.User.DoesNotExist:
             return HttpResponseNotFound()
 
-        # make sure we're not blocked
-        if is_blocked(request.user, user):
-            return HttpResponseNotFound()
-
         if is_api_request(request):
             return ActivitypubResponse(user.to_followers_activity(**request.GET))
 
@@ -125,10 +117,6 @@ class Following(View):
         except models.User.DoesNotExist:
             return HttpResponseNotFound()
 
-        # make sure we're not blocked
-        if is_blocked(request.user, user):
-            return HttpResponseNotFound()
-
         if is_api_request(request):
             return ActivitypubResponse(user.to_following_activity(**request.GET))
 

From 4dacf4df3a42f24b28a3a7649cd1fb4cdad97e3a Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Fri, 30 Apr 2021 09:33:36 -0700
Subject: [PATCH 09/20] Refactor get_user_from_username to raise 404 directly

---
 bookwyrm/tests/views/test_helpers.py |  3 ++-
 bookwyrm/views/feed.py               |  6 +++---
 bookwyrm/views/follow.py             | 20 ++++----------------
 bookwyrm/views/helpers.py            |  9 ++++++++-
 bookwyrm/views/shelf.py              |  5 +----
 bookwyrm/views/user.py               | 16 +++-------------
 6 files changed, 21 insertions(+), 38 deletions(-)

diff --git a/bookwyrm/tests/views/test_helpers.py b/bookwyrm/tests/views/test_helpers.py
index 0dddd2a15..ab5abed0b 100644
--- a/bookwyrm/tests/views/test_helpers.py
+++ b/bookwyrm/tests/views/test_helpers.py
@@ -3,6 +3,7 @@ import json
 from unittest.mock import patch
 import pathlib
 from django.db.models import Q
+from django.http import Http404
 from django.test import TestCase
 from django.test.client import RequestFactory
 import responses
@@ -67,7 +68,7 @@ class ViewsHelpers(TestCase):
             views.helpers.get_user_from_username(self.local_user, "mouse@local.com"),
             self.local_user,
         )
-        with self.assertRaises(models.User.DoesNotExist):
+        with self.assertRaises(Http404):
             views.helpers.get_user_from_username(self.local_user, "mojfse@example.com")
 
     def test_is_api_request(self, _):
diff --git a/bookwyrm/views/feed.py b/bookwyrm/views/feed.py
index 98f365ea4..a03d6b4cc 100644
--- a/bookwyrm/views/feed.py
+++ b/bookwyrm/views/feed.py
@@ -2,7 +2,7 @@
 from django.contrib.auth.decorators import login_required
 from django.core.paginator import Paginator
 from django.db.models import Q
-from django.http import HttpResponseNotFound
+from django.http import HttpResponseNotFound, Http404
 from django.template.response import TemplateResponse
 from django.utils import timezone
 from django.utils.decorators import method_decorator
@@ -62,7 +62,7 @@ class DirectMessage(View):
         if username:
             try:
                 user = get_user_from_username(request.user, username)
-            except models.User.DoesNotExist:
+            except Http404:
                 pass
         if user:
             queryset = queryset.filter(Q(user=user) | Q(mention_users=user))
@@ -94,7 +94,7 @@ class Status(View):
             status = models.Status.objects.select_subclasses().get(
                 id=status_id, deleted=False
             )
-        except (ValueError, models.Status.DoesNotExist, models.User.DoesNotExist):
+        except (ValueError, models.Status.DoesNotExist):
             return HttpResponseNotFound()
 
         # the url should have the poster's username in it
diff --git a/bookwyrm/views/follow.py b/bookwyrm/views/follow.py
index 09c2d53af..c3336a94a 100644
--- a/bookwyrm/views/follow.py
+++ b/bookwyrm/views/follow.py
@@ -14,10 +14,7 @@ from .helpers import get_user_from_username
 def follow(request):
     """follow another user, here or abroad"""
     username = request.POST["user"]
-    try:
-        to_follow = get_user_from_username(request.user, username)
-    except models.User.DoesNotExist:
-        return HttpResponseBadRequest()
+    to_follow = get_user_from_username(request.user, username)
 
     try:
         models.UserFollowRequest.objects.create(
@@ -35,10 +32,7 @@ def follow(request):
 def unfollow(request):
     """unfollow a user"""
     username = request.POST["user"]
-    try:
-        to_unfollow = get_user_from_username(request.user, username)
-    except models.User.DoesNotExist:
-        return HttpResponseBadRequest()
+    to_unfollow = get_user_from_username(request.user, username)
 
     try:
         models.UserFollows.objects.get(
@@ -63,10 +57,7 @@ def unfollow(request):
 def accept_follow_request(request):
     """a user accepts a follow request"""
     username = request.POST["user"]
-    try:
-        requester = get_user_from_username(request.user, username)
-    except models.User.DoesNotExist:
-        return HttpResponseBadRequest()
+    requester = get_user_from_username(request.user, username)
 
     try:
         follow_request = models.UserFollowRequest.objects.get(
@@ -85,10 +76,7 @@ def accept_follow_request(request):
 def delete_follow_request(request):
     """a user rejects a follow request"""
     username = request.POST["user"]
-    try:
-        requester = get_user_from_username(request.user, username)
-    except models.User.DoesNotExist:
-        return HttpResponseBadRequest()
+    requester = get_user_from_username(request.user, username)
 
     try:
         follow_request = models.UserFollowRequest.objects.get(
diff --git a/bookwyrm/views/helpers.py b/bookwyrm/views/helpers.py
index 5a8617fca..31e5f0317 100644
--- a/bookwyrm/views/helpers.py
+++ b/bookwyrm/views/helpers.py
@@ -3,6 +3,7 @@ import re
 from requests import HTTPError
 from django.core.exceptions import FieldError
 from django.db.models import Count, Max, Q
+from django.http import Http404
 
 from bookwyrm import activitypub, models
 from bookwyrm.connectors import ConnectorException, get_data
@@ -12,11 +13,17 @@ from bookwyrm.utils import regex
 
 def get_user_from_username(viewer, username):
     """helper function to resolve a localname or a username to a user"""
-    # raises DoesNotExist if user is now found
+    # raises 404 if the user isn't found
     try:
         return models.User.viewer_aware_objects(viewer).get(localname=username)
     except models.User.DoesNotExist:
+        pass
+
+    # if the localname didn't match, try the username
+    try:
         return models.User.viewer_aware_objects(viewer).get(username=username)
+    except models.User.DoesNotExist:
+        raise Http404()
 
 
 def is_api_request(request):
diff --git a/bookwyrm/views/shelf.py b/bookwyrm/views/shelf.py
index ca05ec1ee..94507d46e 100644
--- a/bookwyrm/views/shelf.py
+++ b/bookwyrm/views/shelf.py
@@ -25,10 +25,7 @@ class Shelf(View):
 
     def get(self, request, username, shelf_identifier=None):
         """display a shelf"""
-        try:
-            user = get_user_from_username(request.user, username)
-        except models.User.DoesNotExist:
-            return HttpResponseNotFound()
+        user = get_user_from_username(request.user, username)
 
         shelves = privacy_filter(request.user, user.shelf_set)
 
diff --git a/bookwyrm/views/user.py b/bookwyrm/views/user.py
index 1cf0b7037..3f77e962d 100644
--- a/bookwyrm/views/user.py
+++ b/bookwyrm/views/user.py
@@ -6,7 +6,6 @@ from PIL import Image
 from django.contrib.auth.decorators import login_required
 from django.core.files.base import ContentFile
 from django.core.paginator import Paginator
-from django.http import HttpResponseNotFound
 from django.shortcuts import redirect
 from django.template.response import TemplateResponse
 from django.utils import timezone
@@ -26,10 +25,7 @@ class User(View):
 
     def get(self, request, username):
         """profile page for a user"""
-        try:
-            user = get_user_from_username(request.user, username)
-        except models.User.DoesNotExist:
-            return HttpResponseNotFound()
+        user = get_user_from_username(request.user, username)
 
         if is_api_request(request):
             # we have a json request
@@ -90,10 +86,7 @@ class Followers(View):
 
     def get(self, request, username):
         """list of followers"""
-        try:
-            user = get_user_from_username(request.user, username)
-        except models.User.DoesNotExist:
-            return HttpResponseNotFound()
+        user = get_user_from_username(request.user, username)
 
         if is_api_request(request):
             return ActivitypubResponse(user.to_followers_activity(**request.GET))
@@ -112,10 +105,7 @@ class Following(View):
 
     def get(self, request, username):
         """list of followers"""
-        try:
-            user = get_user_from_username(request.user, username)
-        except models.User.DoesNotExist:
-            return HttpResponseNotFound()
+        user = get_user_from_username(request.user, username)
 
         if is_api_request(request):
             return ActivitypubResponse(user.to_following_activity(**request.GET))

From 62884c61115648d9f27222044291687f5223d579 Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Fri, 30 Apr 2021 09:48:10 -0700
Subject: [PATCH 10/20] Show mutual counts instead of totals for other users

---
 .../templates/user/relationships/layout.html     |  8 ++++----
 bookwyrm/templates/user/user_preview.html        | 16 ++++++++++++++--
 bookwyrm/templatetags/bookwyrm_tags.py           |  7 +++++++
 3 files changed, 25 insertions(+), 6 deletions(-)

diff --git a/bookwyrm/templates/user/relationships/layout.html b/bookwyrm/templates/user/relationships/layout.html
index e83e53471..5baaf1f96 100644
--- a/bookwyrm/templates/user/relationships/layout.html
+++ b/bookwyrm/templates/user/relationships/layout.html
@@ -6,14 +6,14 @@
 {% with user|username as username %}
 <nav class="tabs">
     <ul>
-        {% url 'user-following' user|username as url %}
-        <li{% if url == request.path or url == request.path|add:'/' %} class="is-active"{% endif %}>
-            <a href="{{ url }}">{% trans "Following" %}</a>
-        </li>
         {% url 'user-followers' user|username as url %}
         <li{% if url == request.path or url == request.path|add:'/' %} class="is-active"{% endif %}>
             <a href="{{ url }}">{% trans "Followers" %}</a>
         </li>
+        {% url 'user-following' user|username as url %}
+        <li{% if url == request.path or url == request.path|add:'/' %} class="is-active"{% endif %}>
+            <a href="{{ url }}">{% trans "Following" %}</a>
+        </li>
     </ul>
 </nav>
 {% endwith %}
diff --git a/bookwyrm/templates/user/user_preview.html b/bookwyrm/templates/user/user_preview.html
index c641c58fb..89b545cec 100644
--- a/bookwyrm/templates/user/user_preview.html
+++ b/bookwyrm/templates/user/user_preview.html
@@ -1,5 +1,6 @@
 {% load i18n %}
 {% load humanize %}
+{% load bookwyrm_tags %}
 
 <div class="media block">
     <div class="media-left">
@@ -12,8 +13,19 @@
         <p><a href="{{ user.remote_id }}">{{ user.username }}</a></p>
         <p>{% blocktrans with date=user.created_date|naturaltime %}Joined {{ date }}{% endblocktrans %}</p>
         <p>
-        <a href="{{ user.local_path }}/followers">{% blocktrans count counter=user.followers.count %}{{ counter }} follower{% plural %}{{ counter }} followers{% endblocktrans %}</a>,
-        <a href="{{ user.local_path }}/following">{% blocktrans with counter=user.following.count %}{{ counter }} following{% endblocktrans %}</a>
+            {% if is_self %}
+
+            <a href="{% url 'user-followers' user|username %}">{% blocktrans count counter=user.followers.count %}{{ counter }} follower{% plural %}{{ counter }} followers{% endblocktrans %}</a>,
+            <a href="{% url 'user-following' user|username %}">{% blocktrans with counter=user.following.count %}{{ counter }} following{% endblocktrans %}</a>
+
+            {% else %}
+
+            {% mutuals_count user as mutuals %}
+            <a href="{% url 'user-followers' user|username %}">
+                {% blocktrans with mutuals_display=mutuals|intcomma count counter=mutuals %}{{ mutuals_display }} follower you follow{% plural %}{{ mutuals_display }} followers you follow{% endblocktrans %}
+            </a>
+
+            {% endif %}
         </p>
     </div>
 </div>
diff --git a/bookwyrm/templatetags/bookwyrm_tags.py b/bookwyrm/templatetags/bookwyrm_tags.py
index 2ed0cbc2b..2697d3ced 100644
--- a/bookwyrm/templatetags/bookwyrm_tags.py
+++ b/bookwyrm/templatetags/bookwyrm_tags.py
@@ -235,3 +235,10 @@ def get_lang():
     """get current language, strip to the first two letters"""
     language = utils.translation.get_language()
     return language[0 : language.find("-")]
+
+
+@register.simple_tag(takes_context=True)
+def mutuals_count(context, user):
+    """how many users that you follow, follow them"""
+    viewer = context["request"].user
+    return user.followers.filter(id__in=viewer.following.all()).count()

From ca12359beebea3f9882ad94094f4d0c6e0a8f008 Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Fri, 30 Apr 2021 10:16:24 -0700
Subject: [PATCH 11/20] Fixes unit tests

---
 bookwyrm/tests/views/test_user.py | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/bookwyrm/tests/views/test_user.py b/bookwyrm/tests/views/test_user.py
index 3b431de1d..244acda65 100644
--- a/bookwyrm/tests/views/test_user.py
+++ b/bookwyrm/tests/views/test_user.py
@@ -6,6 +6,7 @@ from PIL import Image
 from django.contrib.auth.models import AnonymousUser
 from django.core.files.base import ContentFile
 from django.core.files.uploadedfile import SimpleUploadedFile
+from django.http.response import Http404
 from django.template.response import TemplateResponse
 from django.test import TestCase
 from django.test.client import RequestFactory
@@ -76,8 +77,8 @@ class UserViews(TestCase):
         self.rat.blocks.add(self.local_user)
         with patch("bookwyrm.views.user.is_api_request") as is_api:
             is_api.return_value = False
-            result = view(request, "rat")
-        self.assertEqual(result.status_code, 404)
+            with self.assertRaises(Http404):
+                view(request, "rat")
 
     def test_followers_page(self):
         """there are so many views, this just makes sure it LOADS"""
@@ -105,8 +106,8 @@ class UserViews(TestCase):
         self.rat.blocks.add(self.local_user)
         with patch("bookwyrm.views.user.is_api_request") as is_api:
             is_api.return_value = False
-            result = view(request, "rat")
-        self.assertEqual(result.status_code, 404)
+            with self.assertRaises(Http404):
+                view(request, "rat")
 
     def test_following_page(self):
         """there are so many views, this just makes sure it LOADS"""
@@ -166,7 +167,6 @@ class UserViews(TestCase):
         self.assertEqual(self.local_user.name, "New Name")
         self.assertEqual(self.local_user.email, "wow@email.com")
 
-    # idk how to mock the upload form, got tired of triyng to make it work
     def test_edit_user_avatar(self):
         """use a form to update a user"""
         view = views.EditUser.as_view()

From e15d6654e3ec0b461e8955a1f005e5b53dc05673 Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Fri, 30 Apr 2021 10:42:27 -0700
Subject: [PATCH 12/20] Free text field for the site footer

---
 bookwyrm/management/commands/initdb.py         | 12 ++++++++++--
 .../0073_sitesettings_footer_item.py           | 18 ++++++++++++++++++
 bookwyrm/models/site.py                        |  9 +++++++++
 bookwyrm/templates/layout.html                 |  5 +++++
 bookwyrm/templates/settings/site.html          | 12 ++++++++----
 5 files changed, 50 insertions(+), 6 deletions(-)
 create mode 100644 bookwyrm/migrations/0073_sitesettings_footer_item.py

diff --git a/bookwyrm/management/commands/initdb.py b/bookwyrm/management/commands/initdb.py
index 9033249d4..da7dac005 100644
--- a/bookwyrm/management/commands/initdb.py
+++ b/bookwyrm/management/commands/initdb.py
@@ -1,4 +1,5 @@
-from django.core.management.base import BaseCommand, CommandError
+""" What you need in the database to make it work """
+from django.core.management.base import BaseCommand
 from django.contrib.auth.models import Group, Permission
 from django.contrib.contenttypes.models import ContentType
 
@@ -7,12 +8,14 @@ from bookwyrm.settings import DOMAIN
 
 
 def init_groups():
+    """permission levels"""
     groups = ["admin", "moderator", "editor"]
     for group in groups:
         Group.objects.create(name=group)
 
 
 def init_permissions():
+    """ permissin types """
     permissions = [
         {
             "codename": "edit_instance_settings",
@@ -69,6 +72,7 @@ def init_permissions():
 
 
 def init_connectors():
+    """access book data sources"""
     Connector.objects.create(
         identifier=DOMAIN,
         name="Local",
@@ -130,7 +134,11 @@ def init_federated_servers():
 
 
 def init_settings():
-    SiteSettings.objects.create()
+    """ info about the instance """
+    SiteSettings.objects.create(
+        support_link="https://www.patreon.com/bookwyrm",
+        support_title="Patreon",
+    )
 
 
 class Command(BaseCommand):
diff --git a/bookwyrm/migrations/0073_sitesettings_footer_item.py b/bookwyrm/migrations/0073_sitesettings_footer_item.py
new file mode 100644
index 000000000..4aa8384f0
--- /dev/null
+++ b/bookwyrm/migrations/0073_sitesettings_footer_item.py
@@ -0,0 +1,18 @@
+# Generated by Django 3.2 on 2021-04-30 17:25
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ("bookwyrm", "0072_remove_work_default_edition"),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name="sitesettings",
+            name="footer_item",
+            field=models.TextField(blank=True, null=True),
+        ),
+    ]
diff --git a/bookwyrm/models/site.py b/bookwyrm/models/site.py
index 193cffb7a..2c5a21642 100644
--- a/bookwyrm/models/site.py
+++ b/bookwyrm/models/site.py
@@ -19,19 +19,28 @@ class SiteSettings(models.Model):
         max_length=150, default="Social Reading and Reviewing"
     )
     instance_description = models.TextField(default="This instance has no description.")
+
+    # about page
     registration_closed_text = models.TextField(
         default="Contact an administrator to get an invite"
     )
     code_of_conduct = models.TextField(default="Add a code of conduct here.")
     privacy_policy = models.TextField(default="Add a privacy policy here.")
+
+    # registration
     allow_registration = models.BooleanField(default=True)
     allow_invite_requests = models.BooleanField(default=True)
+
+    # images
     logo = models.ImageField(upload_to="logos/", null=True, blank=True)
     logo_small = models.ImageField(upload_to="logos/", null=True, blank=True)
     favicon = models.ImageField(upload_to="logos/", null=True, blank=True)
+
+    # footer
     support_link = models.CharField(max_length=255, null=True, blank=True)
     support_title = models.CharField(max_length=100, null=True, blank=True)
     admin_email = models.EmailField(max_length=255, null=True, blank=True)
+    footer_item = models.TextField(null=True, blank=True)
 
     @classmethod
     def get(cls):
diff --git a/bookwyrm/templates/layout.html b/bookwyrm/templates/layout.html
index 84482cdfc..104b5a7e6 100644
--- a/bookwyrm/templates/layout.html
+++ b/bookwyrm/templates/layout.html
@@ -200,6 +200,11 @@
                 </p>
                 {% endif %}
             </div>
+            {% if site.footer_item %}
+            <div class="column">
+                <p>{{ site.footer_item|safe }}</p>
+            </div>
+            {% endif %}
             {% if site.support_link %}
             <div class="column">
                 <span class="icon icon-heart"></span>
diff --git a/bookwyrm/templates/settings/site.html b/bookwyrm/templates/settings/site.html
index 28009e1a8..1b9424527 100644
--- a/bookwyrm/templates/settings/site.html
+++ b/bookwyrm/templates/settings/site.html
@@ -37,16 +37,16 @@
 
     <section class="block" id="images">
         <h2 class="title is-4">{% trans "Images" %}</h2>
-        <div class="field is-grouped">
-            <div class="control">
+        <div class="columns">
+            <div class="column">
                 <label class="label" for="id_logo">{% trans "Logo:" %}</label>
                 {{ site_form.logo }}
             </div>
-            <div class="control">
+            <div class="column">
                 <label class="label" for="id_logo_small">{% trans "Logo small:" %}</label>
                 {{ site_form.logo_small }}
             </div>
-            <div class="control">
+            <div class="column">
                 <label class="label" for="id_favicon">{% trans "Favicon:" %}</label>
                 {{ site_form.favicon }}
             </div>
@@ -69,6 +69,10 @@
             <label class="label" for="id_admin_email">{% trans "Admin email:" %}</label>
             {{ site_form.admin_email }}
         </div>
+        <div class="control">
+            <label class="label" for="id_footer_item">{% trans "Additional info:" %}</label>
+            {{ site_form.footer_item }}
+        </div>
     </section>
 
     <hr aria-hidden="true">

From f747babb4366948352a4b904e2c28bc1b3e47473 Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Fri, 30 Apr 2021 11:03:36 -0700
Subject: [PATCH 13/20] Changes column spacing

---
 bookwyrm/templates/layout.html | 25 +++++++++++++++----------
 1 file changed, 15 insertions(+), 10 deletions(-)

diff --git a/bookwyrm/templates/layout.html b/bookwyrm/templates/layout.html
index 104b5a7e6..d208989ec 100644
--- a/bookwyrm/templates/layout.html
+++ b/bookwyrm/templates/layout.html
@@ -190,7 +190,7 @@
 <footer class="footer">
     <div class="container">
         <div class="columns">
-            <div class="column">
+            <div class="column is-one-fifth">
                 <p>
                     <a href="/about">{% trans "About this server" %}</a>
                 </p>
@@ -199,21 +199,26 @@
                     <a href="mailto:{{ site.admin_email }}">{% trans "Contact site admin" %}</a>
                 </p>
                 {% endif %}
+                <p>
+                    <a href="https://docs.joinbookwyrm.com/">{% trans "Documentation" %}</a>
+                </p>
+            </div>
+            <div class="column content is-two-fifth">
+                {% if site.support_link %}
+                <p>
+                    <span class="icon icon-heart"></span>
+                    {% blocktrans with site_name=site.name support_link=site.support_link support_title=site.support_title %}Support {{ site_name }} on <a href="{{ support_link }}" target="_blank">{{ support_title }}</a>{% endblocktrans %}
+                </p>
+                {% endif %}
+                <p>
+                    {% trans 'BookWyrm is open source software. You can contribute or report issues on <a href="https://github.com/mouse-reeve/bookwyrm">GitHub</a>.' %}
+                </p>
             </div>
             {% if site.footer_item %}
             <div class="column">
                 <p>{{ site.footer_item|safe }}</p>
             </div>
             {% endif %}
-            {% if site.support_link %}
-            <div class="column">
-                <span class="icon icon-heart"></span>
-                {% blocktrans with site_name=site.name support_link=site.support_link support_title=site.support_title %}Support {{ site_name }} on <a href="{{ support_link }}" target="_blank">{{ support_title }}</a>{% endblocktrans %}
-            </div>
-            {% endif %}
-            <div class="column">
-                {% trans 'BookWyrm is open source software. You can contribute or report issues on <a href="https://github.com/mouse-reeve/bookwyrm">GitHub</a>.' %}
-            </div>
         </div>
     </div>
 </footer>

From 00815b3105dd71bb1f3595c8538282bcf95dfc27 Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Fri, 30 Apr 2021 11:09:06 -0700
Subject: [PATCH 14/20] Python formatting

---
 bookwyrm/management/commands/initdb.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/bookwyrm/management/commands/initdb.py b/bookwyrm/management/commands/initdb.py
index da7dac005..2e88ce5d8 100644
--- a/bookwyrm/management/commands/initdb.py
+++ b/bookwyrm/management/commands/initdb.py
@@ -15,7 +15,7 @@ def init_groups():
 
 
 def init_permissions():
-    """ permissin types """
+    """permissin types"""
     permissions = [
         {
             "codename": "edit_instance_settings",
@@ -134,7 +134,7 @@ def init_federated_servers():
 
 
 def init_settings():
-    """ info about the instance """
+    """info about the instance"""
     SiteSettings.objects.create(
         support_link="https://www.patreon.com/bookwyrm",
         support_title="Patreon",

From b8fc4f99a3cdf911abe67731aac1defb1bfc9cdf Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Fri, 30 Apr 2021 11:11:18 -0700
Subject: [PATCH 15/20] Don't calculate mutuals for logged out users

---
 bookwyrm/templates/user/user_preview.html | 2 +-
 bookwyrm/templatetags/bookwyrm_tags.py    | 2 ++
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/bookwyrm/templates/user/user_preview.html b/bookwyrm/templates/user/user_preview.html
index 89b545cec..c450365bb 100644
--- a/bookwyrm/templates/user/user_preview.html
+++ b/bookwyrm/templates/user/user_preview.html
@@ -18,7 +18,7 @@
             <a href="{% url 'user-followers' user|username %}">{% blocktrans count counter=user.followers.count %}{{ counter }} follower{% plural %}{{ counter }} followers{% endblocktrans %}</a>,
             <a href="{% url 'user-following' user|username %}">{% blocktrans with counter=user.following.count %}{{ counter }} following{% endblocktrans %}</a>
 
-            {% else %}
+            {% elif request.user.is_authenticated %}
 
             {% mutuals_count user as mutuals %}
             <a href="{% url 'user-followers' user|username %}">
diff --git a/bookwyrm/templatetags/bookwyrm_tags.py b/bookwyrm/templatetags/bookwyrm_tags.py
index 2697d3ced..eda3d1023 100644
--- a/bookwyrm/templatetags/bookwyrm_tags.py
+++ b/bookwyrm/templatetags/bookwyrm_tags.py
@@ -241,4 +241,6 @@ def get_lang():
 def mutuals_count(context, user):
     """how many users that you follow, follow them"""
     viewer = context["request"].user
+    if not viewer.is_authenticated:
+        return None
     return user.followers.filter(id__in=viewer.following.all()).count()

From 4b53342fd208382d2005d95f80a3496e1c328a78 Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Fri, 30 Apr 2021 11:26:02 -0700
Subject: [PATCH 16/20] Fixes mock in unit test

---
 bookwyrm/tests/views/test_user.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/bookwyrm/tests/views/test_user.py b/bookwyrm/tests/views/test_user.py
index 244acda65..3fa78e355 100644
--- a/bookwyrm/tests/views/test_user.py
+++ b/bookwyrm/tests/views/test_user.py
@@ -135,8 +135,8 @@ class UserViews(TestCase):
         self.rat.blocks.add(self.local_user)
         with patch("bookwyrm.views.user.is_api_request") as is_api:
             is_api.return_value = False
-            result = view(request, "rat")
-        self.assertEqual(result.status_code, 404)
+            with self.assertRaises(Http404):
+                view(request, "rat")
 
     def test_edit_user_page(self):
         """there are so many views, this just makes sure it LOADS"""

From 9fea07039889c9962eb0e842dad375ebdcd78f3e Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Fri, 30 Apr 2021 12:53:57 -0700
Subject: [PATCH 17/20] Update bookwyrm/management/commands/initdb.py

Co-authored-by: Joachim <joachim.robert@protonmail.com>
---
 bookwyrm/management/commands/initdb.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/bookwyrm/management/commands/initdb.py b/bookwyrm/management/commands/initdb.py
index 2e88ce5d8..71ac511a0 100644
--- a/bookwyrm/management/commands/initdb.py
+++ b/bookwyrm/management/commands/initdb.py
@@ -15,7 +15,7 @@ def init_groups():
 
 
 def init_permissions():
-    """permissin types"""
+    """permission types"""
     permissions = [
         {
             "codename": "edit_instance_settings",

From 01f2d80cbd6a604bcb0a1b192ebc4c56c1b0e62e Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Fri, 30 Apr 2021 13:21:19 -0700
Subject: [PATCH 18/20] Show book descriptions in list items

---
 bookwyrm/templates/lists/list.html | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

diff --git a/bookwyrm/templates/lists/list.html b/bookwyrm/templates/lists/list.html
index 9243d3afb..2902f793f 100644
--- a/bookwyrm/templates/lists/list.html
+++ b/bookwyrm/templates/lists/list.html
@@ -44,8 +44,15 @@
                             </div>
 
                             <div class="column ml-3">
-                                <span>{% include 'snippets/book_titleby.html' %}</span>
-                                {% include 'snippets/stars.html' with rating=item.book|rating:request.user %}
+                                <p>
+                                    {% include 'snippets/book_titleby.html' %}
+                                </p>
+                                <p>
+                                    {% include 'snippets/stars.html' with rating=item.book|rating:request.user %}
+                                </p>
+                                <p>
+                                    {{ book|book_description|to_markdown|default:""|safe|truncatewords_html:20 }}
+                                </p>
                                 {% include 'snippets/shelve_button/shelve_button.html' %}
                             </div>
                         </div>

From c373a0b818dfea5ebe8a7aa1e9ce1c409c68bf3d Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Fri, 30 Apr 2021 13:38:03 -0700
Subject: [PATCH 19/20] Highlight report notifications

---
 bookwyrm/models/user.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/bookwyrm/models/user.py b/bookwyrm/models/user.py
index 2c5bef4ae..7c943bec9 100644
--- a/bookwyrm/models/user.py
+++ b/bookwyrm/models/user.py
@@ -160,7 +160,7 @@ class User(OrderedCollectionPageMixin, AbstractUser):
         """whether any of the unread notifications are conversations"""
         return self.notification_set.filter(
             read=False,
-            notification_type__in=["REPLY", "MENTION", "TAG"],
+            notification_type__in=["REPLY", "MENTION", "TAG", "REPORT"],
         ).exists()
 
     activity_serializer = activitypub.Person

From 544e29fe76754bdaa1589bac7288cddc9d7b3d3c Mon Sep 17 00:00:00 2001
From: Fabien Basmaison <contact@arkhi.org>
Date: Fri, 30 Apr 2021 22:44:53 +0200
Subject: [PATCH 20/20] Fix lists layout for user.

---
 bookwyrm/templates/user/lists.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/bookwyrm/templates/user/lists.html b/bookwyrm/templates/user/lists.html
index b2fd9eefd..8fe149d58 100644
--- a/bookwyrm/templates/user/lists.html
+++ b/bookwyrm/templates/user/lists.html
@@ -23,7 +23,7 @@
 
 
 {% block panel %}
-<section class="block content">
+<section class="block">
     <form name="create-list" method="post" action="{% url 'lists' %}" class="box is-hidden" id="create-list">
         <header class="columns">
             <h3 class="title column">{% trans "Create list" %}</h3>