This is an automated email from the ASF dual-hosted git repository.

sbp pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/tooling-trusted-release.git


The following commit(s) were added to refs/heads/main by this push:
     new 08e782c  Make a few minor improvements
08e782c is described below

commit 08e782cd21c28fa911c2af576ae5d95c6ee2ee9b
Author: Sean B. Palmer <[email protected]>
AuthorDate: Wed May 7 14:51:21 2025 +0100

    Make a few minor improvements
    
    - Ensure that paths are sorted on check pages.
    - Improve the usability of the download pages.
    - Improve the project release page wording.
---
 atr/routes/compose.py                | 1 +
 atr/routes/download.py               | 5 +++--
 atr/templates/releases-finished.html | 8 ++++----
 3 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/atr/routes/compose.py b/atr/routes/compose.py
index 95822ee..9923dd5 100644
--- a/atr/routes/compose.py
+++ b/atr/routes/compose.py
@@ -44,6 +44,7 @@ async def check(
     base_path = util.release_directory(release)
 
     paths = [path async for path in util.paths_recursive(base_path)]
+    paths.sort()
     info = await db.path_info(release, paths)
 
     user_ssh_keys: Sequence[models.SSHKey] = []
diff --git a/atr/routes/download.py b/atr/routes/download.py
index d41941d..2093fd4 100644
--- a/atr/routes/download.py
+++ b/atr/routes/download.py
@@ -225,5 +225,6 @@ async def _list(
         )
         display_name = f"{item_in_dir}/" if await 
aiofiles.os.path.isdir(full_path / item_in_dir) else str(item_in_dir)
         html.append(f'<a href="{link_url}">{display_name}</a>')
-    body = "<br>\n".join(html)
-    return quart.Response(body, mimetype="text/html")
+    head = "<style>body { margin: 1rem; font: 1.25rem/1.5 serif; }</style>"
+    response_body = head + "<br>\n".join(html)
+    return quart.Response(response_body, mimetype="text/html")
diff --git a/atr/templates/releases-finished.html 
b/atr/templates/releases-finished.html
index eadf8d1..ba64e4c 100644
--- a/atr/templates/releases-finished.html
+++ b/atr/templates/releases-finished.html
@@ -1,11 +1,11 @@
 {% extends "layouts/base.html" %}
 
 {% block title %}
-  Completed releases of {{ project.display_name }} ~ ATR
+  Releases of {{ project.display_name }} ~ ATR
 {% endblock title %}
 
 {% block description %}
-  All of the completed releases of {{ project.display_name }} on ATR.
+  All of the finished releases of {{ project.display_name }} on ATR.
 {% endblock description %}
 
 {% block content %}
@@ -13,10 +13,10 @@
     <a href="{{ as_url(routes.root.index) }}" class="atr-back-link">← Back to 
Select a release</a>
   </p>
 
-  <h1>Completed releases of {{ project.display_name }}</h1>
+  <h1>Releases of {{ project.display_name }}</h1>
 
   {% if releases %}
-    <p class="mb-4">The following releases have been completed and published 
for this project.</p>
+    <p class="mb-4">The following releases have been published for this 
project.</p>
     <div class="row row-cols-1 row-cols-md-2 row-cols-lg-3 g-4">
       {% for release in releases %}
         <div class="col">


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to