From d24f024cca2b3cf1c2b3f2b7fe2f01cf595ea689 Mon Sep 17 00:00:00 2001
From: rubenwardy <rw@rubenwardy.com>
Date: Thu, 3 Dec 2020 23:36:24 +0000
Subject: [PATCH] Move screenshots to top of approval queue page

---
 app/templates/todo/list.html | 65 ++++++++++++++++++------------------
 1 file changed, 32 insertions(+), 33 deletions(-)

diff --git a/app/templates/todo/list.html b/app/templates/todo/list.html
index 30b90bbe..0b1b4cec 100644
--- a/app/templates/todo/list.html
+++ b/app/templates/todo/list.html
@@ -6,6 +6,38 @@
 
 {% block content %}
 	<h2 class="mb-4">Approval Queue</h2>
+	{% if canApproveScn and screenshots %}
+		<div class="card my-4">
+			<h3 class="card-header">Screenshots
+				<form class="float-right"  method="post" action="{{ url_for('todo.view') }}">
+					<input type="hidden" name="csrf_token" value="{{ csrf_token() }}" />
+					<input type="hidden" name="action" value="screenshots_approve_all" />
+					<input class="btn btn-sm btn-primary" type="submit" value="Approve All" />
+				</form>
+			</h3>
+			<ul class="card-body d-flex p-0 flex-row flex-wrap justify-content-start align-content-start p-4">
+				{% for s in screenshots %}
+					<li class="packagetile flex-fill"><a href="{{ s.getEditURL() }}"
+						style="background-image: url({{ s.getThumbnailURL(3) or '/static/placeholder.png' }});">
+						<div class="packagegridscrub"></div>
+						<div class="packagegridinfo">
+							<h3>
+								{{ s.title }}
+								<br />
+								<small>{{ s.package.title }} by {{ s.package.author.display_name }}</small>
+							</h3>
+							<p></p>
+						</div>
+					</a></li>
+				{% else %}
+					<li><i>No screenshots need reviewing.</i></li>
+				{% endfor %}
+				{% for i in range(4) %}
+					<li class="packagetile flex-fill"></li>
+				{% endfor %}
+			</ul>
+		</div>
+	{% endif %}
 
 	<div class="row">
 		{% if canApproveNew and (packages or wip_packages) %}
@@ -75,39 +107,6 @@
 		{% endif %}
 	</div>
 
-	{% if canApproveScn and screenshots %}
-		<div class="card my-4">
-			<h3 class="card-header">Screenshots
-				<form class="float-right"  method="post" action="{{ url_for('todo.view') }}">
-					<input type="hidden" name="csrf_token" value="{{ csrf_token() }}" />
-					<input type="hidden" name="action" value="screenshots_approve_all" />
-					<input class="btn btn-sm btn-primary" type="submit" value="Approve All" />
-				</form>
-			</h3>
-			<ul class="card-body d-flex p-0 flex-row flex-wrap justify-content-start align-content-start p-4">
-				{% for s in screenshots %}
-					<li class="packagetile flex-fill"><a href="{{ s.getEditURL() }}"
-						style="background-image: url({{ s.getThumbnailURL(3) or '/static/placeholder.png' }});">
-						<div class="packagegridscrub"></div>
-						<div class="packagegridinfo">
-							<h3>
-								{{ s.title }}
-								<br />
-								<small>{{ s.package.title }} by {{ s.package.author.display_name }}</small>
-							</h3>
-							<p></p>
-						</div>
-					</a></li>
-				{% else %}
-					<li><i>No screenshots need reviewing.</i></li>
-				{% endfor %}
-				{% for i in range(4) %}
-					<li class="packagetile flex-fill"></li>
-				{% endfor %}
-			</ul>
-		</div>
-	{% endif %}
-
 	{% if not (packages or screenshots or releases) %}
 		<p>
 			<i>All done!</i>