From 6c5d4cbb22bc2d8eb9d0b7d3541affd7f9effa18 Mon Sep 17 00:00:00 2001
From: Faris Ansari <netchamp.faris@gmail.com>
Date: Mon, 14 Sep 2020 15:18:59 +0530
Subject: [PATCH] fix: Better implementation for refresh on toggle sidebar

---
 frappe/public/js/frappe/views/reports/report_view.js | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/frappe/public/js/frappe/views/reports/report_view.js b/frappe/public/js/frappe/views/reports/report_view.js
index aec24d9d13..05e8e75eeb 100644
--- a/frappe/public/js/frappe/views/reports/report_view.js
+++ b/frappe/public/js/frappe/views/reports/report_view.js
@@ -48,10 +48,12 @@ frappe.views.ReportView = class ReportView extends frappe.views.ListView {
 	setup_view() {
 		this.setup_columns();
 		super.setup_new_doc_event();
+	}
+
+	toggle_side_bar() {
+		super.toggle_side_bar();
 		// refresh datatable when sidebar is toggled to accomodate extra space
-		$(document.body)
-			.off('toggleListSidebar.report_view')
-			.on('toggleListSidebar.report_view', () => this.render(true));
+		this.render(true);
 	}
 
 	setup_result_area() {