From 681db28da7ffda4662ec382f015f05aa361e0a62 Mon Sep 17 00:00:00 2001
From: weeklies <80141759+weeklies@users.noreply.github.com>
Date: Thu, 16 May 2024 14:12:11 +0100
Subject: [PATCH] AO3-6565 Feedback

---
 app/views/admin/admin_users/index.html.erb | 12 ++++--------
 app/views/admin/admin_users/show.html.erb  |  8 ++------
 2 files changed, 6 insertions(+), 14 deletions(-)

diff --git a/app/views/admin/admin_users/index.html.erb b/app/views/admin/admin_users/index.html.erb
index 9e1800cb387..0fd3302cd9b 100644
--- a/app/views/admin/admin_users/index.html.erb
+++ b/app/views/admin/admin_users/index.html.erb
@@ -31,14 +31,10 @@
       </dd>
       <% if policy(User).can_view_past? %>
         <dt><%= label_tag "settings", t(".settings.label") %></dt>
-        <dd>
-          <ul>
-              <li>
-                <%= check_box_tag "search_past", "1", params[:search_past].present? %>
-                <%= label_tag "search_past", t(".settings.past") %>
-              </li>
-          </ul>
-        </dd>
+        <li>
+          <%= check_box_tag "search_past", "1", params[:search_past].present? %>
+          <%= label_tag "search_past", t(".settings.past") %>
+        </li>
       <% end %>
     </dl>
     <p class="submit"><%= submit_tag ts("Find") %></p>
diff --git a/app/views/admin/admin_users/show.html.erb b/app/views/admin/admin_users/show.html.erb
index 1846bebbdca..a42e5b597ee 100644
--- a/app/views/admin/admin_users/show.html.erb
+++ b/app/views/admin/admin_users/show.html.erb
@@ -40,18 +40,14 @@
         <% if past_logins.present? %>
           <dt><%= t(".info.past_logins", count: past_logins.size) %></dt>
           <dd>
-            <ul>
-              <li><%= past_logins.join(", ") %></li>
-            </ul>
+            <%= past_logins.to_sentence %>
           </dd>
         <% end %>
         <% past_emails = @user.historic_values("email") %>
         <% if past_emails.present? %>
           <dt><%= t(".info.past_emails", count: past_emails.size) %></dt>
           <dd>
-            <ul>
-              <li><%= past_emails.join(", ") %></li>
-            </ul>
+            <%= past_emails.to_sentence %>
           </dd>
         <% end %>
       <% end %>