From 921fdffa72e0a5af6f3c0246c8a061be314ca0e2 Mon Sep 17 00:00:00 2001
From: Rohan Bansal <rohan@bloomstack.com>
Date: Tue, 29 Dec 2020 16:11:03 +0530
Subject: [PATCH] fix: show remove button conditionally

---
 frappe/core/page/background_jobs/background_jobs.js        | 7 +++++++
 .../core/page/background_jobs/background_jobs_outer.html   | 6 ++----
 2 files changed, 9 insertions(+), 4 deletions(-)

diff --git a/frappe/core/page/background_jobs/background_jobs.js b/frappe/core/page/background_jobs/background_jobs.js
index 75d97b9411..397f097472 100644
--- a/frappe/core/page/background_jobs/background_jobs.js
+++ b/frappe/core/page/background_jobs/background_jobs.js
@@ -40,6 +40,13 @@ frappe.pages.background_jobs.refresh_jobs = function() {
 			page.body.find('.list-jobs').remove();
 			$(frappe.render_template('background_jobs', { jobs: r.message || [] })).appendTo(page.content);
 
+			let $remove_failed_btn = page.body.find('.remove-failed');
+			if (r.message && r.message.length > 0) {
+				$remove_failed_btn.show();
+			} else {
+				$remove_failed_btn.hide();
+			}
+
 			if (frappe.get_route()[0] === 'background_jobs') {
 				frappe.background_jobs_timeout = setTimeout(frappe.pages.background_jobs.refresh_jobs, 2000);
 			}
diff --git a/frappe/core/page/background_jobs/background_jobs_outer.html b/frappe/core/page/background_jobs/background_jobs_outer.html
index 8c79806005..f57339735a 100644
--- a/frappe/core/page/background_jobs/background_jobs_outer.html
+++ b/frappe/core/page/background_jobs/background_jobs_outer.html
@@ -6,10 +6,8 @@
 			</label>
 		</div>
 		<div>
-			<button type="button" class="btn btn-primary remove-failed"> {{ __("Remove failed jobs") }}</button>
+			<button type="button" class="btn btn-primary remove-failed" hidden> {{ __("Remove failed jobs") }}</button>
 		</div>
 	</p>
-	<div class="table-area">
-
-	</div>
+	<div class="table-area"></div>
 </div>
\ No newline at end of file