From 18728e73e0741fc12c61cbde541e5ac95e94633d Mon Sep 17 00:00:00 2001
From: George Katsikas <giorgakis.katsikas@gmail.com>
Date: Thu, 30 Nov 2023 13:23:46 +0100
Subject: [PATCH] update publication action list to wide grouped format

---
 .../journals/_publication_actions.html        | 132 ++++++++++++++----
 .../journals/_publication_preparation.html    |   2 +-
 .../journals/publication_detail.html          |   1 -
 3 files changed, 103 insertions(+), 32 deletions(-)

diff --git a/scipost_django/journals/templates/journals/_publication_actions.html b/scipost_django/journals/templates/journals/_publication_actions.html
index 08eaa107d..f266f9a40 100644
--- a/scipost_django/journals/templates/journals/_publication_actions.html
+++ b/scipost_django/journals/templates/journals/_publication_actions.html
@@ -1,32 +1,104 @@
-<ul class="mb-0">
-  <li><a href="{% url 'journals:add_author' doi_label=publication.doi_label %}">Add a missing author</a></li>
-  <li><a href="{% url 'journals:update_author_ordering' doi_label=publication.doi_label %}">Update Author ordering</a></li>
-  <li><a href="{% url 'journals:author_affiliations' doi_label=publication.doi_label %}">Manage Author Affiliations</a></li>
-  <li><a href="{% url 'journals:abstract_jats' publication.doi_label %}">Create/update abstract (JATS version)</a></li>
-  <li><a href="{% url 'journals:create_citation_list_metadata' publication.doi_label %}">Create/update citation list metadata</a></li>
-  <li><a href="{% url 'journals:create_funding_info_metadata' publication.doi_label %}">Create/update funding info metadata</a></li>
-  <li><a href="{% url 'journals:create_metadata_xml' publication.doi_label %}">Create/update the XML metadata</a></li>
-  <li><a href="{% url 'journals:manage_publication_resources' publication.doi_label %}">Manage publication resources</a></li>
-  <li><a href="{% url 'journals:metadata_xml_deposit' publication.doi_label 'test' %}">Test metadata deposit (via Crossref test server)</a></li>
-  <li><a href="{% url 'journals:metadata_xml_deposit' publication.doi_label 'deposit' %}">Deposit the metadata to Crossref</a></li>
-  <li><a href="{% url 'journals:produce_metadata_DOAJ' doi_label=publication.doi_label %}">Produce DOAJ metadata</a></li>
-  <li><a href="{% url 'journals:metadata_DOAJ_deposit' doi_label=publication.doi_label %}">Deposit the metadata to DOAJ</a></li>
-  <li><a href="{% url 'journals:allocate_orgpubfractions' doi_label=publication.doi_label %}">Allocate Org Pub Fractions</a></li>
-  <li><a href="{% url 'journals:harvest_citedby_links' publication.doi_label %}">Update Crossref cited-by links</a></li>
-  <li><a href="{% url 'journals:manage_metadata' %}">Metadata management page</a></li>
-  <li><a href="{% url 'journals:update_references' doi_label=publication.doi_label %}">Update references</a></li>
-  {% if journal.autogeneratedfilecontenttemplate_set %}
-    <li>View autogenerated file contents:
+<div class="row">
+  <div class="col">
+    <h4>Maintenance</h4>
+    <div>Authors</div>
+    <ul>
+      <li>
+        <a href="{% url 'journals:add_author' doi_label=publication.doi_label %}">Add a missing author</a>
+      </li>
+      <li>
+        <a href="{% url 'journals:update_author_ordering' doi_label=publication.doi_label %}">Update author ordering</a>
+      </li>
+      <li>
+        <a href="{% url 'journals:author_affiliations' doi_label=publication.doi_label %}">Manage author affiliations</a>
+      </li>
+    </ul>
+    <div>Publication fields</div>
+    <ul>
+      <li>
+        <a href="{% url 'journals:abstract_jats' publication.doi_label %}">Create/update abstract (JATS version)</a>
+      </li>
+      <li>
+        <a href="{% url 'journals:create_citation_list_metadata' publication.doi_label %}">Create/update citation list metadata</a>
+      </li>
+      <li>
+        <a href="{% url 'journals:create_funding_info_metadata' publication.doi_label %}">Create/update funding info metadata</a>
+      </li>
+    </ul>
+  </div>
+  <div class="col">
+    <h4>Metadata deposit</h4>
+    <div>Crossref</div>
+    <ul>
+      <li>
+        <a href="{% url 'journals:create_metadata_xml' publication.doi_label %}">Create/update the XML metadata</a>
+      </li>
+      <li>
+        <a href="{% url 'journals:metadata_xml_deposit' publication.doi_label 'deposit' %}">Deposit the metadata to Crossref</a>
+      </li>
+      <li>
+        <a href="{% url 'journals:harvest_citedby_links' publication.doi_label %}">Update Crossref cited-by links</a>
+      </li>
+    </ul>
+    <div>DOAJ</div>
+    <ul>
+      <li>
+        <a href="{% url 'journals:produce_metadata_DOAJ' doi_label=publication.doi_label %}">Produce DOAJ metadata</a>
+      </li>
+      <li>
+        <a href="{% url 'journals:metadata_DOAJ_deposit' doi_label=publication.doi_label %}">Deposit the metadata to DOAJ</a>
+      </li>
+    </ul>
+  </div>
+  <div class="col">
+    <h4>Administration</h4>
+    <div>Management</div>
+    <ul>
+      <li>
+        <a href="{% url 'journals:manage_metadata' publication.doi_label %}">Manage metadata</a>
+      </li>
+
+      {% comment %} {% if not publication.is_published %} {% endcomment %}
+      <li>
+        <a href="{% url 'journals:update_publication' publication.doi_label %}">Update Publication object</a>
+      </li>
+      <li>
+        <strong><a href="{% url 'journals:publish_publication' publication.doi_label %}">Publish this Publication</a></strong>
+      </li>
+      {% comment %} {% endif %} {% endcomment %}
+
+    </ul>
+
+    <div>Associations</div>
+    <ul>
+      <li>
+        <a href="{% url 'journals:manage_publication_resources' publication.doi_label %}">Manage publication resources</a>
+      </li>
+      <li>
+        <a href="{% url 'journals:allocate_orgpubfractions' publication.doi_label %}">Allocate Org Pub Fractions</a>
+      </li>
+      {% comment %} <li>
+        <a href="{% url 'journals:update_references' doi_label=publication.doi_label %}">Update references</a>
+      </li> {% endcomment %}
+      <li>
+        <a href="{% url 'journals:draft_accompanying_publication' publication.doi_label %}">Draft an accompanying Publication</a>
+      </li>
+    </ul>
+
+    {% if journal.autogeneratedfilecontenttemplate_set.all %}
+      Autogenerated files
       <ul>
