From 1d55d62b2561a402c902b4c5a66076ab1c278548 Mon Sep 17 00:00:00 2001
From: David Gay <david@davidgay.org>
Date: Sun, 23 May 2021 22:18:43 -0400
Subject: Various appearance improvements

---
 app/views/characters/show.html.erb | 21 ++++++++++++++++++---
 app/views/locations/index.html.erb | 14 +++++++-------
 app/views/locations/show.html.erb  | 17 ++++++++++-------
 3 files changed, 35 insertions(+), 17 deletions(-)

(limited to 'app')

diff --git a/app/views/characters/show.html.erb b/app/views/characters/show.html.erb
index c497ea1..e7f9696 100644
--- a/app/views/characters/show.html.erb
+++ b/app/views/characters/show.html.erb
@@ -54,9 +54,24 @@
 <div class="my-6">
   <h2 class="text-2xl mb-4">Skills</h2>
 
-  <ul>
+  <table class="table-auto mb-8">
+    <thead>
+    <tr>
+      <th class="table-header-padded">Skill</th>
+      <th class="table-header-padded">Level</th>
+      <th class="table-header-padded">XPTNL</th>
+      <th class="table-header-padded">Total XP</th>
+    </tr>
+    </thead>
+    <tbody>
     <% @character.character_skills.ordered_by_skill_name.each do |cs| %>
-      <li><%= cs.skill.name %> Level <%= cs.level %> (<%= cs.xp %> XP)</li>
+      <tr>
+        <td class="table-cell-padded"><%= cs.skill.name %></td>
+        <td class="table-cell-padded"><%= cs.level %></td>
+        <td class="table-cell-padded"><%= cs.xp_to_next_level %></td>
+        <td class="table-cell-padded"><%= cs.xp %></td>
+      </tr>
     <% end %>
-  </ul>
+    </tbody>
+  </table>
 </div>
diff --git a/app/views/locations/index.html.erb b/app/views/locations/index.html.erb
index 758fd35..3c9b837 100644
--- a/app/views/locations/index.html.erb
+++ b/app/views/locations/index.html.erb
@@ -1,8 +1,8 @@
-<h1 class="text-xl">Locations</h1>
+<h1 class="text-3xl mb-4">Locations</h1>
 
-<ul>
-  <% @locations.each do |location| %>
-    <li><span class="font-bold"><%= link_to location.name, location_path(location) %></span>
-      – <%= location.description %></li>
-  <% end %>
-</ul>
+<% @locations.each do |location| %>
+  <div class="my-4">
+    <h2 class="text-xl"><%= link_to location.name, location_path(location) %></h2>
+    <p class="italic"><%= location.description %></p>
+  </div>
+<% end %>
diff --git a/app/views/locations/show.html.erb b/app/views/locations/show.html.erb
index bc63541..bed97ac 100644
--- a/app/views/locations/show.html.erb
+++ b/app/views/locations/show.html.erb
@@ -1,8 +1,11 @@
-<h1 class="text-xl"><%= @location.name %></h1>
+<div class="mb-4">
+  <h1 class="text-3xl"><%= @location.name %></h1>
+  <p class="italic"><%= @location.description %></p>
+</div>
 
-<ul>
-  <% @location.activities.each do |activity| %>
-    <li><span class="font-bold"><%= link_to activity.name, activity_path(activity) %></span>
-      – <%= activity.description %></li>
-  <% end %>
-</ul>
+<% @location.activities.each do |activity| %>
+  <div class="my-4">
+    <h2 class="text-xl"><%= link_to activity.name, activity_path(activity) %></h2>
+    <p class="italic"><%= activity.description %></p>
+  </div>
+<% end %>
-- 
cgit v1.2.3