💄 hide gap in authors list when no one has a profile picture

okfde · Oct 1, 2024 · 8d713dd · 8d713dd
1 parent f64f799
commit 8d713dd
Show file tree

Hide file tree

Showing 3 changed files with 25 additions and 17 deletions.
diff --git a/fragdenstaat_de/fds_blog/templates/fds_blog/includes/_authors.html b/fragdenstaat_de/fds_blog/templates/fds_blog/includes/_authors.html
@@ -1,21 +1,24 @@
 {% load thumbnail %}
 {% load i18n %}
+{% load blog_tags %}
 {% if authors %}
     <div class="hstack gap-3 align-content-center">
-        {# authors are also linked with text below, so spare screenreaders/keyboard nav #}
-        <div class="blog-author-images" aria-hidden="true">
-            {% for author in authors %}
-                {% if author.user.profile_photo %}
-                    <a href="{{ author.get_absolute_url }}" tabindex="-1">
-                        <img width="32"
-                             height="32"
-                             src="{% thumbnail author.user.profile_photo 64x64 %}"
-                             alt=""
-                             class="rounded-circle" />
-                    </a>
-                {% endif %}
-            {% endfor %}
-        </div>
+        {% if authors|has_author_images %}
+            {# authors are also linked with text below, so spare screenreaders/keyboard nav #}
+            <div class="blog-author-images" aria-hidden="true">
+                {% for author in authors %}
+                    {% if author.user.profile_photo %}
+                        <a href="{{ author.get_absolute_url }}" tabindex="-1">
+                            <img width="32"
+                                 height="32"
+                                 src="{% thumbnail author.user.profile_photo 64x64 %}"
+                                 alt=""
+                                 class="rounded-circle" />
+                        </a>
+                    {% endif %}
+                {% endfor %}
+            </div>
+        {% endif %}
         <div class="d-md-flex flex-wrap align-items-center">
             <p class="m-0">
                 {% trans "by" %}

diff --git a/fragdenstaat_de/fds_blog/templates/fds_blog/includes/blog_item_featured_top.html b/fragdenstaat_de/fds_blog/templates/fds_blog/includes/blog_item_featured_top.html
@@ -12,7 +12,7 @@
         <div class="d-flex flex-column col-lg-5 p-3 p-md-4 order-lg-first border-blue blog-top-feature-meta">
             {% include "fds_blog/includes/_item_meta.html" with hide_date=True %}
             <div class="mt-auto pt-3">
-                {% include "fds_blog/includes/_authors.html" with authors=article.authors.all hide_date=True %}
+                {% include "fds_blog/includes/_authors.html" with authors=article.get_authors hide_date=True %}
             </div>
         </div>
     </div>

diff --git a/fragdenstaat_de/fds_blog/templatetags/blog_tags.py b/fragdenstaat_de/fds_blog/templatetags/blog_tags.py
@@ -1,10 +1,10 @@
 import re
-from typing import Tuple
+from typing import List, Tuple
 
 from django import template
 
 from ..managers import articles_published
-from ..models import Article
+from ..models import Article, Author
 
 register = template.Library()
 
@@ -46,3 +46,8 @@ def get_blog_preview(context, amount=6):
     articles = Article.published.all()[:amount]
 
     return {"articles": articles, "request": context.get("request")}
+
+
+@register.filter
+def has_author_images(authors: List[Author]) -> bool:
+    return any(author.user and author.user.profile_photo for author in authors)