From f0847a260e5ff8d6bab0454cd25a8067e52e66bd Mon Sep 17 00:00:00 2001
From: StephDriver <5330770+StephDriver@users.noreply.github.com>
Date: Fri, 3 Oct 2025 09:15:05 +0100
Subject: [PATCH] revert: #4890 non-visible list syntax as breaks copy button

---
 .../elements/journal/citation_modals.html     | 38 ++++++++-----------
 1 file changed, 15 insertions(+), 23 deletions(-)

diff --git a/src/themes/clean/templates/elements/journal/citation_modals.html b/src/themes/clean/templates/elements/journal/citation_modals.html
index 349ddf25d4..2f925e8c6b 100644
--- a/src/themes/clean/templates/elements/journal/citation_modals.html
+++ b/src/themes/clean/templates/elements/journal/citation_modals.html
@@ -10,14 +10,12 @@ <h2 class="modal-title" id="HarvardModalLabel">{% trans 'Harvard-style Citation'
                 </div>
                 <div class="modal-body">
                     <p>
-                        <span id="harvard-cite">
-                            <ul class="list-inline comma-ampersand-sep">
-                                {% for author in article.frozenauthor_set.all %}
-                                    <li>{{ author.last_name }}, {{ author.first_name|slice:"1" }}</li>
-                                {% endfor %}
-                            </ul>
-                            ({{ article.date_published.year }}) '{{ article.title|safe }}',
-                            <em>{% if journal.name %}{{ journal.name }}{% else %}{{ request.press.name }} Preprints{% endif %}</em>. {% if article.issue %}{{ article.issue.volume }}({{ article.issue.issue }}){% endif %}{% if article.page_range %}:{{ article.page_range }}.{% endif %}
+                        <span id="harvard-cite">{% for author in article.frozenauthor_set.all %}
+                            {% if not forloop.first and not forloop.last %},
+                            {% elif forloop.last and not forloop.first %}& {% endif %}{{ author.last_name }},
+                                {{ author.first_name|slice:"1" }}{% if forloop.last %}.{% endif %} {% endfor %}
+                                ({{ article.date_published.year }}) '{{ article.title|safe }}',
+                                <em>{% if journal.name %}{{ journal.name }}{% else %}{{ request.press.name }} Preprints{% endif %}</em>. {% if article.issue %}{{ article.issue.volume }}({{ article.issue.issue }}){% endif %}{% if article.page_range %}:{{ article.page_range }}.{% endif %}
                             {% if article.identifier.id_type == 'doi' %}
                                 {% include "elements/doi_display.html" with doi=article.identifier.identifier title=article.title %}
                             {% endif %}
@@ -43,15 +41,12 @@ <h2 class="modal-title" id="VancouverModalLabel">{% trans 'Vancouver-style Citat
                 </div>
                 <div class="modal-body">
                     <p>
-                        <span id="vancouver-cite">
-                            <ul class="list-inline comma-ampersand-sep">
-                                {% for author in article.frozenauthor_set.all %}
-                                    <li>{{ author.last_name }}, {{ author.first_name|slice:"1" }}</li>
-                                {% endfor %}
-                            </ul>
-                            {{ article.title|safe }}. {% if journal.name %}{{ journal.name }}{% else %}{{ request.press.name }} Preprints{% endif %}. {{ article.date_published.year }} {{ article.date_published.month }};{% if article.issue %} {{ article.issue.volume }}({{ article.issue.issue }}){% endif %}{% if article.page_range %}:{{ article.page_range }}.{% endif %}
-                            {% if article.identifier.id_type == 'doi' %}
-                                {% include "elements/doi_display.html" with doi=article.identifier.identifier title=article.title %}
+                        <span id="vancouver-cite">{% for author in article.frozenauthor_set.all %}{% if not forloop.first and not forloop.last %},
+                            {% elif forloop.last and not forloop.first %}& {% endif %}{{ author.last_name }},
+                                {{ author.first_name|slice:"1" }}{% if forloop.last %}.{% endif %} {% endfor %}
+                                {{ article.title|safe }}. {% if journal.name %}{{ journal.name }}{% else %}{{ request.press.name }} Preprints{% endif %}. {{ article.date_published.year }} {{ article.date_published.month }};{% if article.issue %} {{ article.issue.volume }}({{ article.issue.issue }}){% endif %}{% if article.page_range %}:{{ article.page_range }}.{% endif %}
+                                {% if article.identifier.id_type == 'doi' %}
+                                    {% include "elements/doi_display.html" with doi=article.identifier.identifier title=article.title %}
                             {% endif %}
                         </span>
                         {% include "admin/elements/button_copy_element.html" with element_id="vancouver-cite" %}
@@ -75,12 +70,9 @@ <h2 class="modal-title" id="APAModalLabel">{% trans 'APA-style Citation' %}</h2>
                 </div>
                 <div class="modal-body">
                     <p>
-                        <span id="apa-cite">
-                            <ul class="list-inline comma-ampersand-sep">
-                                {% for author in article.frozenauthor_set.all %}
-                                    <li>{{ author.last_name }}, {{ author.first_name|slice:"1" }}</li>
-                                {% endfor %}
-                            </ul>
+                        <span id="apa-cite">{% for author in article.frozenauthor_set.all %}{% if forloop.last %}{% if not forloop.first %}
+                            &amp; {% endif %}{% endif %}{{ author.last_name }},
+                            {{ author.first_name|slice:"1" }}{% if forloop.last %}.{% endif %} {% endfor %}
                             ({{ article.date_published.year }}, {{ article.date_published.month }} {{ article.date_published.day }}). {{ article.title|safe }}.
                             <em>{% if journal.name %}{{ journal.name }}{% else %}{{ request.press.name }} {% trans 'Preprints' %}{% endif %}</em> {% if article.issue %}{{ article.issue.volume }}({{ article.issue.issue }}){% endif %}{% if article.page_range %}:{{ article.page_range }}.{% endif %}
                             {% if article.identifier.id_type == 'doi' %}