From 6e7893dbea2bfcbb7b1c49d5b09df780d2b4c367 Mon Sep 17 00:00:00 2001 From: joezsweet Date: Wed, 29 Mar 2017 17:25:25 +0200 Subject: [PATCH] hide chart area when mandatory filters are not set using this.chart_area.hide() instead of this.chart_area.toggle(true); --- frappe/public/js/frappe/views/reports/query_report.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/frappe/public/js/frappe/views/reports/query_report.js b/frappe/public/js/frappe/views/reports/query_report.js index ebba451136..75e464c83f 100644 --- a/frappe/public/js/frappe/views/reports/query_report.js +++ b/frappe/public/js/frappe/views/reports/query_report.js @@ -412,7 +412,7 @@ frappe.views.QueryReport = Class.extend({ if(v) filters[f.df.fieldname] = v; }) if(raise && mandatory_fields.length) { - this.chart_area.toggle(true); + this.chart_area.hide(); this.wrapper.find(".waiting-area").empty().toggle(false); this.wrapper.find(".no-report-area").html(__("Please set filters")).toggle(true); if(raise) {