From 2abcd8ee474e0f98c54af8bfe6fb40979f3b18ba Mon Sep 17 00:00:00 2001
From: rubenwardy <rw@rubenwardy.com>
Date: Sat, 26 May 2018 01:29:18 +0100
Subject: [PATCH] Add screenshot recommendation to package alert banner

---
 app/templates/packages/view.html | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/app/templates/packages/view.html b/app/templates/packages/view.html
index e0f03c01..99b45da5 100644
--- a/app/templates/packages/view.html
+++ b/app/templates/packages/view.html
@@ -20,17 +20,21 @@
 				{% else %}
 					A release is required before this package can be approved.
 				{% endif %}
-			{% elif not package.getDownloadRelease() %}
-				Please wait for the release to be approved.
 			{% else %}
-				{% if package.checkPerm(current_user, "APPROVE_NEW") %}
+				{% if package.screenshots.count() == 0 %}
+					<b>You should add at least one screenshot, but this isn't required.</b><br />
+				{% endif %}
+
+				{% if not package.getDownloadRelease() %}
+					Please wait for the release to be approved.
+				{% elif package.checkPerm(current_user, "APPROVE_NEW") %}
 					You can now approve this package if you're ready.
 					<form method="post" action="{{ package.getApproveURL() }}">
 						<input type="hidden" name="csrf_token" value="{{ csrf_token() }}" />
 						<input type="submit" value="Approve" />
 					</form>
 				{% else %}
-					This package needs to be approved before it can be found.
+					Please wait for the package to be approved.
 				{% endif %}
 			{% endif %}
 			<div style="clear: both;"></div>