From 8d91801536a9a76f964a3f4ca4a4e6e982627c7c Mon Sep 17 00:00:00 2001
From: Lars Eggert <lars@eggert.org>
Date: Wed, 16 Mar 2022 11:56:45 +0200
Subject: [PATCH 1/2] fix: Always word-wrap person names in person_link links

Defaulting to "text-nowrap" causes too many issues on narrow screens.
---
 ietf/templates/iesg/discusses.html      | 4 ++--
 ietf/templates/iesg/past_documents.html | 4 ++--
 ietf/templates/meeting/requests.html    | 4 ++--
 ietf/templates/person/person_link.html  | 3 +--
 ietf/templates/person/photo.html        | 2 +-
 5 files changed, 8 insertions(+), 9 deletions(-)

diff --git a/ietf/templates/iesg/discusses.html b/ietf/templates/iesg/discusses.html
index 9f3c0639988..ef9fcde7c44 100644
--- a/ietf/templates/iesg/discusses.html
+++ b/ietf/templates/iesg/discusses.html
@@ -57,11 +57,11 @@ <h1>IESG discuss positions</h1>
                            {% endif %}">
                     <td>{{ doc.displayname_with_link }}</td>
                     {% include "doc/search/status_columns.html" %}
-                    <td>{% person_link doc.ad nowrap=False %}</td>
+                    <td>{% person_link doc.ad %}</td>
                     <td>
                         {% for p in doc.blocking_positions %}
                             {% if p.is_old_pos %}<span class="text-muted">{% endif %}
-                            {% person_link p.balloter nowrap=False %}
+                            {% person_link p.balloter %}
                             ({% if p.discuss_time %}{{ p.discuss_time|timesince_days }} days ago{% endif %}
                             {% if doc.get_state_url != "rfc" and p.rev != doc.rev %}for -{{ p.rev }}{% endif %})
                             <br>
diff --git a/ietf/templates/iesg/past_documents.html b/ietf/templates/iesg/past_documents.html
index 89373faabff..043e47eba86 100644
--- a/ietf/templates/iesg/past_documents.html
+++ b/ietf/templates/iesg/past_documents.html
@@ -46,11 +46,11 @@ <h1>
                         </td>
                         <td>{{ doc.telechat }}</td>
                         {% include "doc/search/status_columns.html" %}
-                        <td>{% person_link doc.ad nowrap=False %}</td>
+                        <td>{% person_link doc.ad %}</td>
                         <td>
                             {% for p in doc.blocking_positions %}
                                 {% if p.is_old_pos %}<span class="text-muted">{% endif %}
-                                {% person_link p.balloter nowrap=False %}
+                                {% person_link p.balloter %}
                                 ({% if p.discuss_time %}{{ p.discuss_time|timesince_days }} days ago{% endif %}
                                 {% if doc.get_state_url != "rfc" and p.rev != doc.rev %}for -{{ p.rev }}{% endif %})
                                 <br>
diff --git a/ietf/templates/meeting/requests.html b/ietf/templates/meeting/requests.html
index 87c8b7d5620..539082a518b 100644
--- a/ietf/templates/meeting/requests.html
+++ b/ietf/templates/meeting/requests.html
@@ -58,10 +58,10 @@ <h2 class="mt-5" id="{{ area.grouper.acronym }}">
                             {% if session.requested_duration %}{{ session.requested_duration|stringformat:"s"|slice:"0:4" }}{% endif %}
                         </td>
                         <td>{{ session.attendees|default:"" }}</td>
-                        <td>{% person_link session.requested_by_person nowrap=False %}</td>
+                        <td>{% person_link session.requested_by_person %}</td>
                         <td>
                             {% if session.group.ad_role %}
-                                {% person_link session.group.ad_role.person nowrap=False %}
+                                {% person_link session.group.ad_role.person %}
                             {% endif %}
                         </td>
                         <td>
diff --git a/ietf/templates/person/person_link.html b/ietf/templates/person/person_link.html
index 96b89feec54..765b57b5c44 100644
--- a/ietf/templates/person/person_link.html
+++ b/ietf/templates/person/person_link.html
@@ -1,5 +1,4 @@
-<span {% if class or nowrap %}class="{% if nowrap %}text-nowrap{% endif %}
-      {{ class }}"
+<span {% if class %}class=" {{ class }}"
       {% endif %}>{% if email or name %}<a {% if class %}class="text-reset"{% endif %}
            title="{% if title %}{{ title }}{% else %}Datatracker profile of {{ name }}.{% endif %}"
            {% if email %} href="{% url 'ietf.person.views.profile' email_or_name=email %}" {% else %} href="{% url 'ietf.person.views.profile' email_or_name=name %}" {% endif %}>{{ name }}</a>{% if email and with_email %} <a {% if class %}class="text-reset"{% endif %}
diff --git a/ietf/templates/person/photo.html b/ietf/templates/person/photo.html
index c59dc369ed4..8200de4a20b 100644
--- a/ietf/templates/person/photo.html
+++ b/ietf/templates/person/photo.html
@@ -19,7 +19,7 @@
     {% if name or email %}</a>{% endif %}
     <div class="card-body">
         <p class="lead mb-1">
-            {% person_link person with_email=False nowrap=False %}
+            {% person_link person with_email=False %}
         </p>
         {% for role in groups %}
             <a title="{{ role.group.name }} {{ role.group.type | upper }}"

From f6ddc789dd2db5fe52c631ab029bfbd63682dea5 Mon Sep 17 00:00:00 2001
From: Lars Eggert <lars@eggert.org>
Date: Wed, 16 Mar 2022 12:12:42 +0200
Subject: [PATCH 2/2] Remove extra space

---
 ietf/templates/person/person_link.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ietf/templates/person/person_link.html b/ietf/templates/person/person_link.html
index 765b57b5c44..dd497f780d7 100644
--- a/ietf/templates/person/person_link.html
+++ b/ietf/templates/person/person_link.html
@@ -1,4 +1,4 @@
-<span {% if class %}class=" {{ class }}"
+<span {% if class %}class="{{ class }}"
       {% endif %}>{% if email or name %}<a {% if class %}class="text-reset"{% endif %}
            title="{% if title %}{{ title }}{% else %}Datatracker profile of {{ name }}.{% endif %}"
            {% if email %} href="{% url 'ietf.person.views.profile' email_or_name=email %}" {% else %} href="{% url 'ietf.person.views.profile' email_or_name=name %}" {% endif %}>{{ name }}</a>{% if email and with_email %} <a {% if class %}class="text-reset"{% endif %}