feat: add new group icon for various author cases in user profile

Author: julhoang

core/views.py

@@ -2008,6 +2008,16 @@ def get_context_data(self, **kwargs):
                 "badge": BadgeToken.BOOST_DAY,
                 "bio": "Boost Day contributor.",
             },
+            {
+                # Collective author: group icon, no link or role (as produced by
+                # build_library_intro_context for e.g. "Various").
+                "name": "Various Authors",
+                "profile_url": None,
+                "role": None,
+                "avatar_url": "",
+                "badge": "",
+                "bio": "",
+            },
         ]
 
         context["contributor_data"] = 10 * [

libraries/utils.py

@@ -470,6 +470,25 @@ def patch_commit_authors(users):
     return users
 
 
+def apply_collective_author_overrides(author_dicts):
+    """Normalize collective authors (e.g. "Various Authors") in V3 profile dicts.
+
+    A collective author renders as a single labelled group icon with no profile
+    link or role. Mutates each dict in place and returns the same list.
+    """
+    from users.templatetags.avatar_tags import (
+        collective_author_label,
+        is_collective_author,
+    )
+
+    for author in author_dicts:
+        if is_collective_author(author["name"]):
+            author["name"] = collective_author_label(author["name"])
+            author["profile_url"] = None
+            author["role"] = None
+    return author_dicts
+
+
 def build_library_intro_context(
     library_version, *, max_authors=None, include_contributors=False
 ):
@@ -520,6 +539,8 @@ def build_library_intro_context(
         for contributor in top_contributors
     )
 
+    apply_collective_author_overrides(author_dicts)
+
     return {
         "library_name": library.display_name,
         "description": library_version.description or library.description or "",

libraries/views.py

@@ -34,6 +34,7 @@
     Tier,
 )
 from .utils import (
+    apply_collective_author_overrides,
     get_view_from_cookie,
     set_view_in_cookie,
     get_prioritized_library_view,
@@ -572,15 +573,17 @@ def get_v3_context_data(self, **kwargs):
             ]
         )
         context["this_release_contributors"] = (
-            this_release or SharedResources.library_release_contributors
+            apply_collective_author_overrides(this_release)
+            or SharedResources.library_release_contributors
         )
 
         all_time = [
             a.to_v3_profile_dict("Contributor")
             for a in base_context.get("previous_contributors", [])
         ]
         context["all_time_contributors"] = (
-            all_time or SharedResources.library_all_contributors
+            apply_collective_author_overrides(all_time)
+            or SharedResources.library_all_contributors
         )
 
         context["is_flagship_lib"] = self.object.tier == Tier.FLAGSHIP

static/css/v3/avatar.css

@@ -106,6 +106,19 @@
   background: var(--color-accent-weak-teal);
 }
 
