From 307cfc0811b7c13507522a8df9c08c8932db125f Mon Sep 17 00:00:00 2001
From: StephDriver <5330770+StephDriver@users.noreply.github.com>
Date: Fri, 20 Dec 2024 12:02:45 +0000
Subject: [PATCH 1/2] a11y #4244 keywords markup as lists

---
 .../clean/templates/journal/article.html      | 19 ++++++++++++++-----
 1 file changed, 14 insertions(+), 5 deletions(-)

diff --git a/src/themes/clean/templates/journal/article.html b/src/themes/clean/templates/journal/article.html
index cd369b1e2..9d6d3f63d 100644
--- a/src/themes/clean/templates/journal/article.html
+++ b/src/themes/clean/templates/journal/article.html
@@ -79,11 +79,20 @@ <h2>{% trans "Abstract" %}</h2>
                     <p>{{ article.abstract | safe }}</p>
                 {% endif %}
                 {% if article.keywords and article.keywords.count > 0 %}
-                    <p><strong>{% trans "Keywords" %}:</strong>
-                    {% for keyword in article.keywords.all %}
-                        {% if journal_settings.general.keyword_list_page %}<a href="{% url 'keyword' keyword.pk %}">{% endif %}
-                    {{ keyword.word }}{% if journal_settings.general.keyword_list_page %}</a>{% endif %}{% if not forloop.last %}, {% endif %}
-                    {% endfor %}</p>
+                    <h3>{% trans "Keywords" %}:<h3>
+                    <ul>
+                        {% for keyword in article.keywords.all %}
+                            <li>
+                                {% if journal_settings.general.keyword_list_page %}
+                                    <a href="{% url 'keyword' keyword.pk %}">
+                                {% endif %}
+                                {{ keyword.word }}
+                                {% if journal_settings.general.keyword_list_page %}
+                                    </a>
+                                {% endif %}
+                            </li>
+                        {% endfor %}
+                    </ul>
                 {% endif %}
                 {% if article.is_published or proofing %}
                     <p><strong>{% trans "How to Cite" %}:</strong>

From 2942284729c897be256c22ca40efc35bac2e78fa Mon Sep 17 00:00:00 2001
From: StephDriver <5330770+StephDriver@users.noreply.github.com>
Date: Fri, 20 Dec 2024 14:35:05 +0000
Subject: [PATCH 2/2] WIP voice over tests

---
 src/themes/clean/templates/journal/article.html | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/src/themes/clean/templates/journal/article.html b/src/themes/clean/templates/journal/article.html
index 9d6d3f63d..5c9de0bea 100644
--- a/src/themes/clean/templates/journal/article.html
+++ b/src/themes/clean/templates/journal/article.html
@@ -79,14 +79,15 @@ <h2>{% trans "Abstract" %}</h2>
                     <p>{{ article.abstract | safe }}</p>
                 {% endif %}
                 {% if article.keywords and article.keywords.count > 0 %}
-                    <h3>{% trans "Keywords" %}:<h3>
-                    <ul>
+                    <h3>{% trans "Keywords" %}:</h3>
+                    <ul class="list-inline">
                         {% for keyword in article.keywords.all %}
-                            <li>
+                            <li class="list-inline-item">
                                 {% if journal_settings.general.keyword_list_page %}
                                     <a href="{% url 'keyword' keyword.pk %}">
                                 {% endif %}
-                                {{ keyword.word }}
+                                <span aria-hidden=true>{{ keyword.word }}</span>
+                                <span class="sr-only">{% trans 'Keyword:' %} {{ keyword.word }}</span>
                                 {% if journal_settings.general.keyword_list_page %}
                                     </a>
                                 {% endif %}