-	{% for autotemplate in journal.autogeneratedfilecontenttemplate_set.all %}
-	  <li><a href="{% url 'journals:view_autogenerated_file' doi_label=publication.doi_label autotemplate_id=autotemplate.id %}" target="_blank">{{ autotemplate.name }} ({{ autotemplate.description }})</a></li>
-	{% endfor %}
+
+        {% for autotemplate in journal.autogeneratedfilecontenttemplate_set.all %}
+          <li>
+            <a href="{% url 'journals:view_autogenerated_file' doi_label=publication.doi_label autotemplate_id=autotemplate.id %}"
+               target="_blank">{{ autotemplate.name }} ({{ autotemplate.description }})</a>
+          </li>
+        {% endfor %}
+
+
       </ul>
-    </li>
-  {% endif %}
-  {% if not publication.is_published %}
-    <li><a href="{% url 'journals:update_publication' publication.doi_label %}">Update Publication object</a></li>
-    <li><strong><a href="{% url 'journals:publish_publication' publication.doi_label %}">Publish this Publication</a></strong></li>
-  {% endif %}
-  <li><a href="{% url 'journals:draft_accompanying_publication' publication.doi_label %}">Draft an accompanying Publication</a></li>
-</ul>
+    {% endif %}
+
+  </div>
+</div>
diff --git a/scipost_django/journals/templates/journals/_publication_preparation.html b/scipost_django/journals/templates/journals/_publication_preparation.html
index 337b4938a..2da316ae8 100644
--- a/scipost_django/journals/templates/journals/_publication_preparation.html
+++ b/scipost_django/journals/templates/journals/_publication_preparation.html
@@ -11,7 +11,7 @@
   <li>{% if publication.has_funding_statement %}<span class="text-success">{% include 'bi/check-square-fill.html' %}</span>{% else %}{% include 'bi/square.html' %}{% endif %}<a href="{% url 'journals:create_funding_info_metadata' publication.doi_label %}">Create/update funding info metadata</a></li>
   <li>{% if publication.grants.exists %}<span class="text-success">{% include 'bi/check-square-fill.html' %}</span>{% else %}{% include 'bi/square.html' %}{% endif %}<a href="{% url 'journals:update_grants' publication.doi_label %}">Create/update grants</a></li>
   <li>{% if publication.has_xml_metadata %}<span class="text-success">{% include 'bi/check-square-fill.html' %}</span>{% else %}{% include 'bi/square.html' %}{% endif %}<a href="{% url 'journals:create_metadata_xml' publication.doi_label %}">Create/update Crossref metadata</a> <em>(please do after citation and funding info are added)</em></li>
-  <li>{% if publication.references.exists %}<span class="text-success">{% include 'bi/check-square-fill.html' %}</span>{% else %}{% include 'bi/square.html' %}{% endif %}<a href="{% url 'journals:update_references' doi_label=publication.doi_label %}">Create/update references</a></li>
+  {% comment %} <li>{% if publication.references.exists %}<span class="text-success">{% include 'bi/check-square-fill.html' %}</span>{% else %}{% include 'bi/square.html' %}{% endif %}<a href="{% url 'journals:update_references' doi_label=publication.doi_label %}">Create/update references</a></li> {% endcomment %}
 </ul>
 
 <h3>Tools</h3>
diff --git a/scipost_django/journals/templates/journals/publication_detail.html b/scipost_django/journals/templates/journals/publication_detail.html
index 71f9cffb8..29e584b1c 100644
--- a/scipost_django/journals/templates/journals/publication_detail.html
+++ b/scipost_django/journals/templates/journals/publication_detail.html
@@ -182,7 +182,6 @@
     <hr class="divider">
     <div class="row">
       <div class="col-12">
-        <h3>Editorial Administration tools</h3>
         {% include 'journals/_publication_actions.html' with publication=publication %}
       </div>
     </div>
-- 
GitLab