From d8f20438b2aae1bdcac43a1369444df6149e8b82 Mon Sep 17 00:00:00 2001
From: Lanre Adelowo <yo@lanre.wtf>
Date: Tue, 19 Jun 2018 22:56:09 +0100
Subject: [PATCH 1/2] Don't display buttons if there are no notices

---
 templates/admin/notice.tmpl | 50 +++++++++++++++++++------------------
 1 file changed, 26 insertions(+), 24 deletions(-)

diff --git a/templates/admin/notice.tmpl b/templates/admin/notice.tmpl
index 745433d180f44..1d649ddb8d8fc 100644
--- a/templates/admin/notice.tmpl
+++ b/templates/admin/notice.tmpl
@@ -34,33 +34,35 @@
 						</tr>
 					{{end}}
 				</tbody>
-				<tfoot class="full-width">
-					<tr>
-						<th></th>
-						<th colspan="5">
-							<div class="ui right">
-								<a class="ui red small button" href="{{AppSubUrl}}/admin/notices/empty">{{.i18n.Tr "admin.notices.delete_all"}}</a>
-							</div>
-							<div class="ui floating upward dropdown small button">
-								<span class="text">{{.i18n.Tr "admin.notices.actions"}}</span>
-								<div class="menu">
-									<div class="item select action" data-action="select-all">
-										{{.i18n.Tr "admin.notices.select_all"}}
+				{{ if gt (len .Notices) 0 }}
+					<tfoot class="full-width">
+							<tr>
+								<th></th>
+								<th colspan="5">
+									<div class="ui right">
+										<a class="ui red small button" href="{{AppSubUrl}}/admin/notices/empty">{{.i18n.Tr "admin.notices.delete_all"}}</a>
 									</div>
-									<div class="item select action" data-action="deselect-all">
-										{{.i18n.Tr "admin.notices.deselect_all"}}
+									<div class="ui floating upward dropdown small button">
+										<span class="text">{{.i18n.Tr "admin.notices.actions"}}</span>
+										<div class="menu">
+											<div class="item select action" data-action="select-all">
+												{{.i18n.Tr "admin.notices.select_all"}}
+											</div>
+											<div class="item select action" data-action="deselect-all">
+												{{.i18n.Tr "admin.notices.deselect_all"}}
+											</div>
+											<div class="item select action" data-action="inverse">
+												{{.i18n.Tr "admin.notices.inverse_selection"}}
+											</div>
+										</div>
 									</div>
-									<div class="item select action" data-action="inverse">
-										{{.i18n.Tr "admin.notices.inverse_selection"}}
+									<div class="ui small teal button" id="delete-selection" data-link="{{.Link}}/delete" data-redirect="{{.Link}}?page={{.Page.Current}}">
+										{{.i18n.Tr "admin.notices.delete_selected"}}
 									</div>
-								</div>
-							</div>
-							<div class="ui small teal button" id="delete-selection" data-link="{{.Link}}/delete" data-redirect="{{.Link}}?page={{.Page.Current}}">
-								{{.i18n.Tr "admin.notices.delete_selected"}}
-							</div>
-						</th>
-					</tr>
-				</tfoot>
+								</th>
+							</tr>
+					</tfoot>
+				{{ end }}
 			</table>
 		</div>
 

From d5752d4ba570f7013d5396e60b2e828692e79bf4 Mon Sep 17 00:00:00 2001
From: Lanre Adelowo <yo@lanre.wtf>
Date: Thu, 5 Jul 2018 15:48:07 +0100
Subject: [PATCH 2/2] remove redundant gt check

---
 templates/admin/notice.tmpl | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/templates/admin/notice.tmpl b/templates/admin/notice.tmpl
index 1d649ddb8d8fc..b1d886be42c9f 100644
--- a/templates/admin/notice.tmpl
+++ b/templates/admin/notice.tmpl
@@ -34,7 +34,7 @@
 						</tr>
 					{{end}}
 				</tbody>
-				{{ if gt (len .Notices) 0 }}
+				{{ if .Notices }}
 					<tfoot class="full-width">
 							<tr>
 								<th></th>