From e51cc4395e17c25b0ed020c56a4b09c2fb5f0504 Mon Sep 17 00:00:00 2001
From: Alex Cabal <alex@alexcabal.com>
Date: Wed, 18 Dec 2024 15:58:50 -0600
Subject: [PATCH] Tweak some styling

---
 templates/ProjectDetailsTable.php |  2 +-
 www/artworks/get.php              | 18 ++++++++---------
 www/css/artwork.css               | 33 -------------------------------
 www/css/core.css                  | 13 ++++++++++++
 4 files changed, 23 insertions(+), 43 deletions(-)

diff --git a/templates/ProjectDetailsTable.php b/templates/ProjectDetailsTable.php
index c4194cef..4d1722bf 100644
--- a/templates/ProjectDetailsTable.php
+++ b/templates/ProjectDetailsTable.php
@@ -62,7 +62,7 @@ $showArtworkStatus = $showArtworkStatus ?? true;
 					<? if($project->Ebook->Artwork !== null){ ?>
 						<i>
 							<a href="<?= $project->Ebook->Artwork->Url ?>"><?= Formatter::EscapeHtml($project->Ebook->Artwork->Name) ?></a>
-						</i> <i>(<?= ucfirst($project->Ebook->Artwork->Status->value) ?>.)</i>
+						</i> (<?= ucfirst($project->Ebook->Artwork->Status->value) ?>)
 					<? }else{ ?>
 						<i>None.</i>
 					<? } ?>
diff --git a/www/artworks/get.php b/www/artworks/get.php
index 39f70959..6b3e3166 100644
--- a/www/artworks/get.php
+++ b/www/artworks/get.php
@@ -85,23 +85,23 @@ catch(Exceptions\InvalidPermissionsException){
 		<?= Template::ImageCopyrightNotice() ?>
 
 		<h2>Metadata</h2>
-		<table class="artwork-metadata">
+		<table class="admin-table">
 			<tr>
-				<td>Title</td>
+				<td>Title:</td>
 				<td><i><?= Formatter::EscapeHtml($artwork->Name) ?></i><? if($isAdminView){ ?> (#<?= $artwork->ArtworkId ?>)<? } ?></td>
 			</tr>
 			<tr>
-				<td>Artist</td>
+				<td>Artist:</td>
 				<td>
 					<a href="<?= $artwork->Artist->Url ?>"><?= Formatter::EscapeHtml($artwork->Artist->Name) ?></a><? if(sizeof($artwork->Artist->AlternateNames ?? []) > 0){ ?> (A.K.A. <span class="author" typeof="schema:Person" property="schema:name"><?= implode('</span>, <span class="author" typeof="schema:Person" property="schema:name">', array_map('Formatter::EscapeHtml', $artwork->Artist->AlternateNames)) ?></span>)<? } ?><? if($artwork->Artist->DeathYear !== null){ ?> (<abbr>d.</abbr> <?= $artwork->Artist->DeathYear ?>)<? } ?><? if($isAdminView){ ?> (#<?= $artwork->Artist->ArtistId ?>)<? } ?>
 				</td>
 			</tr>
 			<tr>
-				<td>Year completed</td>
+				<td>Year completed:</td>
 				<td><? if($artwork->CompletedYear === null){ ?>Unknown<? }else{ ?><? if($artwork->CompletedYearIsCirca){ ?>Circa <? } ?><?= $artwork->CompletedYear ?><? } ?></td>
 			</tr>
 			<tr>
-				<td>Tags</td>
+				<td>Tags:</td>
 				<td>
 					<ul class="tags">
 						<? foreach($artwork->Tags as $tag){ ?>
@@ -113,21 +113,21 @@ catch(Exceptions\InvalidPermissionsException){
 				</td>
 			</tr>
 			<tr>
-				<td>Dimensions</td>
+				<td>Dimensions:</td>
 				<td><?= $artwork->Dimensions ?></td>
 			</tr>
 			<tr>
-				<td>Status</td>
+				<td>Status:</td>
 				<td><?= Template::ArtworkStatus(['artwork' => $artwork]) ?></td>
 			</tr>
 			<? if($isReviewerView){ ?>
 				<tr>
-					<td>Submitted by</td>
+					<td>Submitted by:</td>
 					<td><? if($artwork->Submitter === null){ ?>Anonymous<? }else{ ?><a href="mailto:<?= Formatter::EscapeHtml($artwork->Submitter->Email) ?>"><? if($artwork->Submitter->Name !== null){ ?> <?= Formatter::EscapeHtml($artwork->Submitter->Name) ?><? }else{ ?><?= Formatter::EscapeHtml($artwork->Submitter->Email) ?><? } ?></a><? } ?><? if($isAdminView && $artwork->Submitter !== null){ ?> (#<?= $artwork->Submitter->UserId ?>)<? } ?></td>
 				</tr>
 				<? if($artwork->Reviewer !== null){ ?>
 					<tr>
-						<td>Reviewed by</td>
+						<td>Reviewed by:</td>
 						<td><a href="mailto:<?= Formatter::EscapeHtml($artwork->Reviewer->Email) ?>"><? if($artwork->Reviewer->Name !== null){ ?> <?= Formatter::EscapeHtml($artwork->Reviewer->Name) ?><? }else{ ?><?= Formatter::EscapeHtml($artwork->Reviewer->Email) ?><? } ?></a><? if($isAdminView){ ?> (#<?= $artwork->Reviewer->UserId ?>)<? } ?></td>
 					</tr>
 				<? } ?>
diff --git a/www/css/artwork.css b/www/css/artwork.css
index 6c7eae2e..6ad938cc 100644
--- a/www/css/artwork.css
+++ b/www/css/artwork.css
@@ -243,24 +243,6 @@ label:has(select[name="artwork-status"]){
 	margin-right: auto;
 }
 
-.artwork-metadata{
-	margin-top: 1rem;
-}
-
-.artwork-metadata td:first-child{
-	font-weight: bold;
-	text-align: right;
-	white-space: nowrap;
-}
-
-.artwork-metadata td:last-child{
-	width: 100%;
-}
-
-.artwork-metadata td{
-	padding: 0.5rem;
-}
-
 main h1 ~ a[href^="/images/cover-uploads"],
 .artworks h1 ~ a[href^="/images/cover-uploads"],
 main section.narrow h1 ~ picture{
@@ -345,21 +327,6 @@ main section.narrow h1 ~ picture{
 	margin-top: 1rem;
 }
 
-@media(max-width: 500px){
-	.artwork-metadata td{
-		display: block;
-		padding: 0;
-	}
-
-	.artwork-metadata td:first-child{
-		text-align: left;
-	}
-
-	.artwork-metadata tr + tr td:first-child{
-		padding-top: 1rem;
-	}
-}
-
 @media(max-width: 400px){
 	/* Artist details */
 	form.browse-artwork > fieldset:nth-of-type(1){
diff --git a/www/css/core.css b/www/css/core.css
index e99bc205..3a938d66 100644
--- a/www/css/core.css
+++ b/www/css/core.css
@@ -3912,6 +3912,19 @@ nav.breadcrumbs + h1{
 		white-space: normal;
 		margin: 0 10px;
 	}
+
+	table.admin-table td{
+		display: block;
+		padding: 0;
+	}
+
+	table.admin-table td:first-child{
+		text-align: left;
+	}
+
+	table.admin-table tr + tr td:first-child{
+		padding-top: 1rem;
+	}
 }
 
 @media(max-width: 470px){