Truncate long author lists on front page cards (fixes #2462)

Author: lukepayyapilli

physionet-django/project/fixtures/demo-project.json

@@ -495,6 +495,74 @@
     "project": 20
   }
 },
+{
+  "model": "project.publishedauthor",
+  "pk": 9,
+  "fields": {
+    "user": [
+      "george"
+    ],
+    "display_order": 2,
+    "is_submitting": false,
+    "is_corresponding": false,
+    "approval_datetime": "2018-11-14T21:11:11.781Z",
+    "first_names": "George",
+    "last_name": "Moody",
+    "corresponding_email": "[email protected]",
+    "project": 1
+  }
+},
+{
+  "model": "project.publishedauthor",
+  "pk": 10,
+  "fields": {
+    "user": [
+      "aewj"
+    ],
+    "display_order": 3,
+    "is_submitting": false,
+    "is_corresponding": false,
+    "approval_datetime": "2018-11-14T21:11:11.781Z",
+    "first_names": "Alistair",
+    "last_name": "Johnson",
+    "corresponding_email": "[email protected]",
+    "project": 1
+  }
+},
+{
+  "model": "project.publishedauthor",
+  "pk": 11,
+  "fields": {
+    "user": [
+      "admin"
+    ],
+    "display_order": 4,
+    "is_submitting": false,
+    "is_corresponding": false,
+    "approval_datetime": "2018-11-14T21:11:11.781Z",
+    "first_names": "Admin",
+    "last_name": "Bot",
+    "corresponding_email": "[email protected]",
+    "project": 1
+  }
+},
+{
+  "model": "project.publishedauthor",
+  "pk": 12,
+  "fields": {
+    "user": [
+      "jameslawson"
+    ],
+    "display_order": 5,
+    "is_submitting": false,
+    "is_corresponding": false,
+    "approval_datetime": "2018-11-14T21:11:11.781Z",
+    "first_names": "James",
+    "last_name": "Lawson",
+    "corresponding_email": "[email protected]",
+    "project": 1
+  }
+},
 {
   "model": "project.topic",
   "pk": 1,

physionet-django/project/templatetags/project_templatetags.py

@@ -212,3 +212,18 @@ def call_method(obj, method_name, *args):
 @register.simple_tag(name='can_view_project_files')
 def can_view_project_files(project, user, request=None):
     return can_view_project_files_func(project, user, request)
+
+
+@register.filter(name='truncate_authors')
+def truncate_authors(author_list, max_authors=5):
+    """
+    Truncate a list of authors to max_authors, appending 'et al' if truncated.
+
+    Usage: {{ published_project.author_list|truncate_authors:5 }}
+    """
+    authors = list(author_list)
+    if len(authors) <= max_authors:
+        return ', '.join(author.get_full_name() for author in authors)
+    else:
+        truncated = ', '.join(author.get_full_name() for author in authors[:max_authors])
+        return truncated + ', et al.'

physionet-django/search/templates/search/homepage_content_list.html

@@ -12,10 +12,8 @@ <h3 class="card-title">
       </a>
     </h3>
     {% if not published_project.is_legacy %}
-      <p>
-          {% for author in published_project.author_list %}
-          {{ author.get_full_name }}{% if not forloop.last %}, {% endif %}
-          {% endfor %}
+      <p class="card-authors">
+          {{ published_project.author_list|truncate_authors:3 }}
       </p>
     {% endif %}
 

physionet-django/static/custom/css/style.css

@@ -240,6 +240,14 @@ body .main {
   overflow: hidden;
   color: var(--gray-color);
 }
+.modern-ui .card-authors {
+  font-size: 0.875rem;
+  color: var(--gray-color);
+  display: -webkit-box;
+  -webkit-line-clamp: 2;
+  -webkit-box-orient: vertical;
+  overflow: hidden;
+}
 .modern-ui .badge-dark {
   background-color: var(--gray-200);
   color: var(--secondary-color);