From 9a1ecf3b0b2dddc77e2cabe87da6422d8fe0db2c Mon Sep 17 00:00:00 2001
From: Shadowfacts <me@shadowfacts.net>
Date: Sun, 10 Nov 2019 11:48:41 -0500
Subject: [PATCH] Reorganize account settings page

---
 .../templates/account/show.html.eex           | 95 +++++++++++--------
 1 file changed, 54 insertions(+), 41 deletions(-)

diff --git a/lib/frenzy_web/templates/account/show.html.eex b/lib/frenzy_web/templates/account/show.html.eex
index 5942a7a..9b8cd1d 100644
--- a/lib/frenzy_web/templates/account/show.html.eex
+++ b/lib/frenzy_web/templates/account/show.html.eex
@@ -1,49 +1,62 @@
 <h1>Account Settings</h1>
-<h2><pre><%= @user.username %></pre></h2>
+<h2>Username: <code><%= @user.username %></code></h2>
 
+
+<section class="card mt-4">
+  <h4 class="card-header">Import/Export Data</h4>
+  <ul class="list-group list-group-flush">
+    <li class="list-group-item">
+      <%= form_for @conn, Routes.account_path(@conn, :import), [method: :post, multipart: true], fn f -> %>
+        <%= file_input f, :file, required: true %>
+        <%= submit "Import OPML", class: "btn btn-primary" %>
+      <% end %>
+    </li>
+    <li class="list-group-item">
+      <%= form_for @conn, Routes.account_path(@conn, :export), [method: :post, class: "mt-2"], fn f -> %>
+        <%= submit "Export OPML", class: "btn btn-primary" %>
+      <% end %>
+    </li>
+  </ul>
+</section>
+
+<section class="card mt-4">
+  <h4 class="card-header">Security</h4>
+
+  <ul class="list-group list-group-flush">
+    <li class="list-group-item">
 <a href="<%= Routes.account_path(@conn, :change_password) %>" class="btn btn-secondary">Change Password</a>
 
 <a href="<%= Routes.account_path(@conn, :change_fever_password) %>" class="btn btn-secondary">Change Fever Password</a>
-
-<section class="mt-4">
-  <h2>Import/Export Data</h2>
-  <%= form_for @conn, Routes.account_path(@conn, :import), [method: :post, multipart: true], fn f -> %>
-    <%= file_input f, :file %>
-    <%= submit "Import OPML", class: "btn btn-primary" %>
-  <% end %>
-  <%= form_for @conn, Routes.account_path(@conn, :export), [method: :post, class: "mt-2"], fn f -> %>
-    <%= submit "Export OPML", class: "btn btn-primary" %>
-  <% end %>
-</section>
-
-<section class="mt-4">
-  <h2>Approved Clients</h2>
-
-  <table class="table table-striped">
-    <thead>
-      <tr>
-        <th>Client</th>
-        <th>Revoke Access</th>
-      </tr>
-    </thead>
-    <tbody>
-      <%= for {approved, fervor} <- @clients do %>
+    </li>
+    <li class="list-group-item">
+    <h5 class="card-title">Approved Clients</h5>
+   <table class="table table-striped">
+      <thead>
         <tr>
-          <td>
-            <%= if fervor.website do %>
-              <a href="<%= fervor.website %>"><%= fervor.client_name %></a>
-            <% else %>
-              <%= fervor.client_name %>
-            <% end %>
-          </td>
-          <td>
-            <%= form_tag Routes.account_path(@conn, :remove_client), method: :post do %>
-              <input type="hidden" name="client_id" value="<%= approved.client_id %>">
-              <%= submit "Revoke", class: "btn btn-danger" %>
-            <% end %>
-          </td>
+          <th>Client</th>
+          <th>Revoke Access</th>
         </tr>
-      <% end %>
-    </tbody>
-  </table>
+      </thead>
+      <tbody>
+        <%= for {approved, fervor} <- @clients do %>
+          <tr>
+            <td>
+              <%= if fervor.website do %>
+                <a href="<%= fervor.website %>"><%= fervor.client_name %></a>
+              <% else %>
+                <%= fervor.client_name %>
+              <% end %>
+            </td>
+            <td>
+              <%= form_tag Routes.account_path(@conn, :remove_client), method: :post do %>
+                <input type="hidden" name="client_id" value="<%= approved.client_id %>">
+                <%= submit "Revoke", class: "btn btn-danger" %>
+              <% end %>
+            </td>
+          </tr>
+        <% end %>
+      </tbody>
+    </table> </li>
+  </ul>
+
 </section>