From fb2eea956ef7416eb755ebecf58db7f9a57215c4 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Gabriel=20Rodr=C3=ADguez=20Alberich?= Date: Sat, 17 Dec 2022 01:06:29 +0100 Subject: [PATCH] Several pagination improvements (#170) Home/Notification gets pagination, Follows becomes ListView --- activities/views/follows.py | 46 ++++++++++++++----------- activities/views/timelines.py | 18 ++++++---- static/css/style.css | 6 ++++ takahe/urls.py | 2 +- templates/activities/follows.html | 16 +++++++-- templates/activities/home.html | 12 ++++++- templates/activities/local.html | 12 +++++-- templates/activities/notifications.html | 12 +++++-- templates/activities/tag.html | 12 +++++-- 9 files changed, 95 insertions(+), 41 deletions(-) diff --git a/activities/views/follows.py b/activities/views/follows.py index 44d8adc..6b0881a 100644 --- a/activities/views/follows.py +++ b/activities/views/follows.py @@ -1,33 +1,39 @@ +from django.db.models import Q from django.utils.decorators import method_decorator -from django.views.generic import TemplateView +from django.views.generic import ListView from users.decorators import identity_required -from users.models import FollowStates +from users.models import Follow, FollowStates @method_decorator(identity_required, name="dispatch") -class FollowsPage(TemplateView): +class Follows(ListView): """ Shows followers/follows. """ template_name = "activities/follows.html" + extra_context = { + "section": "follows", + } + paginate_by = 50 + + def get_queryset(self): + return Follow.objects.filter( + Q(source=self.request.identity) | Q(target=self.request.identity), + state__in=FollowStates.group_active(), + ).order_by("-created") def get_context_data(self): - # Gather all identities with a following relationship with us - identities = {} - for outbound_follow in self.request.identity.outbound_follows.filter( - state__in=FollowStates.group_active() - ): - identities.setdefault(outbound_follow.target, {})[ - "outbound" - ] = outbound_follow - for inbound_follow in self.request.identity.inbound_follows.filter( - state__in=FollowStates.group_active() - ): - identities.setdefault(inbound_follow.source, {})["inbound"] = inbound_follow - - return { - "section": "follows", - "identities": sorted(identities.items(), key=lambda i: i[0].username), - } + context = super().get_context_data() + identities = [] + for follow in context["page_obj"].object_list: + if follow.source == self.request.identity: + identity = follow.target + follow_type = "outbound" + else: + identity = follow.source + follow_type = "inbound" + identities.append((identity, follow_type)) + context["page_obj"].object_list = identities + return context diff --git a/activities/views/timelines.py b/activities/views/timelines.py index d797cfd..84e490f 100644 --- a/activities/views/timelines.py +++ b/activities/views/timelines.py @@ -1,3 +1,4 @@ +from django.core.paginator import Paginator from django.shortcuts import get_object_or_404, redirect from django.template.defaultfilters import linebreaks_filter from django.utils.decorators import method_decorator @@ -22,20 +23,23 @@ class Home(FormView): def get_context_data(self): context = super().get_context_data() - context["events"] = list( + events = ( TimelineEvent.objects.filter( identity=self.request.identity, type__in=[TimelineEvent.Types.post, TimelineEvent.Types.boost], ) .select_related("subject_post", "subject_post__author") .prefetch_related("subject_post__attachments", "subject_post__mentions") - .order_by("-created")[:50] + .order_by("-created") ) context["interactions"] = PostInteraction.get_event_interactions( - context["events"], self.request.identity + events, self.request.identity ) context["current_page"] = "home" context["allows_refresh"] = True + paginator = Paginator(events, 50) + page_number = self.request.GET.get("page") + context["page_obj"] = paginator.get_page(page_number) return context def form_valid(self, form): @@ -74,7 +78,7 @@ class Tag(ListView): .tagged_with(self.hashtag) .select_related("author") .prefetch_related("attachments", "mentions") - .order_by("-created")[:50] + .order_by("-created") ) def get_context_data(self): @@ -103,7 +107,7 @@ class Local(ListView): Post.objects.local_public() .select_related("author", "author__domain") .prefetch_related("attachments", "mentions", "emojis") - .order_by("-created")[:50] + .order_by("-created") ) def get_context_data(self): @@ -131,7 +135,7 @@ class Federated(ListView): ) .select_related("author", "author__domain") .prefetch_related("attachments", "mentions", "emojis") - .order_by("-created")[:50] + .order_by("-created") ) def get_context_data(self): @@ -175,7 +179,7 @@ class Notifications(ListView): types.append(type) return ( TimelineEvent.objects.filter(identity=self.request.identity, type__in=types) - .order_by("-created")[:50] + .order_by("-created") .select_related( "subject_post", "subject_post__author", diff --git a/static/css/style.css b/static/css/style.css index 4160d26..05e3812 100644 --- a/static/css/style.css +++ b/static/css/style.css @@ -1089,6 +1089,12 @@ table.metadata td .emoji { margin-right: 4px; } +.pagination { + display: flex; + justify-content: center; + gap: 1em; +} + .load-more { margin: 10px 0; text-align: center; diff --git a/takahe/urls.py b/takahe/urls.py index 1e02622..07ccc50 100644 --- a/takahe/urls.py +++ b/takahe/urls.py @@ -22,7 +22,7 @@ urlpatterns = [ path("explore/tags/", explore.ExploreTag.as_view(), name="explore-tag"), path( "follows/", - follows.FollowsPage.as_view(), + follows.Follows.as_view(), name="follows", ), # Settings views diff --git a/templates/activities/follows.html b/templates/activities/follows.html index 18c7811..e2c0ece 100644 --- a/templates/activities/follows.html +++ b/templates/activities/follows.html @@ -4,17 +4,17 @@ {% block content %}
- {% for identity, details in identities %} + {% for identity, follow_type in page_obj %} {{ identity.html_name_or_handle }} @{{ identity.handle }} - {% if details.outbound %} + {% if follow_type == "outbound" %} Following {% endif %} - {% if details.inbound %} + {% if follow_type == "inbound" %} Follows You {% endif %} @@ -22,4 +22,14 @@

You have no follows.

{% endfor %}
+ + {% endblock %} diff --git a/templates/activities/home.html b/templates/activities/home.html index 546da0d..06fa88c 100644 --- a/templates/activities/home.html +++ b/templates/activities/home.html @@ -4,7 +4,7 @@ {% block title %}Home{% endblock %} {% block content %} - {% for event in events %} + {% for event in page_obj %} {% if event.type == "post" %} {% include "activities/_post.html" with post=event.subject_post %} {% elif event.type == "boost" %} @@ -21,4 +21,14 @@ {% empty %} Nothing to show yet. {% endfor %} + + {% endblock %} diff --git a/templates/activities/local.html b/templates/activities/local.html index c03b0c0..c3d180b 100644 --- a/templates/activities/local.html +++ b/templates/activities/local.html @@ -9,7 +9,13 @@ No posts yet. {% endfor %} - {% if page_obj.has_next %} -
Next Page
- {% endif %} + {% endblock %} diff --git a/templates/activities/notifications.html b/templates/activities/notifications.html index 890a2ad..89ca4d8 100644 --- a/templates/activities/notifications.html +++ b/templates/activities/notifications.html @@ -32,7 +32,13 @@ No notifications yet. {% endfor %} - {% if page_obj.has_next %} -
Next Page
- {% endif %} + {% endblock %} diff --git a/templates/activities/tag.html b/templates/activities/tag.html index a319b6a..f0684f0 100644 --- a/templates/activities/tag.html +++ b/templates/activities/tag.html @@ -10,7 +10,13 @@ No posts yet. {% endfor %} - {% if page_obj.has_next %} -
Next Page
- {% endif %} + {% endblock %}