From 5278b8e5fdc804cfa0f578cfc5cc1a52dc3605ea Mon Sep 17 00:00:00 2001
From: Andy Smith <andy@37signals.com>
Date: Tue, 20 Jan 2026 15:33:37 -0600
Subject: [PATCH] Touch up the Download Export page

---
 app/views/account/exports/show.html.erb    | 14 ++++++--------
 app/views/imports/new.html.erb             |  1 -
 app/views/users/data_exports/show.html.erb | 14 ++++++--------
 3 files changed, 12 insertions(+), 17 deletions(-)

diff --git a/app/views/account/exports/show.html.erb b/app/views/account/exports/show.html.erb
index f01c13dc5..77fd79b0d 100644
--- a/app/views/account/exports/show.html.erb
+++ b/app/views/account/exports/show.html.erb
@@ -10,20 +10,18 @@
   </div>
 <% end %>
 
-<div class="panel panel--wide shadow center flex flex-column gap">
-  <h2 class="txt-large margin-none font-weight-black"><%= @page_title %></h2>
+<div class="panel panel--wide shadow center flex flex-column align-center gap">
+  <h2 class="txt-large font-weight-black margin-none"><%= @page_title %></h2>
 
   <% if @export.present? %>
-    <p class="margin-none-block-start">Your export is ready. The download should start automatically.</p>
+    <div>Your export is ready. The download should start automatically.</div>
 
-    <%= link_to rails_blob_path(@export.file, disposition: "attachment"),
+    <%= link_to "Download your data", rails_blob_path(@export.file, disposition: "attachment"),
           id: "download-link",
           class: "btn btn--link",
-          data: { turbo: false, controller: "auto-click" } do %>
-      Download your data
-    <% end %>
+          data: { turbo: false, controller: "auto-click" } %>
   <% else %>
-    <p class="margin-none-block-start">That download link has expired. You’ll need to <%= link_to "request a new export", account_settings_path, class: "txt-lnk" %>.</p>
+    <div>That download link has expired. You’ll need to <%= link_to "request a new export", account_settings_path, class: "txt-link" %>.</div>
   <% end %>
 
 </div>
diff --git a/app/views/imports/new.html.erb b/app/views/imports/new.html.erb
index ca90c32ff..886a0b48f 100644
--- a/app/views/imports/new.html.erb
+++ b/app/views/imports/new.html.erb
@@ -9,7 +9,6 @@
   </header>
 
   <%= form_with url: imports_path, class: "flex flex-column gap", data: { controller: "form upload-preview" }, multipart: true do |form| %>
-
     <label class="btn input--upload">
       <div data-upload-preview-target="placeholder">Choose a file…</div>
       <div data-upload-preview-target="fileName" hidden></div>
diff --git a/app/views/users/data_exports/show.html.erb b/app/views/users/data_exports/show.html.erb
index 2a1d63805..28bdb9861 100644
--- a/app/views/users/data_exports/show.html.erb
+++ b/app/views/users/data_exports/show.html.erb
@@ -10,20 +10,18 @@
   </div>
 <% end %>
 
-<div class="panel panel--wide shadow center flex flex-column gap">
-  <h2 class="txt-large margin-none font-weight-black"><%= @page_title %></h2>
+<div class="panel panel--wide shadow center flex flex-column align-center gap">
+  <h2 class="txt-large font-weight-black margin-none"><%= @page_title %></h2>
 
   <% if @export.present? %>
-    <p class="margin-none-block-start">Your export is ready. The download should start automatically.</p>
+    <div>Your export is ready. The download should start automatically.</div>
 
-    <%= link_to rails_blob_path(@export.file, disposition: "attachment"),
+    <%= link_to "Download your data", rails_blob_path(@export.file, disposition: "attachment"),
           id: "download-link",
           class: "btn btn--link",
-          data: { turbo: false, controller: "auto-click" } do %>
-      Download your data
-    <% end %>
+          data: { turbo: false, controller: "auto-click" } %>
   <% else %>
-    <p class="margin-none-block-start">That download link has expired. You'll need to <%= link_to "request a new export", user_path(@user), class: "txt-lnk" %>.</p>
+    <div>That download link has expired. You'll need to <%= link_to "request a new export", user_path(@user), class: "txt-link" %>.</div>
   <% end %>
 
 </div>