+/* Collective author (e.g. "Various Authors") */
+.avatar--collective {
+  box-sizing: border-box;
+  padding: var(--space-medium);
+  background: var(--color-accent-weak-yellow);
+}
+
+.avatar--collective .avatar__icon {
+  width: 100%;
+  height: 100%;
+  color: var(--color-icon-primary);
+}
+
 /* Placeholder (e.g. Header when no image and no name) */
 .avatar-placeholder {
   width: var(--avatar-size-md);

static/css/v3/contributors-list.css

@@ -4,6 +4,7 @@
 
 .contributors-list .card__column {
     gap: var(--space-medium);
+    overflow-y: auto;
 }
 
 .contributors-list.all-contributors {

templates/includes/icon.html

@@ -123,6 +123,8 @@
     <path d="M10 5.99996V4.66663H11.3333V5.99996H10ZM11.3333 9.99996V8.66663H8.66667V7.33329H11.3333V5.99996H12.6667V7.33329H14V8.66663H12.6667V9.99996H11.3333ZM11.3333 9.99996V11.3333H10V9.99996H11.3333ZM7.33333 7.33329V8.66663H4.66667V9.99996H3.33333V8.66663H2V7.33329H3.33333V5.99996H4.66667V7.33329H7.33333ZM4.66667 9.99996H6V11.3333H4.66667V9.99996ZM6 4.66663V5.99996H4.66667V4.66663H6Z" />
   {% elif icon_name == "users" %}
     <path d="M7.33333 0H3.33333V1.33333H2V5.33333H3.33333V6.66667H7.33333V5.33333H3.33333V1.33333H7.33333V0ZM7.33333 1.33333H8.66667V5.33333H7.33333V1.33333ZM0 9.33333H1.33333V12H9.33333V13.3333H0V9.33333ZM1.33333 9.33333H9.33333V8H1.33333V9.33333ZM10.6667 9.33333H9.33333V13.3333H10.6667V9.33333ZM10 0H12.6667V1.33333H10V0ZM12.6667 5.33333H10V6.66667H12.6667V5.33333ZM12.6667 1.33333H14V5.33333H12.6667V1.33333ZM16 9.33333H14.6667V12H12V13.3333H16V9.33333ZM12 8H14.6667V9.33333H12V8Z" />
+  {% elif icon_name == "people-group" %}
+    <path d="M11 2H5V4H3V10H5V12H11V10H5V4H11V2ZM11 4H13V10H11V4ZM0 16H2V20H14V22H0V16ZM2 16H14V14H2V16ZM16 16H14V22H16V16ZM15 2H19V4H15V2ZM19 10H15V12H19V10ZM19 4H21V10H19V4ZM24 16H22V20H18V22H24V16ZM18 14H22V16H18V14Z" />
   {% elif icon_name == "building-community" %}
     <path d="M13.3334 1.33337H14.6667V14.6667H1.33337V9.33337H2.66671V13.3334H5.33337V10.6667H6.66671V13.3334H9.33337V9.33337H10.6667V13.3334H13.3334V2.66671H6.66671V4.00004H5.33337V1.33337H13.3334ZM8.00004 8.00004H9.33337V9.33337H8.00004V8.00004ZM6.66671 6.66671H8.00004V8.00004H6.66671V6.66671ZM5.33337 6.66671V5.33337H6.66671V6.66671H5.33337ZM4.00004 8.00004V6.66671H5.33337V8.00004H4.00004ZM4.00004 8.00004H2.66671V9.33337H4.00004V8.00004ZM10.6667 4.00004H12V5.33337H10.6667V4.00004ZM9.33337 4.00004H8.00004V5.33337H9.33337V4.00004ZM10.6667 6.66671H12V8.00004H10.6667V6.66671Z" />
   {% elif icon_name == "zap" %}

templates/v3/includes/_avatar_v3.html

@@ -9,9 +9,10 @@
     - size     (optional)  Avatar size: sm | md | lg | xl | xxxl (default: md)
 
   Rendering priority:
-    1. Image   — when `src` is provided
-    2. Initials — when `name` is provided (falls back to "?" if initials can't be derived)
-    3. Placeholder — generic "?" icon when neither `src` nor `name` is given
+    1. Group icon — when `name` is a collective label (e.g. "Various Authors")
+    2. Image   — when `src` is provided
+    3. Initials — when `name` is provided (falls back to "?" if initials can't be derived)
+    4. Placeholder — generic "?" icon when none of the above are provided
 
   Accessibility:
     - When `name` is provided, the avatar is announced to screen readers via aria-label.
@@ -26,7 +27,11 @@
 {% endcomment %}
 {% load avatar_tags %}
 {% with size=size|default:"md" variant=variant|default:"yellow" %}
-  {% if src %}
+  {% if name|is_collective_author %}
+    <span class="avatar avatar--{{ size }} avatar--collective" role="img" aria-label="{{ name }}">
+      {% include "includes/icon.html" with icon_name="people-group" icon_class="avatar__icon" %}
+    </span>
+  {% elif src %}
     <span {% if id %}id="{{id}}"{% endif %} class="avatar avatar--{{ size }}"{% if name %} role="img" aria-label="{{ name }}"{% else %} aria-hidden="true"{% endif %}>
       <img src="{{ src }}" class="avatar avatar--{{ size }} avatar__img" alt="" loading="lazy" decoding="async">
     </span>

users/templatetags/avatar_tags.py

@@ -1,9 +1,25 @@
+import re
 from typing import Literal
 from django import template
 from django.template.loader import render_to_string
 
 register = template.Library()
 
+_COLLECTIVE_AUTHOR_RE = re.compile(r"^\s*various\b", re.IGNORECASE)
+COLLECTIVE_AUTHOR_LABEL = "Various Authors"
+
+
+@register.filter
+def is_collective_author(value):
+    """True when `value` is a collective author label (e.g. "Various Authors")."""
+    return isinstance(value, str) and bool(_COLLECTIVE_AUTHOR_RE.match(value))
+
+
+@register.filter
+def collective_author_label(value):
+    """Canonical display label for a collective author, else `value` unchanged."""
+    return COLLECTIVE_AUTHOR_LABEL if is_collective_author(value) else value
+
 
 @register.filter(is_safe=True)
 def avatar_initials(value):

users/tests/test_avatar_tags.py

@@ -0,0 +1,37 @@
+import pytest
+
+from users.templatetags.avatar_tags import (
+    collective_author_label,
+    is_collective_author,
+)
+
+
+@pytest.mark.parametrize(
+    "value,expected",
+    [
+        ("Various Authors", True),
+        ("Various", True),
+        ("various authors", True),
+        ("  Various Authors", True),
+        ("John Various", False),
+        ("Variant", False),
+        ("", False),
+        (None, False),
+        (123, False),
+    ],
+)
+def test_is_collective_author(value, expected):
+    assert is_collective_author(value) is expected
+
+
+@pytest.mark.parametrize(
+    "value,expected",
+    [
+        ("Various", "Various Authors"),
+        ("various", "Various Authors"),
+        ("Various Authors", "Various Authors"),
+        ("Jane Doe", "Jane Doe"),
+    ],
+)
+def test_collective_author_label(value, expected):
+    assert collective_author_label(value) == expected

versions/views.py

@@ -28,6 +28,7 @@
 from libraries.models import Commit, CommitAuthor, ReleaseReport
 from libraries.tasks import generate_release_report
 from libraries.utils import (
+    apply_collective_author_overrides,
     set_selected_boost_version,
     determine_selected_boost_version,
     library_doc_latest_transform,
@@ -181,10 +182,12 @@ def get_version_heading(self, obj, is_current_release, v3=False):
 
     def get_v3_contributors(self, version):
         """Shape the release's top contributors for the v3 contributors card."""
-        return [
-            author.to_v3_profile_dict("Contributor")
-            for author in self.get_top_contributors_release(version)
-        ]
+        return apply_collective_author_overrides(
+            [
+                author.to_v3_profile_dict("Contributor")
+                for author in self.get_top_contributors_release(version)
+            ]
+        )
 
     def get_v3_context_data(self, **kwargs):
         obj = self.object