From b95ba9eb586f8d530547562bf1747da29fc85d13 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ondr=CC=8Cej=20Ny=CC=81vlt?= <ondrej@nyv.lt>
Date: Thu, 17 Aug 2023 11:03:34 +0200
Subject: [PATCH] Style article detail

---
 view/static/main.css               | 13 ++++-----
 view/templates/article_detail.html | 45 ++++++++++++++++++++++++++----
 2 files changed, 45 insertions(+), 13 deletions(-)

diff --git a/view/static/main.css b/view/static/main.css
index 5e6e668..ef68fff 100644
--- a/view/static/main.css
+++ b/view/static/main.css
@@ -75,6 +75,7 @@ th,
 tr,
 td {
 	text-align: inherit;
+	border-collapse: collapse;
 }
 
 a {
@@ -97,6 +98,10 @@ del {
 	text-decoration-color: hsl(0 50% 40% / 50%);
 }
 
+code {
+	font-size: inherit;
+}
+
 summary {
 	cursor: pointer;
 	list-style: none;
@@ -243,6 +248,7 @@ details[open] summary::before {
 	border-radius: var(--radius-m);
 	background-color: white;
 	box-shadow: 0 2px 0 hsl(0 0% 50% / 20%);
+	overflow: hidden;
 }
 
 .pagination {
@@ -363,10 +369,3 @@ details[open] summary::before {
 .changeset table th {
 	padding-right: 1rem;
 }
-
-/* Article detail */
-
-.diffs-list {
-	list-style-type: none;
-	padding: 0;
-}
diff --git a/view/templates/article_detail.html b/view/templates/article_detail.html
index 13daf54..ee2f734 100644
--- a/view/templates/article_detail.html
+++ b/view/templates/article_detail.html
@@ -1,13 +1,46 @@
 {% extends "base.html" %}
 
+{% block head %}
+<style>
+.page-heading {
+	font-size: var(--font-size-l);
+	margin-bottom: 2rem;
+}
+
+.diffs-list {
+	list-style-type: none;
+	padding: 0;
+}
+
+.diff-table th, .diff-table td {
+	padding: 0.5rem 0.75rem;
+}
+
+.diff-table th {
+	font-size: var(--font-size-xs);
+	color: var(--color-muted);
+	font-weight: 500;
+}
+
+.diff-table tr:not(:last-of-type) {
+	border-bottom: 1px solid var(--border-color);
+}
+</style>
+{% endblock head %}
+
 {% block body %}
 <div class="container">
-	<h1>Diffs for the article at {{ article_url }}</h1>
+	<h1 class="page-heading">Diffs for the article at <code>{{ article_url }}</code></h1>
 
-	<ol class="diffs-list">
-		{% for diff in diffs %}
-		<li>{{ diff.diff_time }} {{ diff.diff_html|safe }}</li>
-		{% endfor %}
-	</ol>
+	<div class="card">
+		<table class="diff-table">
+			{% for diff in diffs %}
+			<tr>
+				<th>{{ diff.diff_time }}</th>
+				<td>{{ diff.diff_html|safe }}</td>
+			</tr>
+			{% endfor %}
+		</table>
+	</div>
 </div>
 {% endblock body %}